ICEfaces
  1. ICEfaces
  2. ICE-4734

Modal background for popup not appearing for modal cascade

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.8.1
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Environment:
      Icefaces from trunk rev 19035

      Description

      Cascaded modal popups are rendered without background iframe (or background frame is disabled by js?)

      My case.
      One jspx with few modal popups included. Each are rendererd on demand.
      First I render and enable one then on button command I render second on top. This second popup is without iframe and bottom popup is accesiible for editing.

        Activity

        Krashan Brahmanjara created issue -
        Hide
        Krashan Brahmanjara added a comment -

        Test case based on code from ICE-4595

        Show
        Krashan Brahmanjara added a comment - Test case based on code from ICE-4595
        Krashan Brahmanjara made changes -
        Field Original Value New Value
        Attachment case.zip [ 11861 ]
        Hide
        Krashan Brahmanjara added a comment -

        I checked some older version of Icefaces 9since rev. 17324. It never worked correctly.

        Show
        Krashan Brahmanjara added a comment - I checked some older version of Icefaces 9since rev. 17324. It never worked correctly.
        Hide
        Krashan Brahmanjara added a comment -

        After some investigation i see that part of this problem is an z-index attribute.

        If user got few modalPopups and try to display them in cascade then new rendered popup got
        iframe and div with the same z-index attribute like modal in background - this value should be incremented.

        If user try to assign z-index attribute to modalPopup this attribute is ignored ant can't be changed.

        Show
        Krashan Brahmanjara added a comment - After some investigation i see that part of this problem is an z-index attribute. If user got few modalPopups and try to display them in cascade then new rendered popup got iframe and div with the same z-index attribute like modal in background - this value should be incremented. If user try to assign z-index attribute to modalPopup this attribute is ignored ant can't be changed.
        Hide
        Krashan Brahmanjara added a comment -

        The same and partially fixed in ICE-4685

        Show
        Krashan Brahmanjara added a comment - The same and partially fixed in ICE-4685
        Hide
        Krashan Brahmanjara added a comment -

        Fixed before rev.19147

        Show
        Krashan Brahmanjara added a comment - Fixed before rev.19147
        Hide
        Dirk Mahler added a comment - - edited

        The same problem exists in our project and is still reproducible with ICEfaces 1.8.2-RC1.

        Show
        Dirk Mahler added a comment - - edited The same problem exists in our project and is still reproducible with ICEfaces 1.8.2-RC1.
        Hide
        Dennis Bayer added a comment -

        Same for me. The modal background is not shown with the environment described at http://jira.icefaces.org/browse/ICE-4595 using IceFaces 1.8.2.

        Show
        Dennis Bayer added a comment - Same for me. The modal background is not shown with the environment described at http://jira.icefaces.org/browse/ICE-4595 using IceFaces 1.8.2.
        Hide
        Krashan Brahmanjara added a comment -

        Problem was solved about two months ago with refactoring of z-index attribute.
        Works correct on Icefaces from trunk version 19142.
        Try this version.

        Show
        Krashan Brahmanjara added a comment - Problem was solved about two months ago with refactoring of z-index attribute. Works correct on Icefaces from trunk version 19142. Try this version.

          People

          • Assignee:
            Unassigned
            Reporter:
            Krashan Brahmanjara
          • Votes:
            3 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated: