ICEfaces
  1. ICEfaces
  2. ICE-10413

Navigation issue using themeSelect demo then showcase search field

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: EE-4.0.0.GA, 4.1
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces 4 trunk r43592. Tomcat 7, all browsers.
    • Assignee Priority:
      P3

      Description

      To reproduce:
      Use showcase ace:themeSelect demo to change the theme.
      In the showcase demo search field type tabSet to display the ace:tabSet demo selections.
      Choose the ace:tabSet > Server Side demo.
      Click on the 2nd tab in the Server Side demo.
      The page reloads with the themeSelect Demo.
      Change the theme again and you are redirected back to the ace:tabSet > Server Side demo.
      Note**
      This issue does not occur when changing the theme, then clicking on the sidebar link to navigate to the ace:tabSet demos. It only occurs when using the Showcase search field. This issue occurs with all of the tabSet demos except ace:tabSet Client Side.
      A few attempts were made at reproducing this issue with demos other than those in ace:tabSet. No further occurrences were found but testing was limited to only a few samples and was not comprehensive.

        Activity

        Hide
        Carmen Cristurean added a comment - - edited

        Issue reproduced with IF4 trunk rev. 43997 (FF34, Chrome40, IE11).

        Show
        Carmen Cristurean added a comment - - edited Issue reproduced with IF4 trunk rev. 43997 (FF34, Chrome40, IE11).
        Hide
        Ken Fyten added a comment -

        Note that you need to follow the steps precisely to see this issue. Very strange.

        Show
        Ken Fyten added a comment - Note that you need to follow the steps precisely to see this issue. Very strange.
        Hide
        Mircea Toma added a comment -

        The tabset JS code submits the form with execute=@all, render=@all attributes when switching tabs. Because of that the form used for navigation is also used.

        The fix modifies tabset.js to use ice.sef (execute=@form, render=@all) function when switching tabs.

        Show
        Mircea Toma added a comment - The tabset JS code submits the form with execute=@all , render=@all attributes when switching tabs. Because of that the form used for navigation is also used. The fix modifies tabset.js to use ice.sef ( execute=@form, render=@all ) function when switching tabs.
        Hide
        Carmen Cristurean added a comment -

        Testing with EE-4.0.0 r41049/ IF4 trunk r44151/ browsers: IE11, FF34, Chrome40: the problem from description can no longer be reproduced.

        Reopening because on ace:tabSet > Proxy demo, the tabs cannot be changed anymore, that may be a regression of the above fix (all browsers).
        No errors are visible in browser console or in the server logs.

        Show
        Carmen Cristurean added a comment - Testing with EE-4.0.0 r41049/ IF4 trunk r44151/ browsers: IE11, FF34, Chrome40: the problem from description can no longer be reproduced. Reopening because on ace:tabSet > Proxy demo, the tabs cannot be changed anymore, that may be a regression of the above fix (all browsers). No errors are visible in browser console or in the server logs.
        Hide
        Mircea Toma added a comment -

        Modified NavigationController so that navigation is avoided when the request is a post-back with no 'exp' parameter defined.
        Rolled back tabset.js change to use once again full submit when switching tabs.

        Show
        Mircea Toma added a comment - Modified NavigationController so that navigation is avoided when the request is a post-back with no 'exp' parameter defined. Rolled back tabset.js change to use once again full submit when switching tabs.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk r44179. Tomcat 7, FF 34, Chrome 40, IE 11, 10, 9, 8.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r44179. Tomcat 7, FF 34, Chrome 40, IE 11, 10, 9, 8.
        Hide
        Jack Van Ooststroom added a comment -

        Verified with Poll as well.

        Show
        Jack Van Ooststroom added a comment - Verified with Poll as well.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Liana Munroe
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: