In the new commit, I removed it and said so in the message. It's
unnecessary--I tested.

Keertan Kini
Class of 2016
Massachusetts Institute of Technology
Phone: (219)-308-8296


On Mon, Nov 18, 2013 at 6:02 PM, Jos <notifications@github.com> wrote:

> In appinventor/lib/blockly/src/core/flyout.js:
>
> > @@ -355,7 +355,8 @@ Blockly.Flyout.prototype.show = function(xmlList) {
> >        // There is no good way to handle comment bubbles inside the flyout.
> >        // Blocks shouldn't come with predefined comments, but someone will
> >        // try this, I'm sure.  Kill the comment.
> > -      Blockly.Comment && child.setCommentText(null);
> > +
> > +      Blockly.Comment && child.setCommentText(null) && child.setDoItBubbleText(null);
>
> Hi, so you are the one that wrote the code; you are the one that has to
> tell us if the line is needed or not. We cannot do a review if you don't
> know if your code works or there's lines that might or might not be needed.
> Please have another look and let us know.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mit-cml/appinventor-sources/pull/68/files#r7742988>
> .
>