Quite a minor thing I know but should be set right.
Image menus appear as a sub-link of Build>Menus but not on the actual page.

Comments

RedTop’s picture

It would be nice for the options to appear on the regular menu page itself (as happens with menu_icon module).

marcus_clements’s picture

Category: bug » feature
wickedskaman’s picture

Priority: Minor » Major

I'd like to add that it would be nice for these menus to more completely integrate with Drupal's menu system so they are exposed to modules like Menu Block and Site Map.

I don't know if I should make this it's own feature request or not. What does everyone else think?

RedTop’s picture

I concur... it would be a very nice option, especially now D7 has image handling (imagecache) built in.

wickedskaman’s picture

I have a suggestion along those lines... when creating an image menu would it be counter intuitive for the module to simultaneously create a database entry as a regular menu entry as well using the imagemenu data provided? So there would be menu data in the imagemenu db table as well as the system menu table. A positive here is that when uninstalling imagemenu for whatever reason your menu structure is still in the system in case you still need those menu structures in regular text. I think this would create less work for integrating the module sooner rather than later as well. I think it's the quickest and least messy approach. I found myself doing this as a work-around manually anyway so it would be nice for the module to sync and automate the work for us. That way deleting in imagemenu cascade deletes the system entry as well when we specifically delete items.

The dirty work is probably in upgrading previous imagemenu versions for people, namely taking their existing imagemenus and mirroring them in the default menu system if they opt into the whole mirroring thing when upgrading to the new version.

Thoughts?

RedTop’s picture

My suggestion would be that the module (in D7 at least) adds options to the existing menu system rather than offering a new type of menu as is the case now. It would create a more 'integrated' experience.

Menu settings could allow administrators to configure which menus are image-enabled and which imagecache setting should be used for each image-enabled menu. Menu-items in image-enabled menus would offer the option to add an image and a mouse-over image in addition to the text or replacing the text altogether (immediately addressing http://drupal.org/node/786204). Drupal7 is pretty savvy with images now that imagecache has been built in!

Database-wise... maybe add three extra columns to the drupal_menu_links table, two pointing to the relevant imachecache entry/id (called 'imagepath' and 'mouseover' now) and one to save whether text should be shown or not? When uninstalling it would not be that hard to simply delete the columns from the table leaving the 'standard' menu structure in place?

I'm not well-versed in database design but duplicate information usually means there's an easier and more manageable way to do things.

marcus_clements’s picture

I think the Drupal 7 version of this module will indeed be an opportunity for a complete rewrite. I completely agree that much more integrated approach will be the way to go.
@RedTop - please repost your suggestions here http://drupal.org/node/990480- http://drupal.org/node/1003072 let's get a consensus on functionality for the new version.

RedTop’s picture

The link is broken... http://drupal.org/node/990480 (without the hyphen) points to an issue about a D7 port of power import.

Would you like me to start a new issue?

marcus_clements’s picture

Oh dear I obviously wasn't concentrating when I wrote that post.

Please add suggestions and hints for the D7 version of Imagemenu here:
http://drupal.org/node/1003072

pobster’s picture

Issue summary: View changes
Status: Active » Closed (outdated)

Closing as D6.x is now unsupported.