ICEfaces
  1. ICEfaces
  2. ICE-10116

icecore:navigationNotifier issues on IE, Firefox

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P02
    • Fix Version/s: 4.0, EE-3.3.0.GA_P03
    • Component/s: ICECORE-Components
    • Labels:
      None
    • Environment:
      ICEfaces4 trunk rev. 41589
      Browsers: IE, Firefox
      Server: Tomcat7
    • Assignee Priority:
      P1

      Description

      QA test application:
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/navigationNotifier

      Issues found:
      1) This component works only in Chrome, but not in IE or Firefox.

      2) If using icecore:navigationNotifier with an ace: component on the page (as example ace:panel), there is a browser console error on page load or when mousing over the page (all browsers):

          Uncaught TypeError: undefined is not a function ace-jquery.js.jsf?ln=icefaces.ace&v=4_0_0_140704:3040

        Activity

        Hide
        Liana Munroe added a comment -

        Our test application and the showcase demo both expect that notification will occur (showcase) or a counter will be incremented (QA test app) only when the Browser Back/Forward buttons are used.

        Issues:
        IE 10 - QA test application passes consistently but the in the showcase application it passes about 50% of the time. In fact, every second time.

        IE 11 - The tag does not function at all in either application.

        FF 31 - The notification is triggered when the page is reloaded, as well as when the Browser Back/Forward buttons are used.

        The QA test app can be found at:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/navigationNotifier

        Show
        Liana Munroe added a comment - Our test application and the showcase demo both expect that notification will occur (showcase) or a counter will be incremented (QA test app) only when the Browser Back/Forward buttons are used. Issues: IE 10 - QA test application passes consistently but the in the showcase application it passes about 50% of the time. In fact, every second time. IE 11 - The tag does not function at all in either application. FF 31 - The notification is triggered when the page is reloaded, as well as when the Browser Back/Forward buttons are used. The QA test app can be found at: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/navigationNotifier
        Hide
        Mircea Toma added a comment -

        Re-implemented icecore:navigationNotifier to use HTML5 features. Only the browsers that support HTML5's new history API will actually be able to detect back/forward navigation.
        Here's the list of supported browsers: Chrome, Safari, Firefox, IE10, and IE11.

        Show
        Mircea Toma added a comment - Re-implemented icecore:navigationNotifier to use HTML5 features. Only the browsers that support HTML5's new history API will actually be able to detect back/forward navigation. Here's the list of supported browsers: Chrome, Safari, Firefox, IE10, and IE11.
        Hide
        Carmen Cristurean added a comment - - edited

        Rev. 42540: in the QA test application, the component is not fully functional in Chrome; the counter is updated only when clicking to go back, and not when clicking the "forward" button; not an issue in IE10/11 or Firefox31.
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/navigationNotifier

        Show
        Carmen Cristurean added a comment - - edited Rev. 42540: in the QA test application, the component is not fully functional in Chrome; the counter is updated only when clicking to go back, and not when clicking the "forward" button; not an issue in IE10/11 or Firefox31. http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/navigationNotifier
        Hide
        Mircea Toma added a comment - - edited

        Improved navigation and page reload detection algorithm. Also worked around Chrome's limitation in updating the history state during page unload.

        Show
        Mircea Toma added a comment - - edited Improved navigation and page reload detection algorithm. Also worked around Chrome's limitation in updating the history state during page unload.
        Hide
        Carmen Cristurean added a comment -

        Verified with rev. 42551 in Chrome37, IE10, FF31 in showcase and QA test app.

        Show
        Carmen Cristurean added a comment - Verified with rev. 42551 in Chrome37, IE10, FF31 in showcase and QA test app.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Carmen Cristurean
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: