hYes---the child.setCommentText(null) will return undefined, which will
short-circuit the comment. I recommitted the deletion of that extra test
because I tried a version with/without the child.setDoItBubbleText(null),
and it made no difference when actually running.

If that change didn't propagate up to the pull request, I can send another.
Unless you think that the method call should be included, in which case, I
can just put it above the && line

Keertan Kini
Class of 2016
Massachusetts Institute of Technology
Phone: (219)-308-8296


On Sat, Nov 9, 2013 at 6:30 PM, Jeffrey Schiller
<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);
>
> My point is that child.setDoItBubbleText(null) will *never* get invoked.
> Do you know why?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mit-cml/appinventor-sources/pull/68/files#r7547154>
> .
>