History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: SM-63
Type: Improvement Improvement
Status: Open Open
Priority: Major Major
Assignee: Matt Raible
Reporter: Stephen Lynch
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Struts Menu

Allow Menus to have onmouseovers etc. just as menu items do

Created: 19/Oct/06 09:54 AM   Updated: 26/Jan/07 11:23 PM
Component/s: Displayers
Affects Version/s: 2.4.1
Fix Version/s: None


 Description  « Hide
I need to be able to have onmouseovers etc.. on menus as well as menuitems. The patch below applies the same logic as menuitems.

Is there a reason why this is not done at the moment? If not, I'd appreciate it if you could apply this improvement.

### Eclipse Workspace Patch 1.0
#P navigator
Index: src/java/net/sf/navigator/displayer/ListMenuDisplayer.java
===================================================================
RCS file: /cvsroot/struts-menu/navigator/src/java/net/sf/navigator/displayer/ListMenuDisplayer.java,v
retrieving revision 1.16
diff -u -r1.16 ListMenuDisplayer.java
--- src/java/net/sf/navigator/displayer/ListMenuDisplayer.java 9 Jul 2006 08:08:09 -0000 1.16
+++ src/java/net/sf/navigator/displayer/ListMenuDisplayer.java 19 Oct 2006 14:48:11 -0000
@@ -66,7 +66,8 @@
                 out.println(displayStrings.getMessage("lmd.menu.actuator.top",
                         domId,
                         getMessage(menu.getTitle()),
- menuClass));
+ menuClass,
+ getExtra(menu)));
             }
             
for (int i = 0; i < components.length; i++) {
Index: src/java/net/sf/navigator/displayer/DisplayerStrings.properties
===================================================================
RCS file: /cvsroot/struts-menu/navigator/src/java/net/sf/navigator/displayer/DisplayerStrings.properties,v
retrieving revision 1.12
diff -u -r1.12 DisplayerStrings.properties
--- src/java/net/sf/navigator/displayer/DisplayerStrings.properties 17 Oct 2006 22:31:41 -0000 1.12
+++ src/java/net/sf/navigator/displayer/DisplayerStrings.properties 19 Oct 2006 14:48:10 -0000
@@ -41,7 +41,7 @@
 lmd.menu.top=\t<li class="menubar">\n
 lmd.menu.bottom=\t</li>\n
 lmd.menu.actuator.link=\t<a href="#" id="{0}Actuator" class="actuator">&nbsp;</a><a href="{3}" title="{1}" class="base">{1}</a>\n\t\t<ul id="{0}Menu" class="{2}">\n
-lmd.menu.actuator.top=\t<a href="#" id="{0}Actuator" class="actuator">{1}</a>\n\t\t<ul id="{0}Menu" class="{2}">\n
+lmd.menu.actuator.top=\t<a href="#" id="{0}Actuator" class="actuator" {3} >{1}</a>\n\t\t<ul id="{0}Menu" class="{2}">\n
 lmd.menu.actuator.bottom=\t\t</ul>\n\t</li>\n
 lmd.menu.item=\t<li><a href="{0}" title="{1}"{2}>{3}</a></li>
 lmd.menu.standalone=\t<a href="{0}" title="{1}" class="standalone"{2}>{3}</a>


 All   Comments   Change History   FishEye      Sort Order:
Matt Raible - 28/Oct/06 09:56 PM
This was fixed fro the CSSListMenuDisplayer in 2.4.2 (SM-53) - I'll make sure it happens for the ListMenuDisplayer as part of the next release.