i would like to have a menu, which produces html-output more like the default menu, eg ul/li lists instead of tables,
and i would like to have the title as text included in the link. maybe with an option in the admin backend (if/)where to place it.

Possible options for admins:
Do not show title
Show image left/title right
Show title left/image right
Show image top/title bottom
Show title top/image bottom

maybe an extra option for horiz/vert+left/right alignment could be useful. or just add a css class for styling.

please keep me informed!
nice module!

Sam

Comments

pobster’s picture

pobster commented

They're only output as tables by default because they're easiest to display images stitched together with no gaps. In the settings though it's possible to specify list/ table/ css for output (for the block only I seem to remember, although it's easy to add them to the filter... Maybe I already did? I can't remember...) Anyways, point being that the classes output allows you to theme and style your menus however you like? Is this what you're looking for? Hmmm... It's entirely possible that I released that only for 4.7 and not 5.x as I don't use 5.x sorry!

Dunno about the other stuff... I'm probably going to write a bit of a rewriting session for a lot of my modules and I'm sure this one will change a LOT as there's a lot of things I don't like about it. Mind you... I don't actually use it myself (!) so maybe it won't be first on the list.

Thanks,

Pobster
PS. I *love* Austria - it's beautiful!

johnphethean’s picture

johnphethean commented

Hi I have managed to do this by editing the imagemenu.module code.
Line 456 (version 6.x-1.1) -
.'" name="imagemenu_'. $prefix .'_'. $item['mid'] .'"'. $script .' /><br />'.check_plain($item['title']).'</a>'. $menu .'</li>';
This displays the title directly below the image. Use your own code where I've put <br />'.check_plain($item['title']).' to place the text relative to the image.

Maybe someone could build in a config option and switch case for the different layout options suggested above? This is definitely a feature i would use.

marcus_clements’s picture

marcus_clements commented
Status: Active » Needs work

If you would like to post a patch of your proposed changes against the latest 6.x version I'll be happy to review and include.

johnphethean’s picture

johnphethean commented

Sorry rayvaughn I'm not actually very good at this, just know a bit of code etc. I wouldn't even know where to start to create a patch! Also very busy at the moment but might find the time sometime to look into it.

marcus_clements’s picture

marcus_clements commented
Status: Needs work » Closed (fixed)

Fair enough. A few people have requested this in the issue queue so I'm going to close this issue and probably incorporate the functionality into the next release.