ICEfaces
  1. ICEfaces
  2. ICE-7019

panelTabSet partialSubmit=false does not change tabs in Firefox

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1, EE-2.0.0.GA, 2.0.2
    • Fix Version/s: 3.0, EE-2.0.0.GA_P01
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Firefox, ICEfaces 2
    • Assignee Priority:
      P1

      Description

      When using partialSubmit="false" on the panelTabSet this allows the tabs content to execute the required validation. When testing this in Firefox with ICEfaces 2 the tab change does not occur and an error is reported in the Firebug console. Testing in IE this seems to work without any issues.

        Activity

        Hide
        Mircea Toma added a comment -

        In Firefox the event object is passed in as parameter to the registered event handler while in IE the event object is a global variable. The issue occurs when the event handler registered on the tab element is invoking the 'onclick' handler without any parameters, that works fine in IE but in FF iceSubmit() function is missing the event object since it should have been passed on when the handler was invoked.

        Show
        Mircea Toma added a comment - In Firefox the event object is passed in as parameter to the registered event handler while in IE the event object is a global variable. The issue occurs when the event handler registered on the tab element is invoking the 'onclick' handler without any parameters, that works fine in IE but in FF iceSubmit() function is missing the event object since it should have been passed on when the handler was invoked.
        Hide
        Mircea Toma added a comment -

        Modified event handler code rendered by PanelTabSetRenderer to pass on the event object when invoking 'onclick' handler.

        Show
        Mircea Toma added a comment - Modified event handler code rendered by PanelTabSetRenderer to pass on the event object when invoking 'onclick' handler.
        Hide
        Carmen Cristurean added a comment -

        Tested successfully on ICEfaces2.1 trunk revision #25049, using Tomcat6 and Firefox 3.6.

        Show
        Carmen Cristurean added a comment - Tested successfully on ICEfaces2.1 trunk revision #25049, using Tomcat6 and Firefox 3.6.

          People

          • Assignee:
            Carmen Cristurean
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: