[ICE-1103] MenuItem: link is still rendered when component disabled - ICEsoft JIRA Issue Tracker

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Cannot Reproduce
Affects Version/s: 1.5.1
Fix Version/s: 1.6DR#1, 1.6
Component/s: ICE-Components
Labels:
None
Environment:
Operating System: Windows XP
Platform: PC

Description

noted in the forum: http://www.icefaces.org/JForum/posts/list/0/3143.page#15080

If a MenuItem is disabled, it seems that the bound link attribute is still
rendered and clickable.

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

Hide
simpleMenu.zip

23/Jan/07 12:43 PM

29 kB

Greg McCleary
simpleMenu/build.properties 0.2 kB

simpleMenu/build.xml 9 kB

simpleMenu/tom.bat 0.0 kB

simpleMenu/META-INF/context.xml 0.1 kB

simpleMenu/src/.project 0.4 kB

simpleMenu/src/.classpath 0.5 kB

simpleMenu/src/test/MenuBarBean.java 5 kB

simpleMenu/src/test/MenuBarBean.class 2 kB

simpleMenu/src/test/TestBean.java 2 kB

simpleMenu/.../DefaultCollectionItemsArray.java 0.8 kB

simpleMenu/web/index.jsp 0.1 kB

simpleMenu/web/menuBar.jspx 2 kB

simpleMenu/web/menu.html 2 kB

simpleMenu/web/WEB-INF/web.xml 4 kB

simpleMenu/web/WEB-INF/faces-config.xml 1 kB

simpleMenu/web/images/menu/zoomout.gif 0.6 kB

simpleMenu/web/.../menu/actualsize.gif 0.6 kB

simpleMenu/web/images/menu/arrow.gif 0.2 kB

simpleMenu/web/images/menu/fitinpage.gif 0.4 kB

simpleMenu/web/.../menu/fitinwindow.gif 0.4 kB

simpleMenu/web/images/menu/open.gif 0.6 kB

simpleMenu/web/images/menu/print.gif 0.6 kB

simpleMenu/web/images/menu/recent.gif 0.6 kB

simpleMenu/web/images/menu/save.gif 0.6 kB

simpleMenu/web/images/menu/zoomin.gif 0.6 kB

simpleMenu/web/css/showcase_style.css 18 kB

simpleMenu/web/menuBar_full.jspx 7 kB

simpleMenu/web/test.jspx 0.8 kB

simpleMenu/web/menuBar_test.jspx 1 kB

simpleMenu/web/menuBar_horizontal.jspx 7 kB

Showing 30 of 32 items Download Zip
Show

simpleMenu.zip

23/Jan/07 12:43 PM

29 kB

Greg McCleary

Activity

Ascending order - Click to sort in descending order

Philip Breau created issue - 13/Dec/06 3:53 PM

Hide

Permalink

Greg McCleary added a comment - 15/Dec/06 1:51 PM

The disabled menuItem functionality has never been fully implemented.

The disabled functionality of MenuItems will work as follows;
1) Disabling a Top Level vertical or horizontal menu item will disable all of
it's child submenu items.
2) Hover behavior will still expand submenus but they will be disabled and the
hover will not hilite like a regular active menu item.

The disabled styleclasses for the MenuBar need to be created and added to our
xp.css and royale.css.

// disabled classes for top level horizontal
iceMenu-dis
iceMenu-dis a
// disabled classes for top level vertical
iceMenu_verticalItem-dis
iceMenu_verticalItem-dis a
// disabled classes for submenus in a vertical menubar
iceSubMenu_vertical-dis
iceSubMenu_vertical-dis a
// disabled classes for submenus in a horizontal menubar
iceSubMenu-dis
iceSubMenu-dis a

major cleanup is required in our CSS_DEFAULT.java *

Show

Greg McCleary added a comment - 15/Dec/06 1:51 PM The disabled menuItem functionality has never been fully implemented. The disabled functionality of MenuItems will work as follows; 1) Disabling a Top Level vertical or horizontal menu item will disable all of it's child submenu items. 2) Hover behavior will still expand submenus but they will be disabled and the hover will not hilite like a regular active menu item. The disabled styleclasses for the MenuBar need to be created and added to our xp.css and royale.css. // disabled classes for top level horizontal iceMenu-dis iceMenu-dis a // disabled classes for top level vertical iceMenu_verticalItem-dis iceMenu_verticalItem-dis a // disabled classes for submenus in a vertical menubar iceSubMenu_vertical-dis iceSubMenu_vertical-dis a // disabled classes for submenus in a horizontal menubar iceSubMenu-dis iceSubMenu-dis a major cleanup is required in our CSS_DEFAULT.java *

