ICEfaces
  1. ICEfaces
  2. ICE-8830

REGRESSION: ice:panelTooltip displays incorrectly when used in an ice:panelPopup

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.0.0.GA, EE-1.8.2.GA_P04, 3.2
          • Fix Version/s: EE-3.2.0.GA, EE-1.8.2.GA_P06, 3.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All
          • Assignee Priority:
            P1
          • Salesforce Case Reference:
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Add the following CSS to override the positioning:

                        .icePnlTlTip {
                             position: fixed !important;
                        }
            Show
            Add the following CSS to override the positioning:             .icePnlTlTip {                  position: fixed !important;             }

            Description

            When using an ice:panelTooltip in a modal ice:panelPopup, the tooltip is displayed in an incorrect position. This was resolved in the P02 and P03 releases but has since been broken. The following JIRA was the originally reported issue: ICE-5607
            1. screenshot-01.png
              193 kB

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case Reference 5007000000QWB9PAAX
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case that shows issue. Load welcomeICEfaces.iface and hover over the icon in the popup.

              Show
              Arran Mccullough added a comment - Attached test case that shows issue. Load welcomeICEfaces.iface and hover over the icon in the popup.
              Arran Mccullough made changes -
              Attachment Case11846Example.zip [ 15120 ]
              Attachment Case11846Example.war [ 15121 ]
              Ken Fyten made changes -
              Summary ice:panelTooltip displays incorrectly when used in an ice:panelPopup REGRESSION: ice:panelTooltip displays incorrectly when used in an ice:panelPopup
              Assignee yip.ng [ yip.ng ]
              Fix Version/s EE-1.8.2.GA_P06 [ 10470 ]
              Assignee Priority P1 [ 10010 ]
              Hide
              Permalink
              yip.ng added a comment - - edited

              Error running test app. Need to fix first:

              javax.servlet.ServletException: java.lang.Exception: javax.faces.FacesException: org.w3c.dom.DOMException: HIERARCHY_REQUEST_ERR: An attempt was made to insert a node where it is not permitted.
              com.icesoft.faces.webapp.http.servlet.MainServlet.service(MainServlet.java:213)
              javax.servlet.http.HttpServlet.service(HttpServlet.java:722)

              Show
              yip.ng added a comment - - edited Error running test app. Need to fix first: javax.servlet.ServletException: java.lang.Exception: javax.faces.FacesException: org.w3c.dom.DOMException: HIERARCHY_REQUEST_ERR: An attempt was made to insert a node where it is not permitted. com.icesoft.faces.webapp.http.servlet.MainServlet.service(MainServlet.java:213) javax.servlet.http.HttpServlet.service(HttpServlet.java:722)
              Hide
              Permalink
              yip.ng added a comment - - edited

              Test app. runtime error above fixed.

              Reverted to revision in ICE-5607 (23129). Working. See video http://screencast.com/t/uYVrUKHEXjTe.

              Since then many many changes in tooltip_panelpopup.js (together with PanelPopupRenderer.java, PanelTooltip.java, FormRenderer.java) by Mircea. (Different check-ins.) Regression occurs at revision 26133, part of ICE-7053. See screenshot-01.

              Show
              yip.ng added a comment - - edited Test app. runtime error above fixed. Reverted to revision in ICE-5607 (23129). Working. See video http://screencast.com/t/uYVrUKHEXjTe . Since then many many changes in tooltip_panelpopup.js (together with PanelPopupRenderer.java, PanelTooltip.java, FormRenderer.java) by Mircea. (Different check-ins.) Regression occurs at revision 26133, part of ICE-7053 . See screenshot-01.
              yip.ng made changes -
              Attachment screenshot-01.png [ 15280 ]
              Arran Mccullough made changes -
              Attachment Case10297Example.zip [ 15299 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32923 Mon Dec 24 16:54:57 MST 2012 yip.ng ICE-8830: REGRESSION: ice:panelTooltip displays incorrectly when used in an ice:panelPopup.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/tooltip_panelpopup.js
              Hide
              Permalink
              yip.ng added a comment -

              Added check and special processing for when tooltip is in dialog. All other scenarios untouched and therefore shouldn't cause regressions for all changes after ICE-5607 by Mircea.

              Tested: IE 7,8,9, FF 17. Also tested test app. for ICE-7053, the JIRA which caused this regression.

              Sending content: C:\svn\ossrepo\icefaces\trunk\icefaces\bridge\lib\extras\tooltip_panelpopup.js
              Completed: At revision: 32923

              Show
              yip.ng added a comment - Added check and special processing for when tooltip is in dialog. All other scenarios untouched and therefore shouldn't cause regressions for all changes after ICE-5607 by Mircea. Tested: IE 7,8,9, FF 17. Also tested test app. for ICE-7053 , the JIRA which caused this regression. Sending content: C:\svn\ossrepo\icefaces\trunk\icefaces\bridge\lib\extras\tooltip_panelpopup.js Completed: At revision: 32923
              yip.ng made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32958 Fri Jan 04 14:45:21 MST 2013 yip.ng ICE-8830: REGRESSION: ice:panelTooltip displays incorrectly when used in an ice:panelPopup.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js
              Hide
              Permalink
              yip.ng added a comment -

              Changes propagated to icefaces3:

              M: C:\svn\ossrepo\icefaces3\trunk\icefaces\compat\core\src\main\javascript\extras\extras.js#32958

              Show
              yip.ng added a comment - Changes propagated to icefaces3: M: C:\svn\ossrepo\icefaces3\trunk\icefaces\compat\core\src\main\javascript\extras\extras.js#32958
              Ken Fyten made changes -
              Fix Version/s EE-3.2.0.GA [ 10332 ]
              Fix Version/s 3.3 [ 10370 ]
              Affects Version/s 3.2 [ 10338 ]
              Affects Version/s EE-3.0.0.GA [ 10262 ]
              Affects Version/s EE-1.8.2.GA_P05 [ 10331 ]
              Hide
              Permalink
              Jerome Ruzol added a comment -

              Issue re-appeared on trunk rev#32999, retested using trunk rev#32957 and the test failed in IE and FF.

              Show
              Jerome Ruzol added a comment - Issue re-appeared on trunk rev#32999, retested using trunk rev#32957 and the test failed in IE and FF.
              Jerome Ruzol made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              yip.ng added a comment -

              Can't reproduce. See videos:

              1.8
              FF: http://screencast.com/t/6XbXSh3o7
              IE7: http://screencast.com/t/TWSFOD6L
              IE8: http://screencast.com/t/t2DD6ALe
              IE9: http://screencast.com/t/aYEQelNW143

              3.2
              FF: http://screencast.com/t/1llBjSmPash
              IE7: http://screencast.com/t/FuJHzkSCR5
              IE8: http://screencast.com/t/rT2AUJGQiHJ
              IE9: http://screencast.com/t/KV79mw2qIv

              Show
              yip.ng added a comment - Can't reproduce. See videos: 1.8 FF: http://screencast.com/t/6XbXSh3o7 IE7: http://screencast.com/t/TWSFOD6L IE8: http://screencast.com/t/t2DD6ALe IE9: http://screencast.com/t/aYEQelNW143 3.2 FF: http://screencast.com/t/1llBjSmPash IE7: http://screencast.com/t/FuJHzkSCR5 IE8: http://screencast.com/t/rT2AUJGQiHJ IE9: http://screencast.com/t/KV79mw2qIv
              Hide
              Permalink
              yip.ng added a comment -

              From QA:

              ICE-2113 (all browsers) - fails manually in FF and IE - when clicking to render a modal popup panel, this is rendered, but not entirely visible on the page (see ICE-2113.png). This is also an issue w/Mojarra.

              Show
              yip.ng added a comment - From QA: ICE-2113 (all browsers) - fails manually in FF and IE - when clicking to render a modal popup panel, this is rendered, but not entirely visible on the page (see ICE-2113 .png). This is also an issue w/Mojarra.
              yip.ng made changes -
              Attachment ICE-2113.war [ 15341 ]
              Hide
              Permalink
              yip.ng added a comment - - edited

              Can't reproduce ICE-2113 regression either. Tested: FF18, IE7,8,9. See video: http://screencast.com/t/YG1boIfYd2sh. Further, they are different components. One is tooltip; one is modal popup. This JIRA shouldn't be the cause of the other's regression, if any.

              Show
              yip.ng added a comment - - edited Can't reproduce ICE-2113 regression either. Tested: FF18, IE7,8,9. See video: http://screencast.com/t/YG1boIfYd2sh . Further, they are different components. One is tooltip; one is modal popup. This JIRA shouldn't be the cause of the other's regression, if any.
              Hide
              Permalink
              Jerome Ruzol added a comment -

              As per Yip, the issue with ICE-2113 is not related to this JIRA and that he cannot reproduce the issue.

              Show
              Jerome Ruzol added a comment - As per Yip, the issue with ICE-2113 is not related to this JIRA and that he cannot reproduce the issue.
              Jerome Ruzol made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Cannot Reproduce [ 5 ]
              Jerome Ruzol made changes -
              Resolution Cannot Reproduce [ 5 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              yip.ng made changes -
              Attachment ICE-2113.war [ 15341 ]
              Hide
              Permalink
              Jerome Ruzol added a comment -

              regression ICE-2113 src code

              Show
              Jerome Ruzol added a comment - regression ICE-2113 src code
              Jerome Ruzol made changes -
              Attachment ICE-2113.zip [ 15345 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Resolving, the ICE-2113 regression is not related to this fix.

              Show
              Ken Fyten added a comment - Resolving, the ICE-2113 regression is not related to this fix.
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              yip.ng added a comment -

              I can finally reproduce the problem. It occurs only when running from QA subversion checkout. If you cut and paste source code into compat showcase, everything works fine. Strange.

              Show
              yip.ng added a comment - I can finally reproduce the problem. It occurs only when running from QA subversion checkout. If you cut and paste source code into compat showcase, everything works fine. Strange.
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified fix with ICEfaces EE-1.8.2.GA Build1 in IE6/7/8/9/FF18/Chrome24.

              Show
              Carmen Cristurean added a comment - Verified fix with ICEfaces EE-1.8.2.GA Build1 in IE6/7/8/9/FF18/Chrome24.
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: