Wicket
  1. Wicket
  2. WICKET-5224

ModalWindow is not visible in Safari when opened from a link at the bottom of a large page

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.8.0
    • Fix Version/s: 6.13.0, 7.0.0
    • Component/s: wicket-extensions
    • Labels:
      None
    • Environment:
      Safari 5.1.7 (7534.57.2) Windows, Safari (iOS 6.1.3)

      Description

      I am not able to see a ModalWindow in Safari and I expect to see it centered on my view port.

      Steps:
      1. Start with a large web page with a link to open the ModalWindow at the bottom of the page. You need a large page where you have to scroll down to find the link.
      2. Click the link
      3. Observe that the mask for the ModalWindow displays, but the ModalWindow does not. Scrolling back up, re-sizing, and maximizing will still not display the ModalWindow.

        Issue Links

          Activity

          Hide
          Jered Myers added a comment - - edited

          When I remove the patch in WICKET-4948 on line 486 in modal.js, the problem seems to be resolved. I do not have a way to test on the Mac version of Safari. I am using Safari 5.1.7 on Windows.

          Show
          Jered Myers added a comment - - edited When I remove the patch in WICKET-4948 on line 486 in modal.js, the problem seems to be resolved. I do not have a way to test on the Mac version of Safari. I am using Safari 5.1.7 on Windows.
          Hide
          Jered Myers added a comment -

          I replicated this problem with an iPad (iOS 6.1.3) with Safari. There is a difference from my Windows Safari tests, in that I am able to see the ModalWindow when I scroll to the top of the page and then rapidly scroll back down to the bottom of the page. As soon as the window stops scrolling, the modal seems to move down out of site again...so the modal seems to move with the scroll bar.

          Show
          Jered Myers added a comment - I replicated this problem with an iPad (iOS 6.1.3) with Safari. There is a difference from my Windows Safari tests, in that I am able to see the ModalWindow when I scroll to the top of the page and then rapidly scroll back down to the bottom of the page. As soon as the window stops scrolling, the modal seems to move down out of site again...so the modal seems to move with the scroll bar.
          Hide
          Jered Myers added a comment -

          Has anybody been able to look into this yet? It is holding back a release for me as my application depends on the ModalWindow being available for Safari users. I can do a custom Wicket build and patch by removing the fix for WICKET-4948, but I would rather not do that if this is going to be addressed soon.

          Show
          Jered Myers added a comment - Has anybody been able to look into this yet? It is holding back a release for me as my application depends on the ModalWindow being available for Safari users. I can do a custom Wicket build and patch by removing the fix for WICKET-4948 , but I would rather not do that if this is going to be addressed soon.
          Hide
          Martin Grigorov added a comment -
          Show
          Martin Grigorov added a comment - Igor Vaynberg Ping
          Hide
          Igor Vaynberg added a comment -

          confirmed, safari mustve changed its behavior between than and now

          Show
          Igor Vaynberg added a comment - confirmed, safari mustve changed its behavior between than and now
          Hide
          Jered Myers added a comment -

          The quickstart passes now. Thanks!

          Show
          Jered Myers added a comment - The quickstart passes now. Thanks!
          Hide
          Igor Vaynberg added a comment -

          looks like this issue is back :/

          Show
          Igor Vaynberg added a comment - looks like this issue is back :/
          Hide
          Igor Vaynberg added a comment -

          reverted the revert, seems to have fixed it.

          Show
          Igor Vaynberg added a comment - reverted the revert, seems to have fixed it.

            People

            • Assignee:
              Igor Vaynberg
              Reporter:
              Jered Myers
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 1h
                1h
                Remaining:
                Remaining Estimate - 1h
                1h
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Development