Hide

Permalink

Greg McCleary added a comment - 23/Jan/07 12:43 PM

Created an attachment (id=139)
testcase for disabling individual menuItems and their sub-menus

Show

Greg McCleary added a comment - 23/Jan/07 12:43 PM Created an attachment (id=139) testcase for disabling individual menuItems and their sub-menus

Hide

Permalink

Greg McCleary added a comment - 23/Jan/07 12:45 PM

Fixed in HEAD [ rev. 12990 ]

Show

Greg McCleary added a comment - 23/Jan/07 12:45 PM Fixed in HEAD [ rev. 12990 ]

Icefaces Administrator made changes - 16/Feb/07 11:44 AM

Field	Original Value	New Value
issue.field.bugzillaimportkey	1141	12372

Ken Fyten made changes - 16/Feb/07 7:27 PM

Fix Version/s		1.6DR1 [ 10035 ]
Fix Version/s	1.6 [ 10031 ]

Hide

Permalink

Philip Breau added a comment - 21/Feb/07 3:58 PM

fix doesn't work for top level menu items.

suggested fix diffed to 1.5.3:

— C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java 2006-11-07 14:38:10.000000000 -0700
+++ C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java 2007-02-21 13:54:29.000000000 -0700
@@ -145,21 +145,23 @@
Element topLevelDiv = domContext.createRootElement(HTML.DIV_ELEM);
topLevelDiv.setAttribute(HTML.ID_ATTR, clientId);
}
Element topLevelDiv = (Element) domContext.getRootNode();
topLevelDiv.setAttribute(HTML.NAME_ATTR, "TOP_LEVEL");

+ boolean isDisabled = ((MenuItem)uiComponent).isDisabled();
if (vertical)

{ - topLevelDiv.setAttribute(HTML.CLASS_ATTR, - CSS_DEFAULT.MENU_VERTICAL_ITEM_STYLE); + topLevelDiv.setAttribute(HTML.CLASS_ATTR, + CSS_DEFAULT.MENU_VERTICAL_ITEM_STYLE + ( isDisabled ? "-dis" : "" ) ); }

else

{ - topLevelDiv.setAttribute(HTML.CLASS_ATTR, - CSS_DEFAULT.MENU_HORIZONTAL_STYLE); + topLevelDiv.setAttribute(HTML.CLASS_ATTR, + CSS_DEFAULT.MENU_HORIZONTAL_STYLE + (isDisabled ? "-dis" : "") ); }

if (uiComponent.getChildCount() > 0) {
+ if (uiComponent.getChildCount() > 0 && !isDisabled )
Unknown macro: { String displayEvent = HTML.ONMOUSEOVER_ATTR; if (vertical) { topLevelDiv.setAttribute(displayEvent, "Ice.Menu.hideOrphanedMenusNotRelatedTo(this);" + expand("this", clientId + "_sub", KEYWORD_THIS)); @@ -254,13 +256,19 @@ div.appendChild(iconImg); } }

// create a span for text
Element span = domContext.createElement(HTML.SPAN_ELEM);

span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel");
+ if (!menuItem.isDisabled()) { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR, "iceLink"); + }
else
{ + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel-dis"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR, "iceLink-dis"); + }
div.appendChild(span);
// create text
Node text = domContext.createTextNode(DOMUtils.escapeAnsi(menuItem.getValue().toString()));
span.appendChild(text);

return anchor;
@@ -303,13 +311,19 @@
iconImg.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowImage ");
div.appendChild(iconImg);
}

// create a span for text
Element span = domContext.createElement(HTML.SPAN_ELEM);

span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel");
+ if (!menuItem.isDisabled()) { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink"); + } else { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel-dis"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink-dis"); + }
div.appendChild(span);
// create text
Node text = domContext.createTextNode(DOMUtils.escapeAnsi(menuItem.getValue().toString()));
span.appendChild(text);

return anchor;
@@ -348,13 +362,19 @@
iconImg.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowImage ");
div.appendChild(iconImg);
}

