Huh, I'm not sure why I didn't see that CL.  I'll have to watch more
carefully.  Could you please change it to:

BlocklyApps.DISPLAY_NAV = BlocklyApps.getStringParamFromUrl('menu',
'true') === 'true';

Thanks.

Ellen



On Fri, Sep 6, 2013 at 9:20 AM, brokenairplane <notifications@github.com>wrote:

> In static/common.js:
>
> >   */
> >  BlocklyApps.DISPLAY_NAV = BlocklyApps.getStringParamFromUrl('menu', 'true');
> > -BlocklyApps.DISPLAY_NAV = BlocklyApps.DISPLAY_NAV === 'true';  // Convert to bool.
> > +BlocklyApps.DISPLAY_NAV = BlocklyApps.DISPLAY_NAV === 'true';
>
> I synced this from codedotorg and I don't know what is the intent behind
> it.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/code-dot-org/blockly/pull/13/files#r6212162>
> .
>