Menus plugin breaks the Settings on Craft 2.5.2715

[echantigny]

I'm testing the 2.5 beta and noticed that the plugin is break the settings dialog boxes by making the buttons invisible to the user. If I disable the plugin, everything goes back to normal.

[ryansnowden]

Same for me. JS error in the console:

Uncaught SyntaxError: Invalid regular expression: /(^|.)Garnish670893443(.|$)/: Stack Overflow Uncaught RangeError: Maximum call stack size exceeded

[ryansnowden]

This is due to some css in the settings.html file

{% set hideSave %}
.buttons {
    display:none;
}

#settings-newmenucontainer.buttons {
    display:block;
}
{% endset %}

{# {% includeCss hideSave %} #}

It seems unnecessary so I just commented it out.

[echantigny]

If its a piece of required css for some reason in the plugin, we should find a way to be more precise than just .buttons. I'll investigate a bit more if I have time. Thanks

[ryansnowden]

Saw no reason for it, besides, perhaps legacy.

On 9 Nov 2015, at 10:23 PM, echantigny [email protected] wrote:

If its a piece of required css for some reason in the plugin, we should find a way to be more precise than just .buttons. I'll investigate a bit more if I have time. Thanks

— Reply to this email directly or view it on GitHub https://github.com/familiar-studio/craft-menus/issues/12#issuecomment-155077284.

[auralon]

Would be nice to see the necessary patch issued as a pull request, merged and then tagged and released.

[keithmancuso]

Hey guys, sorry for the radio silence on this. We're been hard at work on other projects but im working on some updates to this as we speak.

This issue was because we were trying to hide the default "Save" button on the settings page since there are not settings to save there... but it stopped working in 2.5 anyways.

Should be able to post an update this week and sorry again for not being responsive sooner.

[boborchard]

Nothing has happened, so I opened a PR against this plugin here: https://github.com/familiar-studio/craft-menus/pull/15