// create a span for text
Element span = domContext.createElement(HTML.SPAN_ELEM);
- span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel");
+ if (!menuItem.isDisabled()) { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink"); + }
else
{ + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel-dis"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink-dis"); + }
div.appendChild(span);
// create text
Node text = domContext.createTextNode(DOMUtils.escapeAnsi(menuItem.getValue().toString()));
span.appendChild(text);

return anchor;
@@ -486,13 +506,13 @@
link.setValue(nextSubMenuItem.getValue());
link.setParent(nextSubMenuItem);
link.setId(LINK_SUFFIX);
if (nextSubMenuItem.isDisabled())

{ link.setDisabled(nextSubMenuItem.isDisabled()); }

link.setStyleClass("");
+ //link.setStyleClass("");
Node lastCursorParent = domContext.getCursorParent();
domContext.setCursorParent(subMenuItemDiv);
if (vertical) { addChildren(link, nextSubMenuItem, menuComponent); }
else
{ addTopLevelChildren(link, nextSubMenuItem, menuComponent); @@ -575,13 +595,17 @@ image.setStyleClass("iceSubMenuRowImage"); div.getChildren().add(image); }

HtmlOutputText outputText = new HtmlOutputText();
outputText.setValue(link.getValue());

outputText.setStyleClass("iceSubMenuRowLabel");
+ if (!nextSubMenuItem.isDisabled()) { + outputText.setStyleClass("iceSubMenuRowLabel"); + } else { + outputText.setStyleClass("iceSubMenuRowLabel-dis"); + }
link.setValue("");
div.getChildren().add(outputText);

link.getChildren().add(div);
}

@@ -607,13 +631,17 @@
image.setStyleClass("iceSubMenuRowImage");
div.getChildren().add(image);
}

HtmlOutputText outputText = new HtmlOutputText();
outputText.setValue(link.getValue());
- outputText.setStyleClass("iceSubMenuRowLabel");
+ if (!nextSubMenuItem.isDisabled()) { + outputText.setStyleClass("iceSubMenuRowLabel"); + }
else
{ + outputText.setStyleClass("iceSubMenuRowLabel-dis"); + }
link.setValue("");
div.getChildren().add(outputText);

link.getChildren().add(div);
}

— C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\core\src\com\icesoft\faces\resources\css\royale\royale.css 2006-11-03 14:30:58.000000000 -0700
+++ C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\core\src\com\icesoft\faces\resources\css\royale\royale.css 2007-02-21 14:51:22.000000000 -0700
@@ -1431,12 +1431,127 @@
.iceSubMenuDividerVert

{ height: auto; float: left; width: 100%; }

+/* submenu disabled style classes */
+.iceSubMenuRowLabel-dis

