ICEfaces
  1. ICEfaces
  2. ICE-5204

Modal Popup Allows Tabbing Under Modal Window in Composite Component Showcase

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.8.2-EE-GA
    • Fix Version/s: None
    • Component/s: Bridge, ICE-Components
    • Labels:
      None
    • Environment:
      N/A

      Description

      When using a modal popup in the regular ICEfaces component-showcase, you are unable to tab through elements under the modal window. In the ICEpack icefaces-composite-comps-showcase, this is not the case, you are able to tab through elements under the modal window.

      There are two previous issues related to this ICE-3563 and ICE-4120.

      I performed some javascript debugging on the icefaces-composite-comps showcase and it appears that in style.js Ice.modal.running boolean is initialized properly the first time you use a modal popup (and you cannot tab underneath). However, I do not see the 'stop' function called that sets the 'running' boolean back to false.

      When the javascript debugging is used on the regular component-showcase the 'stop' method is always called when the modal popup is closed.

        Activity

        Hide
        Brad Kroeger added a comment -

        The ICEpack issue is IPCK-55.

        Show
        Brad Kroeger added a comment - The ICEpack issue is IPCK-55 .
        Hide
        Brad Kroeger added a comment -

        Another thing I noticed in the content that is rendered to the page after launching the modal popup:

        component-showcase:

        <![CDATA[Ice.autoPosition.stop('iceform:icepnltabset:0:modalPnlPop');Ice.autoCentre.stop('iceform:icepnltabset:0:modalPnlPop');Ice.iFrameFix.start('iceform:icepnltabset:0:modalPnlPop','/component-showcase/xmlhttp/blank');//1886643168]]>

        composite-comps-showcase:

        <![CDATA[Ice.autoPosition.stop('iceform:icepnltabset:0:j_id65');Ice.autoCentre.stop('iceform:icepnltabset:0:j_id65');Ice.iFrameFix.start('iceform:icepnltabset:0:j_id65','/icefaces-composite-comps-showcase/xmlhttp/blank');Ice.Focus.setFocus('iceform:icepnltabset:0:toggleButton');//133955275]]>

        The two differences here being the id was generated for composite-comps-showcase and Ice.Focus.setFocus is setting focus on the button that launched the popup. This doesnt happen in the standard showcase.

        Show
        Brad Kroeger added a comment - Another thing I noticed in the content that is rendered to the page after launching the modal popup: component-showcase: <![CDATA [Ice.autoPosition.stop('iceform:icepnltabset:0:modalPnlPop');Ice.autoCentre.stop('iceform:icepnltabset:0:modalPnlPop');Ice.iFrameFix.start('iceform:icepnltabset:0:modalPnlPop','/component-showcase/xmlhttp/blank');//1886643168] ]> composite-comps-showcase: <![CDATA [Ice.autoPosition.stop('iceform:icepnltabset:0:j_id65');Ice.autoCentre.stop('iceform:icepnltabset:0:j_id65');Ice.iFrameFix.start('iceform:icepnltabset:0:j_id65','/icefaces-composite-comps-showcase/xmlhttp/blank');Ice.Focus.setFocus('iceform:icepnltabset:0:toggleButton');//133955275] ]> The two differences here being the id was generated for composite-comps-showcase and Ice.Focus.setFocus is setting focus on the button that launched the popup. This doesnt happen in the standard showcase.
        Hide
        Abhijit Jere added a comment -

        I also observed that all buttons in the main window under modal popup can be activated by using accesskey. This almost makes accesskey attribute of the commandButton unusable. Is this being tracked by another bug?

        Show
        Abhijit Jere added a comment - I also observed that all buttons in the main window under modal popup can be activated by using accesskey. This almost makes accesskey attribute of the commandButton unusable. Is this being tracked by another bug?

          People

          • Assignee:
            Unassigned
            Reporter:
            Brad Kroeger
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: