
|
If you were logged in you would be able to see more operations.
|
|
|
Struts Menu
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
|
|
|
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"> </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>
|
|
Description
|
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"> </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>
|
Show » |
|
SM-53) - I'll make sure it happens for the ListMenuDisplayer as part of the next release.