Class added with menu attributes on mega menu not appearing [#2053623]

When I set class to "button" with menu attribute it does not appear in the front end of my site, a change is observed in the admin panel
http://screencast.com/t/zBrx6CXQIIOG
but not in the actual frontend html of my website which is obviously where I really need it.

I am using mega menu 7.x-1.x-dev which now only has a dev release in 7

Comment	File	Size	Author
#5	tb_megamenu-Classes-not-appearing-1691632-4.patch	598 bytes	SwapS
#5

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

silurius CreditAttribution: silurius commented 6 August 2013 at 23:42

I don't think this is related to megamenu, because I'm seeing the same issue as well here. Classes added to the menu link are simply not applied outside of the menu admin screen.

Comment #2

Zero4ffect CreditAttribution: Zero4ffect commented 21 August 2013 at 19:11

Silurius, I'm might assume that you say "I don't think this is related to megamenu" because you're not using MegaMenu? However, I try not to assume anything. =)

Comment #3

thinkdrupal CreditAttribution: thinkdrupal commented 12 October 2013 at 00:59

Which Theme are you using? The theme is probably the issue. The marinelli theme caused the issue for me. Added some code in function marinelli_mega_menu. This is located in theme.inc.

Comment #4

hwasem CreditAttribution: hwasem commented 31 January 2014 at 18:19

I'm using a subtheme of Adaptive Themes with Tb_megamenu and I'm not seeing classes, either. This issue is a bit old. I'm curious if you guys have come up with anything in the interim.

Comment #5

SwapS CreditAttribution: SwapS commented 23 February 2014 at 13:02

Issue summary:	View changes
Status:	Active	» Needs review

File	Size
tb_megamenu-Classes-not-appearing-1691632-4.patch	598 bytes

For #4, Reason for the issue : tb_megamenu is nt checking if classes has been added by any other module. Issue isn't with menu_attributes module but it is with tb_megamenu module

Attached patch should help. :)

Comment #6

23 February 2014 at 13:03

Status:

Needs review

» Needs work

The last submitted patch, 5: tb_megamenu-Classes-not-appearing-1691632-4.patch, failed testing.

Comment #7

SwapS CreditAttribution: SwapS commented 23 February 2014 at 16:06

Status:

Needs work

» Closed (fixed)

1 file was hidden/shown/deleted

File	Size
tb_megamenu-Classes-not-appearing-1691632-4.patch	598 bytes

Seems wrong place for patch. Bot tried applying this patch for menu_attribute, but this will be applicable for tb_megamenu module.
There is no issue with menu_attribute module.
@hwasem : patch in #5 should solve your problem.

Comment #8

hwasem CreditAttribution: hwasem commented 25 February 2014 at 22:44

Thanks for this, SwapS. Do you have the issue the patch came from so I can review and better understand it? I can't see to find it. It tried issue 1691632 as list on the patch, but that was for Facebook Share module. It is giving a white screen after the patch is applied.

Thanks again!