ICEfaces
  1. ICEfaces
  2. ICE-6538

Modal Popup Launched from ace:tabSet Not Covering Separate Div with ice:menuBar

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1, EE-1.8.2.GA_P03
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      JSF 2.0.3 ICEfaces 2

      Description

      I have the following layout in my template. The 'menu' insert contains an h:form with an ice:menuBar nested inside. When I launch a modal popup from the 'tabset' insert, which contains an ace:tabSet component, the modal pane is only covering the winDiv h:panelGroup, not the menu insert.

      When the modal popup is activated and you interact with the ice:menuBar that is not covered, none of the actions are fired. Looks like this is just a matter of the modal pane covering that region as well.

          <h:body styleClass="ice-skin-sam">

              <!-- MENU -->
              <ui:insert name="menu"/>

              <h:panelGroup id="winDiv" layout="block">
                  <!-- TABSET -->
                  <ui:insert name="tabset"/>
              </h:panelGroup>

          <h:outputStylesheet library="org.icefaces.component.skins" name="sam.css" />
          </h:body>

        Activity

        Hide
        yip.ng added a comment -

        Do you have a sample test app. for this?

        Show
        yip.ng added a comment - Do you have a sample test app. for this?
        Hide
        Brad Kroeger added a comment -

        You can test on the application I previously sent you the link to on repo that I think you've already got running. From the menu, launch the tab for 'Screen 1', hit the 'Retrieve' button, select a row and hit the 'details' button to launch the modal popup.

        Show
        Brad Kroeger added a comment - You can test on the application I previously sent you the link to on repo that I think you've already got running. From the menu, launch the tab for 'Screen 1', hit the 'Retrieve' button, select a row and hit the 'details' button to launch the modal popup.
        Hide
        Brad Kroeger added a comment -

        When the modal popup is launched, also seeing vertical and horizontal scrollbars added to the browser suggesting the modal overlay is talking up a very large space.

        Show
        Brad Kroeger added a comment - When the modal popup is launched, also seeing vertical and horizontal scrollbars added to the browser suggesting the modal overlay is talking up a very large space.
        Hide
        yip.ng added a comment - - edited

        Same mistake we often made in assuming positions are always relative to the document, forgetting or unaware of possible intervening positioning contexts and layers. In this case the YUI tabset has a relative position, introducing an extra positioning context.

        Fixed. See screenshot 1.

        Revision: 23919


        Modified : /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js

        Show
        yip.ng added a comment - - edited Same mistake we often made in assuming positions are always relative to the document, forgetting or unaware of possible intervening positioning contexts and layers. In this case the YUI tabset has a relative position, introducing an extra positioning context. Fixed. See screenshot 1. Revision: 23919 Modified : /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js
        Hide
        Ken Fyten added a comment -

        Please check for the same issue on icefaces/trunk.

        Show
        Ken Fyten added a comment - Please check for the same issue on icefaces/trunk.
        Hide
        yip.ng added a comment -

        Revision: 23923


        Modified : /icefaces/trunk/icefaces/bridge/lib/extras/style.js

        Show
        yip.ng added a comment - Revision: 23923 Modified : /icefaces/trunk/icefaces/bridge/lib/extras/style.js

          People

          • Assignee:
            yip.ng
            Reporter:
            Brad Kroeger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: