Hi,

I have installed the module, and I see that it works in normal case (e.g. on Bartik with its default main menu display, although that broke the design a bit). However, it does not work with Supefish as a drop down menu.
Please check it.

Thanks,
Perry.

CommentFileSizeAuthor
#17 buggy.PNG34 KBIzoman
#7 Welcome to DruShu | DruShu.jpg119.77 KBdshumaker

Comments

markhalliwell’s picture

Does it at least attach the view with the menu item in Superfish? Please see the note on this project page, you may need to open the issue with the maintainers of Superfish.

prabhatjn’s picture

Status: Active » Postponed (maintainer needs more info)

I will look into it as I will get time.

Thanks,
Perry.

markhalliwell’s picture

Assigned: Unassigned » markhalliwell
Category: bug » support
Priority: Major » Minor
markhalliwell’s picture

Priority: Minor » Normal
Status: Postponed (maintainer needs more info) » Needs work

Will keep this open until I have some time to do some testing with this module and submit a patch.

prezaeis’s picture

any updates on this?

i attached a views to a menu item aswel and its not doing anything, although on the menu page it states that a view is attached to the menu and no error is given

c0ldfury’s picture

It doesn't work because Views adds several wrapper divs around the html list. This module needs to fish out the list elements from the view and only add them, instead of adding the entire div.

dshumaker’s picture

Version: 7.x-1.x-dev » 7.x-1.3
StatusFileSize
new119.77 KB

Does anyone have an example of this module working with Superfish? I installed it on a generic drupal site and the view is being stripped out by something (superfish js I guess).

However I don't really understand the "must be included" statement on the project page:

If the contributed theme or module implements drop down menus, using CSS or JavaScript, a new child selector (li > .view) must be included. The view can then be styled appropriately from there.

I don't see how just adding a .view css class will stop superfish from stripping content out.

I tried formatting the view in table, unformatted, and html list format and superfish doesn't like any of those formats.

Perhaps If somebody understands the "child selector must be included" note above and has implemented it on a site then they could comment here (or document it somewhere) and help us out by saying exactly what needs to happen.

Superfish is so plug and play I'd hate to have to scrap it and go write my own css on hover menu system just to handle the few menu items that will use the menu_views.

Many thanks.
-Dan

c0ldfury’s picture

I switched to menu minipanels until this is fixed. Does superfish like effects with views in a menu. The issue is actually with the superfish module iirc.

pedrorocha’s picture

I tried hard now to make it work, but after some time, i realized that Superfish(and probably every other JS menu modules) aren't compatible because he doesn't use "theme_link" to create his items, as they need extreme control over the output to integrate with superfish plugin(or the respective plugins, with the other JS menu modules), so it couldn't be overridden by "menu_views_link".

I don't know the solution, but for me, will be a manually created menu, that will work...

markhalliwell’s picture

Version: 7.x-1.3 » 7.x-2.x-dev
Status: Needs work » Needs review

I went ahead and committed changes to the 2.x branch:
http://drupalcode.org/project/menu_views.git/commit/f1df661

This is dependent upon the patch in this Superfish issue becoming committed and released: #1534734-1: Theme function for menu items.

Also referencing: #1617802: Create a release from 7.x-1.x-dev to support Menu Views

markhalliwell’s picture

Status: Needs review » Fixed
Steve Polito Design’s picture

I have the most updated Superfish(7.x-1.9-beta4) and Menu Views(7.x-2.0-rc2+3-dev) modules installed and it's still not working. What am I doing wrong?

markhalliwell’s picture

Steve Polito Design’s picture

Thank you! Using the Dev version worked perfectly.

Status: Fixed » Closed (fixed)

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

IamOnStage’s picture

I have installed the most up to date versions of Superfish and Menu Views.
Menu views 2.2
Superfish 1.9+25-dev

And this is not working with Superfish?

Izoman’s picture

Issue summary: View changes
StatusFileSize
new34 KB

It doesn't work on latest version of menu views Version: 7.x-2.2 and superfish Version: 7.x-1.9.
To test, I also tried showing main menu in standard drupal block and it worked.

So what's going on?

Izoman’s picture

Status: Closed (fixed) » Active
Izoman’s picture

I simply remade Superfish block using css together with standard drupal block.
If anyone needs css to make Superfish like block, just pm me.

markhalliwell’s picture

Assigned: markhalliwell » Unassigned
Status: Active » Closed (fixed)

1) don't re-open closed issues and 2) I definitely don't have time to sit here and debug this module for you with other modules.