Menus plugin breaks the Settings on Craft 2.5.2715 about craft-menus HOT 7 OPEN

Same for me. JS error in the console:

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

from craft-menus.

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.

from craft-menus.

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

from craft-menus.

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 #12 (comment).

from craft-menus.

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

from craft-menus.

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.

from craft-menus.

Nothing has happened, so I opened a PR against this plugin here: #15

from craft-menus.