Wicket
  1. Wicket
  2. WICKET-5379

IE7: AutoCompleteTextField inside a ModalWindow shows auto complete dropdown behind ModalWindow

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.9.1, 6.11.0
    • Fix Version/s: 6.13.0, 7.0.0-M1
    • Component/s: None
    • Labels:
      None

      Description

      Similar scenario to WICKET-5378 - Create a ModalWindow, add an AutoCompleteTextField. Use IE7 and open the ModalWindow. Enter text in the AutoCompleteTextField, and depending on the size of your ModalWindow you might not see anything, because the dropdown suggestions list appears behind the ModalWindow (the z-index is wrong).

      Although this bug is present in 6.11.0, my quickstart uses 6.9.1 because it is easier to see the bug there as WICKET-5378 obscures it in 6.11.0 by moving the dropdown suggestions far away from the ModalWindow. But if you want to see it in 6.11.0, change the wicket.version in the POM and resize the ModalWindow to be larger (so that it at least partically covers the area where the dropdown suggestions list appears).

      1. WICKET-5379-6.12.0.zip
        12 kB
        Alex Grant
      2. WICKET-5379-quickstart.zip
        12 kB
        Alex Grant

        Issue Links

          Activity

          Alex Grant created issue -
          Alex Grant made changes -
          Field Original Value New Value
          Description Similar scenario to WICKET-5378 - Create a ModalWindow, add an AutoCompleteTextField. Open the ModalWindow. Enter text in the AutoCompleteTextField, and depending on the size of your ModalWindow you might not see anything, because the dropdown suggestions list appears behind the ModalWindow (the z-index is wrong).

          Although this bug is present in 6.11.0, my quickstart uses 6.9.1 because it is easier to see the bug there as WICKET-5378 obscures it in 6.11.0 by moving the dropdown suggestions far away from the ModalWindow. But if you want to see it in 6.11.0, change the wicket.version in the POM and resize the ModalWindow to be larger (so that it at least partically covers the area where the dropdown suggestions list appears).
          Similar scenario to WICKET-5378 - Create a ModalWindow, add an AutoCompleteTextField. Use IE7 and open the ModalWindow. Enter text in the AutoCompleteTextField, and depending on the size of your ModalWindow you might not see anything, because the dropdown suggestions list appears behind the ModalWindow (the z-index is wrong).

          Although this bug is present in 6.11.0, my quickstart uses 6.9.1 because it is easier to see the bug there as WICKET-5378 obscures it in 6.11.0 by moving the dropdown suggestions far away from the ModalWindow. But if you want to see it in 6.11.0, change the wicket.version in the POM and resize the ModalWindow to be larger (so that it at least partically covers the area where the dropdown suggestions list appears).
          Hide
          Alex Grant added a comment -

          Attached a quickstart, WICKET-5379-quickstart.zip.

          Show
          Alex Grant added a comment - Attached a quickstart, WICKET-5379 -quickstart.zip.
          Alex Grant made changes -
          Attachment WICKET-5379-quickstart.zip [ 12605842 ]
          Alex Grant made changes -
          Link This issue relates to WICKET-5378 [ WICKET-5378 ]
          Martin Grigorov made changes -
          Assignee Andrea Del Bene [ bitstorm ]
          Andrea Del Bene made changes -
          Link This issue Is contained by WICKET-5382 [ WICKET-5382 ]
          Andrea Del Bene made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 7.0.0 [ 12322958 ]
          Fix Version/s 6.12.0 [ 12324889 ]
          Resolution Fixed [ 1 ]
          Hide
          Alex Grant added a comment -

          This does not appear to be fixed in 6.12.0.
          I have attached a new quickstart (WICKET-5379-6.12.0.zip) with the pom.xml updated to 6.12.0 that still shows the problem.

          I do not have Reopen permissions so I'll just raise a separate Jira. I hope this is the correct process.

          Show
          Alex Grant added a comment - This does not appear to be fixed in 6.12.0. I have attached a new quickstart ( WICKET-5379 -6.12.0.zip) with the pom.xml updated to 6.12.0 that still shows the problem. I do not have Reopen permissions so I'll just raise a separate Jira. I hope this is the correct process.
          Alex Grant made changes -
          Attachment WICKET-5379-6.12.0.zip [ 12613426 ]
          Andrea Del Bene made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Andrea Del Bene made changes -
          Link This issue Is contained by WICKET-5413 [ WICKET-5413 ]
          Andrea Del Bene made changes -
          Link This issue Is contained by WICKET-5413 [ WICKET-5413 ]
          Hide
          Andrea Del Bene added a comment -

          IE 7 has (of course...) a problem computing the z-index of an element if its parent has no z-index explicitly set. I've changed the code of modal.js to add "style='z-index: 20001'" to the container div (the one with class='w_content_container') if the browser is IE7. The value 20001 is the same taken from modal.css (see div.wicket-modal ). However, I don't know where this value comes from....
          If anyone has a better idea please let me know.

          Show
          Andrea Del Bene added a comment - IE 7 has (of course...) a problem computing the z-index of an element if its parent has no z-index explicitly set. I've changed the code of modal.js to add "style='z-index: 20001'" to the container div (the one with class='w_content_container') if the browser is IE7. The value 20001 is the same taken from modal.css (see div.wicket-modal ). However, I don't know where this value comes from.... If anyone has a better idea please let me know.
          Hide
          Martin Grigorov added a comment -

          IE7 is not supported anymore. There are 4 newer versions of IE now.

          Andrea's improvement will be included in 6.13. Test current 6.13.0-SNAPSHOT and provide patch if it doesn't work for you.

          Show
          Martin Grigorov added a comment - IE7 is not supported anymore. There are 4 newer versions of IE now. Andrea's improvement will be included in 6.13. Test current 6.13.0-SNAPSHOT and provide patch if it doesn't work for you.
          Martin Grigorov made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 6.13.0 [ 12325564 ]
          Fix Version/s 6.12.0 [ 12324889 ]
          Resolution Fixed [ 1 ]
          Hide
          Alex Grant added a comment -

          Thank your for your help; I have tested the fix and it works for me.

          We're in the process of getting our customers off IE7, hopefully this will help to speed that process up.
          Is there somewhere I can point to that officially documents Wicket's supported browsers?

          I assume this is a little out of date now
          https://cwiki.apache.org/confluence/display/WICKET/FAQs#FAQs-WhichbrowsershavebeentestedwithWicketAJAX%3F

          Show
          Alex Grant added a comment - Thank your for your help; I have tested the fix and it works for me. We're in the process of getting our customers off IE7, hopefully this will help to speed that process up. Is there somewhere I can point to that officially documents Wicket's supported browsers? I assume this is a little out of date now https://cwiki.apache.org/confluence/display/WICKET/FAQs#FAQs-WhichbrowsershavebeentestedwithWicketAJAX%3F
          Alex Grant made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Hide
          Martin Grigorov added a comment -

          The Wiki page is updated.

          Show
          Martin Grigorov added a comment - The Wiki page is updated.
          Hide
          Alex Grant added a comment -

          Thanks again for your help.

          Show
          Alex Grant added a comment - Thanks again for your help.
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          3d 18h 10m 1 Andrea Del Bene 03/Oct/13 18:54
          Resolved Resolved Reopened Reopened
          40d 3h 39m 1 Andrea Del Bene 12/Nov/13 21:33
          Reopened Reopened Resolved Resolved
          2d 9h 50m 1 Martin Grigorov 15/Nov/13 07:24
          Resolved Resolved Closed Closed
          13h 4m 1 Alex Grant 15/Nov/13 20:28

            People

            • Assignee:
              Andrea Del Bene
              Reporter:
              Alex Grant
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development