{ + color: #ABABAB; + text-decoration: none; + cursor: default; +}

+
+.iceSubMenuRow-dis

{ + color: #ABABAB; + text-decoration: none; + cursor: default; + padding: 0px; + display: block; + height: auto; +}

+
+.iceSubMenuRow-dis a

{ + color: #333333; + text-decoration: none; + cursor: default; + display: block; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + +}

+
+.iceSubMenuRow-dis a:hover

{ + color: #ABABAB; + text-decoration: none; + border-bottom: 1px dotted #CCCCCC; + cursor: default; +}

+.iceMenu-dis

{ + float: left; + padding: 0px; + color: #ABABAB; + text-decoration: none; + display: block; + white-space: nowrap; +}

+
+.iceMenu-dis a

{ + background-color: #DDDDDD; + height: auto; + padding-left: 10px; + padding-right: 10px; + padding-top: 3px; + padding-bottom: 3px; + color: #333333; /* Must be >1.01em to avoid layout artifacts on Macs */ + font-size: 1.1em; + margin: 0px; + border-right: 1px solid #FFFFFF; + width: auto; + float: left; + display: block; + text-decoration: none; + cursor: default; + +}

+
+.iceMenu-dis a:hover

{ + background-color: #2A6CC2; + border-right: 1px solid #FFFFFF; + text-decoration: none; + color: #ABABAB; + cursor: default; +}

+
+.iceMenu-dis hr, .subiceMenu-dis hr

{ + border: none 0; + border-top: 1px solid #999999; + width: 100%; + height: 1px; + margin-top: 3px; + padding: 0px; + color: #ABABAB; + cursor: default; + text-align: left; +}

+
+.iceMenu_verticalItem-dis

{ + width: 160px; + color: #ABABAB; + text-decoration: none; + display: block; +}

+
+.iceMenu_verticalItem-dis a

{ + background-color: #DDDDDD; + height: auto; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + color: #333333; + font-size: 1.1em; + margin-bottom: 1px; + float: none; + display: block; + text-decoration: none; + border: 0px solid #FFFFFF; + cursor: default; + width: 100%; +}

+
+.iceMenu_verticalItem a:hover

{ + background-color: #2A6CC2; + text-decoration: none; + color: #ABABAB; + cursor: default; +}

+
+
/*
----------- PanelPopup ----------
*/

.icePanelPopup

{ color: #666666; --- C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\core\src\com\icesoft\faces\resources\css\xp\xp.css 2006-11-09 15:35:40.000000000 -0700 +++ C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\core\src\com\icesoft\faces\resources\css\xp\xp.css 2007-02-21 14:02:03.000000000 -0700 @@ -1357,12 +1357,60 @@ width: 100%; height: 1px; margin-top: 3px; padding: 0px; text-align: left; }

+.iceMenu-dis

{ + float: left; + padding: 0px; + color: #ABABAB; + text-decoration: none; + display: block; + white-space: nowrap; +}

+
+.iceMenu-dis a

+
+.iceMenu-dis a:hover

{ + background-color: #2A6CC2; + border-right: 1px solid #FFFFFF; + text-decoration: none; + color: #ABABAB; + cursor: default; +}

+
+.iceMenu-dis hr, .subiceMenu-dis hr

{ + border: none 0; + border-top: 1px solid #999999; + width: 100%; + height: 1px; + margin-top: 3px; + padding: 0px; + color: #ABABAB; + cursor: default; + text-align: left; +}

/* classes for the vertical version of the iceMenu*/
.iceMenu_vertical {
float: left;
padding: 0px;
white-space: nowrap;
@@ -1392,12 +1440,44 @@

.iceMenu_verticalItem a:hover

{ background-color: #2A6CC2; color: #FFFFFF; text-decoration: none; }

+.iceMenu_verticalItem-dis

{ + width: 160px; + color: #ABABAB; + text-decoration: none; + display: block; +}

+
+.iceMenu_verticalItem-dis a

+
+.iceMenu_verticalItem a:hover

{ + background-color: #2A6CC2; + text-decoration: none; + color: #ABABAB; + cursor: default; +}

.iceSubMenu {
position: absolute;
background-color: #EFEFEF;
width: 160px;
@@ -1486,12 +1566,47 @@
.iceSubMenuDividerVert

{ height: auto; float: left; width: 100%; }

+/* submenu disabled style classes */
+.iceSubMenuRowLabel-dis

{ + color: #ABABAB; + text-decoration: none; + cursor: default; +}

+
+.iceSubMenuRow-dis

{ + color: #ABABAB; + text-decoration: none; + cursor: default; + padding: 0px; + display: block; + height: auto; +}

+
+.iceSubMenuRow-dis a

{ + color: #333333; + text-decoration: none; + cursor: default; + display: block; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + +}

+
+.iceSubMenuRow-dis a:hover

{ + color: #ABABAB; + text-decoration: none; + border-bottom: 1px dotted #CCCCCC; + cursor: default; +}

+
/*
----------- PanelPopup ----------
*/

.icePanelPopup {
color: #666666;

Show

Philip Breau added a comment - 21/Feb/07 3:58 PM fix doesn't work for top level menu items. suggested fix diffed to 1.5.3: — C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java 2006-11-07 14:38:10.000000000 -0700 +++ C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java 2007-02-21 13:54:29.000000000 -0700 @@ -145,21 +145,23 @@ Element topLevelDiv = domContext.createRootElement(HTML.DIV_ELEM); topLevelDiv.setAttribute(HTML.ID_ATTR, clientId); } Element topLevelDiv = (Element) domContext.getRootNode(); topLevelDiv.setAttribute(HTML.NAME_ATTR, "TOP_LEVEL"); + boolean isDisabled = ((MenuItem)uiComponent).isDisabled(); if (vertical) { - topLevelDiv.setAttribute(HTML.CLASS_ATTR, - CSS_DEFAULT.MENU_VERTICAL_ITEM_STYLE); + topLevelDiv.setAttribute(HTML.CLASS_ATTR, + CSS_DEFAULT.MENU_VERTICAL_ITEM_STYLE + ( isDisabled ? "-dis" : "" ) ); } else { - topLevelDiv.setAttribute(HTML.CLASS_ATTR, - CSS_DEFAULT.MENU_HORIZONTAL_STYLE); + topLevelDiv.setAttribute(HTML.CLASS_ATTR, + CSS_DEFAULT.MENU_HORIZONTAL_STYLE + (isDisabled ? "-dis" : "") ); } + if (uiComponent.getChildCount() > 0) { + if (uiComponent.getChildCount() > 0 && !isDisabled ) Unknown macro: { String displayEvent = HTML.ONMOUSEOVER_ATTR; if (vertical) { topLevelDiv.setAttribute(displayEvent, "Ice.Menu.hideOrphanedMenusNotRelatedTo(this);" + expand("this", clientId + "_sub", KEYWORD_THIS)); @@ -254,13 +256,19 @@ div.appendChild(iconImg); } } // create a span for text Element span = domContext.createElement(HTML.SPAN_ELEM); span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + if (!menuItem.isDisabled()) { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR, "iceLink"); + } else { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel-dis"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR, "iceLink-dis"); + } div.appendChild(span); // create text Node text = domContext.createTextNode(DOMUtils.escapeAnsi(menuItem.getValue().toString())); span.appendChild(text); return anchor; @@ -303,13 +311,19 @@ iconImg.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowImage "); div.appendChild(iconImg); } // create a span for text Element span = domContext.createElement(HTML.SPAN_ELEM); span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + if (!menuItem.isDisabled()) { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink"); + } else { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel-dis"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink-dis"); + } div.appendChild(span); // create text Node text = domContext.createTextNode(DOMUtils.escapeAnsi(menuItem.getValue().toString())); span.appendChild(text); return anchor; @@ -348,13 +362,19 @@ iconImg.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowImage "); div.appendChild(iconImg); } // create a span for text Element span = domContext.createElement(HTML.SPAN_ELEM); - span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + if (!menuItem.isDisabled()) { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink"); + } else { + span.setAttribute(HTML.CLASS_ATTR, "iceSubMenuRowLabel-dis"); + anchor.setAttribute(HTML.STYLE_CLASS_ATTR,"iceLink-dis"); + } div.appendChild(span); // create text Node text = domContext.createTextNode(DOMUtils.escapeAnsi(menuItem.getValue().toString())); span.appendChild(text); return anchor; @@ -486,13 +506,13 @@ link.setValue(nextSubMenuItem.getValue()); link.setParent(nextSubMenuItem); link.setId(LINK_SUFFIX); if (nextSubMenuItem.isDisabled()) { link.setDisabled(nextSubMenuItem.isDisabled()); } link.setStyleClass(""); + //link.setStyleClass(""); Node lastCursorParent = domContext.getCursorParent(); domContext.setCursorParent(subMenuItemDiv); if (vertical) { addChildren(link, nextSubMenuItem, menuComponent); } else { addTopLevelChildren(link, nextSubMenuItem, menuComponent); @@ -575,13 +595,17 @@ image.setStyleClass("iceSubMenuRowImage"); div.getChildren().add(image); } HtmlOutputText outputText = new HtmlOutputText(); outputText.setValue(link.getValue()); outputText.setStyleClass("iceSubMenuRowLabel"); + if (!nextSubMenuItem.isDisabled()) { + outputText.setStyleClass("iceSubMenuRowLabel"); + } else { + outputText.setStyleClass("iceSubMenuRowLabel-dis"); + } link.setValue(""); div.getChildren().add(outputText); link.getChildren().add(div); } @@ -607,13 +631,17 @@ image.setStyleClass("iceSubMenuRowImage"); div.getChildren().add(image); } HtmlOutputText outputText = new HtmlOutputText(); outputText.setValue(link.getValue()); - outputText.setStyleClass("iceSubMenuRowLabel"); + if (!nextSubMenuItem.isDisabled()) { + outputText.setStyleClass("iceSubMenuRowLabel"); + } else { + outputText.setStyleClass("iceSubMenuRowLabel-dis"); + } link.setValue(""); div.getChildren().add(outputText); link.getChildren().add(div); } — C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\core\src\com\icesoft\faces\resources\css\royale\royale.css 2006-11-03 14:30:58.000000000 -0700 +++ C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\core\src\com\icesoft\faces\resources\css\royale\royale.css 2007-02-21 14:51:22.000000000 -0700 @@ -1431,12 +1431,127 @@ .iceSubMenuDividerVert { height: auto; float: left; width: 100%; } +/* submenu disabled style classes */ +.iceSubMenuRowLabel-dis { + color: #ABABAB; + text-decoration: none; + cursor: default; +} + +.iceSubMenuRow-dis { + color: #ABABAB; + text-decoration: none; + cursor: default; + padding: 0px; + display: block; + height: auto; +} + +.iceSubMenuRow-dis a { + color: #333333; + text-decoration: none; + cursor: default; + display: block; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + +} + +.iceSubMenuRow-dis a:hover { + color: #ABABAB; + text-decoration: none; + border-bottom: 1px dotted #CCCCCC; + cursor: default; +} +.iceMenu-dis { + float: left; + padding: 0px; + color: #ABABAB; + text-decoration: none; + display: block; + white-space: nowrap; +} + +.iceMenu-dis a { + background-color: #DDDDDD; + height: auto; + padding-left: 10px; + padding-right: 10px; + padding-top: 3px; + padding-bottom: 3px; + color: #333333; /* Must be >1.01em to avoid layout artifacts on Macs */ + font-size: 1.1em; + margin: 0px; + border-right: 1px solid #FFFFFF; + width: auto; + float: left; + display: block; + text-decoration: none; + cursor: default; + +} + +.iceMenu-dis a:hover { + background-color: #2A6CC2; + border-right: 1px solid #FFFFFF; + text-decoration: none; + color: #ABABAB; + cursor: default; +} + +.iceMenu-dis hr, .subiceMenu-dis hr { + border: none 0; + border-top: 1px solid #999999; + width: 100%; + height: 1px; + margin-top: 3px; + padding: 0px; + color: #ABABAB; + cursor: default; + text-align: left; +} + +.iceMenu_verticalItem-dis { + width: 160px; + color: #ABABAB; + text-decoration: none; + display: block; +} + +.iceMenu_verticalItem-dis a { + background-color: #DDDDDD; + height: auto; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + color: #333333; + font-size: 1.1em; + margin-bottom: 1px; + float: none; + display: block; + text-decoration: none; + border: 0px solid #FFFFFF; + cursor: default; + width: 100%; +} + +.iceMenu_verticalItem a:hover { + background-color: #2A6CC2; + text-decoration: none; + color: #ABABAB; + cursor: default; +} + + /* ----------- PanelPopup ---------- */ .icePanelPopup { color: #666666; --- C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\core\src\com\icesoft\faces\resources\css\xp\xp.css 2006-11-09 15:35:40.000000000 -0700 +++ C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\core\src\com\icesoft\faces\resources\css\xp\xp.css 2007-02-21 14:02:03.000000000 -0700 @@ -1357,12 +1357,60 @@ width: 100%; height: 1px; margin-top: 3px; padding: 0px; text-align: left; } +.iceMenu-dis { + float: left; + padding: 0px; + color: #ABABAB; + text-decoration: none; + display: block; + white-space: nowrap; +} + +.iceMenu-dis a { + background-color: #DDDDDD; + height: auto; + padding-left: 10px; + padding-right: 10px; + padding-top: 3px; + padding-bottom: 3px; + color: #333333; /* Must be >1.01em to avoid layout artifacts on Macs */ + font-size: 1.1em; + margin: 0px; + border-right: 1px solid #FFFFFF; + width: auto; + float: left; + display: block; + text-decoration: none; + cursor: default; + +} + +.iceMenu-dis a:hover { + background-color: #2A6CC2; + border-right: 1px solid #FFFFFF; + text-decoration: none; + color: #ABABAB; + cursor: default; +} + +.iceMenu-dis hr, .subiceMenu-dis hr { + border: none 0; + border-top: 1px solid #999999; + width: 100%; + height: 1px; + margin-top: 3px; + padding: 0px; + color: #ABABAB; + cursor: default; + text-align: left; +} + /* classes for the vertical version of the iceMenu*/ .iceMenu_vertical { float: left; padding: 0px; white-space: nowrap; @@ -1392,12 +1440,44 @@ .iceMenu_verticalItem a:hover { background-color: #2A6CC2; color: #FFFFFF; text-decoration: none; } +.iceMenu_verticalItem-dis { + width: 160px; + color: #ABABAB; + text-decoration: none; + display: block; +} + +.iceMenu_verticalItem-dis a { + background-color: #DDDDDD; + height: auto; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + color: #333333; + font-size: 1.1em; + margin-bottom: 1px; + float: none; + display: block; + text-decoration: none; + border: 0px solid #FFFFFF; + cursor: default; + width: 100%; +} + +.iceMenu_verticalItem a:hover { + background-color: #2A6CC2; + text-decoration: none; + color: #ABABAB; + cursor: default; +} + .iceSubMenu { position: absolute; background-color: #EFEFEF; width: 160px; @@ -1486,12 +1566,47 @@ .iceSubMenuDividerVert { height: auto; float: left; width: 100%; } +/* submenu disabled style classes */ +.iceSubMenuRowLabel-dis { + color: #ABABAB; + text-decoration: none; + cursor: default; +} + +.iceSubMenuRow-dis { + color: #ABABAB; + text-decoration: none; + cursor: default; + padding: 0px; + display: block; + height: auto; +} + +.iceSubMenuRow-dis a { + color: #333333; + text-decoration: none; + cursor: default; + display: block; + padding-left: 3px; + padding-right: 3px; + padding-top: 3px; + padding-bottom: 3px; + +} + +.iceSubMenuRow-dis a:hover { + color: #ABABAB; + text-decoration: none; + border-bottom: 1px dotted #CCCCCC; + cursor: default; +} + /* ----------- PanelPopup ---------- */ .icePanelPopup { color: #666666;

Philip Breau made changes - 21/Feb/07 3:58 PM

Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]
Estimated Complexity		Low
Support Case References		https://www.icesoft.ca:4443/supportilla/show_bug.cgi?id=3894
Assignee	Greg McCleary [ gregory.mccleary ]	Rob Mayhew [ rob.mayhew ]
Supported Customers		714

Ken Fyten made changes - 26/Feb/07 10:26 AM

Fix Version/s		1.6 [ 10031 ]
Fix Version/s	1.6DR#1 [ 10035 ]

Ken Fyten made changes - 28/Feb/07 11:25 PM

Assignee Priority

Ken Fyten made changes - 06/Apr/07 3:12 PM

Assignee

Rob Mayhew [ rob.mayhew ]

Adnan Durrani [ adnan.durrani ]

Ken Fyten made changes - 17/Apr/07 6:56 PM

Fix Version/s		1.6DR#4 [ 10060 ]
Fix Version/s	1.6 [ 10031 ]
Assignee Priority	P2	P3

Ken Fyten made changes - 23/Apr/07 12:58 PM

Assignee Priority

Hide

Permalink

Adnan Durrani added a comment - 24/Apr/07 12:30 PM

I have tested this issue and found that it has been already fixed in 1.6 DR#1. I can disable any level of menu including top level.

Show

Adnan Durrani added a comment - 24/Apr/07 12:30 PM I have tested this issue and found that it has been already fixed in 1.6 DR#1. I can disable any level of menu including top level.

Adnan Durrani made changes - 24/Apr/07 12:30 PM

Status	Reopened [ 4 ]	Resolved [ 5 ]
Fix Version/s		1.6DR#1 [ 10035 ]
Fix Version/s	1.6DR#4 [ 10060 ]
Resolution		Cannot Reproduce [ 5 ]

Ken Fyten made changes - 02/Jul/07 5:34 PM

Fix Version/s

1.6 [ 10031 ]

Ken Fyten made changes - 05/Jul/07 3:48 PM

Status	Resolved [ 5 ]	Closed [ 6 ]
Assignee Priority	P2
Assignee	Adnan Durrani [ adnan.durrani ]

People

Assignee:

Unassigned

Reporter:

Philip Breau

Votes:

0 Vote for this issue

Watchers:

1 Start watching this issue

Dates

Created:

13/Dec/06 3:53 PM

Updated:

05/Jul/07 3:48 PM

Resolved:

24/Apr/07 12:30 PM