Hi,
here's a small patch I often use to have a submenu open if the current page belongs to it (nav-bar style).
Included in the patch is the corrected javascript and mdofications to the settings page to enable it, or not.
Could be useful I think.

CommentFileSizeAuthor
#7 openatstart.diff2.02 KBjdanthinne
#3 openmenuatstart-7.x-2.0-beta2.patch2.42 KBgarethsprice
#2 openmenuatstart.patch2.3 KBaxe312
openedmenu.patch1.81 KBjdanthinne

Comments

axe312’s picture

axe312 commented

Hahaha! Iam so happy that I found that before I started to write the same ^^

Pls add it to the core!!!

axe312’s picture

axe312 commented
StatusFileSize
new openmenuatstart.patch2.3 KB

Patch of #1 failed. I applied it manually and fixed a bug.

Heres the new one. Works with the newest 6.x-2.x-dev version! (maybe also works on stable version)

garethsprice’s picture

garethsprice commented
Version: 6.x-2.x-dev » 7.x-2.0-beta2
StatusFileSize
new openmenuatstart-7.x-2.0-beta2.patch2.42 KB

Here's the patch rolled against 7.x-2.0-beta2. Great work, thanks!!

Barfly’s picture

Barfly commented

Hi,

I tried the patch "openmenuatstart-7.x-2.0-beta2.patch" but it doesn't work for me.

Could you please upload your "nice_menu.module" and "nice_menu.js" allready patched please :)

Thanks

Barfly’s picture

Barfly commented

My bad, it 's working but only for "core" menu (navigation, main ...)

it doesn't work for "custom" menu.
I tried to look @ the code but i didn't understand what it does and why custom menu had not the "submenu open at start".

jdanthinne, do you have an answer?

Thank you.

vordude’s picture

vordude commented
Status: Needs review » Needs work

Not particularly interested in it myself. If this is solid, I'd consider adding it to the module. I don't plan on spending much of my time on it.

jdanthinne’s picture

jdanthinne commented
StatusFileSize
new openatstart.diff2.02 KB

Here's my lastest diff file.
It's working fine for both 6.x and 7.x.

@Barfly : the patch is working on every menu on my side. What are you calling "custom" menu?
@vordude : sad. I think it's "solid", and I use it on 1 website out of 2, very useful for two levels navigation bar. What needs "work"?

tomlee’s picture

tomlee commented

openmenuatstart-7.x-2.0-beta2.patch works for me but hunk 3 in nice_menus.module should be removed, otherwise it failed.

Please add it to the core!

astonvictor’s picture

astonvictor commented
Issue summary: View changes
Status: Needs work » Closed (outdated)

I'm closing it because the issue was created a long time ago without any further steps.

if you still need it then raise a new one.
thanks