O use the menu Icon module and when i add icons to menu items thata are embeded in a mega menu, the icons disappear because the css classes are completely overridden.

It could be nice if both modules might be compatible t make megamenus such as the picture on the project page.

Thank you

CommentFileSizeAuthor
#3 megamenu.1049060.menu_icons.patch1.09 KBAnonymous (not verified)

Comments

Roulion’s picture

The module i'm talking about
http://drupal.org/project/menu_icons

Anonymous’s picture

It looks like the menu and link list items need the class .menu-[mlid] for the menu_icons stylesheet to work.

Anonymous’s picture

Component: User interface » Code
Assigned: Unassigned »
Status: Active » Needs review
StatusFileSize
new1.09 KB

Try this patch. It puts in menu-[mlid[ classes on every list item, which is what menu_icons expects.

Anonymous’s picture

Committed patch to integrate this module with menu_icons in #497458

Anonymous’s picture

I thought I would download menu_icons and try this out myself. It appears that there is more needed than just the right CSS classes. When I look at the page source, the menu_icons generated CSS is blank.

Anonymous’s picture

Looks like there might be a bug with menu_icons where the CSS file isn't generated unless you upload an icon. I uploaded an icon and it looks like things *might* work except the menu_icons CSS is loaded before theme CSS files and skin styles can overwrite the menu_icon rules.

I might could fix this by putting the .menu-# class on the anchors in addition to the list items.

Anonymous’s picture

Status: Needs review » Fixed

Committed the change to move .menu-[mlid] classes to the anchor attributes to 6.x-2.x-dev in #497580

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nithinkolekar’s picture

Version: 6.x-2.x-dev » 7.x-1.x-dev
Issue summary: View changes
Status: Closed (fixed) » Active

changing to active for 7.x version (tested with Menu Icons 7.x-3.0-beta)