Details
-
Type: Bug
-
Status: Closed
-
Priority: Minor
-
Resolution: Fixed
-
Affects Version/s: 1.8.2-EE-GA_P02
-
Fix Version/s: EE-1.8.2.GA_P03
-
Component/s: ICE-Components
-
Labels:None
-
Environment:-
Description
-
Hide
- sc9738.war
- 7.31 MB
- Tyler Johnson
-
- META-INF/MANIFEST.MF 0.0 kB
- WEB-INF/lib/backport-util-concurrent.jar 319 kB
- WEB-INF/lib/commons-digester.jar 140 kB
- WEB-INF/lib/FastInfoset.jar 285 kB
- WEB-INF/lib/icefaces-comps.jar 3.16 MB
- ICEfacesPage1.jspx 3 kB
- WEB-INF/lib/jxl.jar 708 kB
- WEB-INF/.../krysalis-jCharts-1.0.0-alpha-1.jar 151 kB
- WEB-INF/lib/commons-beanutils.jar 226 kB
- css/menuStyle.css 2 kB
- images/blue004.jpg 3 kB
- WEB-INF/faces-config.xml 0.3 kB
- WEB-INF/lib/commons-fileupload.jar 56 kB
- WEB-INF/lib/jsf-impl.jar 837 kB
- images/images.jpg 0.5 kB
- index.jsp 0.1 kB
- WEB-INF/lib/jsf-api.jar 355 kB
- WEB-INF/lib/icefaces.jar 1.21 MB
- WEB-INF/web.xml 4 kB
- WEB-INF/lib/commons-logging.jar 52 kB
- WEB-INF/lib/commons-collections.jar 558 kB
-
- menuAbove.jpg
- 61 kB
-
- overTop.jpg
- 87 kB
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Fixed bug that produced screenshot 1.
Revision: 24528
Modified : /icefaces/trunk/icefaces/bridge/component/menu.js
For screenshot2 (overTop), the current behaviour is to position the submenu overtop of the root menu in order to maximize the amount of the submenu that can be made visible on the page, only in cases where there the sub-menu cannot be fully displayed either below or above the root-menu item. After review, this is judged to be a desirable feature, even if it means that in these cases the sub-menu may not appear in exactly the expected/usual location. It also offers benefits in mobile or limited-screen real-estate environments by ensuring the user will see and be able to use the sub-menu in virtually all cases. There is also precedent for this in desktop UI menu behaviour, where the specific position of a submenu is altered as required to ensure the entire sub-menu is visible on the screen.
Case in screenshot 1 should not occur. It should always be like screenshot 2. This is yet another bug introduced by the optimizations done in ICE-4106. A value which should be calculated dynamically on the spot was "optimized" to be calculated only at the start of the function.
In screenshot 2, the menu is not wrongly positioned on top of the menu bar, it is to avoid the menu positioning beyond the top of the viewport. This is a requirement of ICE-3658. Do we want to revert that?
If the space above is larger than the space below, then better position menu above even if can't fit completely. Ohterwise if the menu bar is close to the bottom of the viewport user would complain again why we don't put the menu above.
Another tweak to the menu positioning logic. I'd suggest that you don't position the menu above the menubar unless it can fit completely in the available space.
Also, the top of the menu should always be under (or over) the menu-bar, not on top of it.
Tested successfully on icefaces1.8 trunk revision# 24720 using FF3.6, IE8 & chrome10.