Details
-
Type: Improvement
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: EE-1.8.2.GA_P03
-
Fix Version/s: 2.1-Beta2, 3.0, EE-1.8.2.GA_P04, EE-2.0.0.GA_P01
-
Component/s: ICE-Components
-
Labels:None
-
Environment:ICEfaces EE 1.8.2 P03, IE, Firefox
Description
-
Hide
- Case10297Example.war
- 6.79 MB
- Arran Mccullough
-
- META-INF/MANIFEST.MF 0.1 kB
- META-INF/context.xml 0.1 kB
- WEB-INF/classes/com/.../example/Item.class 1 kB
- WEB-INF/classes/.../example/TestBean.class 1 kB
- WEB-INF/faces-config.xml 0.6 kB
- WEB-INF/lib/backport-util-concurrent.jar 316 kB
- WEB-INF/lib/commons-beanutils.jar 223 kB
- WEB-INF/lib/commons-collections.jar 554 kB
- WEB-INF/lib/commons-digester.jar 137 kB
- WEB-INF/lib/commons-fileupload.jar 56 kB
- WEB-INF/lib/commons-logging.jar 51 kB
- WEB-INF/lib/icefaces-comps.jar 1.73 MB
- WEB-INF/lib/icefaces-facelets.jar 592 kB
- WEB-INF/lib/icefaces.jar 1.20 MB
- WEB-INF/lib/jsf-api-1.2.jar 347 kB
- WEB-INF/lib/jsf-impl-1.2.jar 818 kB
- WEB-INF/lib/jxl.jar 689 kB
- WEB-INF/.../krysalis-jCharts-1.0.0-alpha-1.jar 148 kB
- WEB-INF/web.xml 4 kB
- welcomeICEfaces.xhtml 3 kB
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
The fix works with IE7 as well.
This issue is almost fixed. In the attached test case if you display the tooltip for most rows it works perfectly. If you try to display the tooltip for the last three rows in the table to tooltip displays away from where it should be.
This last problem is caused by the fact that the tooltip panel is also a draggable panel. The draggable positioning code takes precedence and brings the panel into the visible viewport in case it's positioned outside. The tooltip positioning then tries to move the panel close to where the 'mouseover' event coordinates where. Unfortunately the code assumes that the panel was positioned exactly where the 'mouseover' coordinates where without being aware that the draggable positioning code already changed panel's top/left coordinates.
Corrected tooltip positioning algorithm. The algorithm uses now the captured mouse coordinates instead of the panel's top/left coordinates.
Since the last commit the tooltip demo in Component Showcase throughs a JS error in IE7.
This commit includes JDK 1.5+ API usage that must be corrected for icefaces/trunk to maintain JDK 1.4 compatibility.
- The tooltip is never displayed when hovering over the province text. This also causes an exception in the server log:
<Apr 10, 2012 6:26:42 AM MDT> <Error> <HTTP> <BEA-101017> <[ServletContext(id=26999190,name=component-showcase,context-path=/component-showcase)] Root cause of ServletException.
java.lang.NoSuchMethodError: java.lang.String.replace(Ljava/lang/CharSequence;Ljava/lang/CharSequence;)Ljava/lang/String;
at com.icesoft.faces.component.paneltooltip.PanelTooltip.applyStyle(PanelTooltip.java:216)
at com.icesoft.faces.component.panelpopup.PanelPopupRenderer.encodeBegin(PanelPopupRenderer.java:260)
at com.icesoft.faces.component.paneltooltip.PanelTooltipRenderer.encodeBegin(PanelTooltipRenderer.java:75)
Fixed by using JDK 1.4 API to replace character sequences in strings.
Changed viewportOffset function to include body's scroll offsets when calculating the total viewport offset for a given element. This effectively restores the algorithm used in pre-1.7 PrototypeJS.