Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#8481 closed bug (fixed)

Position in flip mode does not take into consideration scrollTop and scrollLeft

Reported by: roman.chizhevsky Owned by:
Priority: minor Milestone: 1.9.0
Component: ui.position Version: 1.8.22
Keywords: Cc:
Blocked by: Blocking:

Description

Position utility - http://wiki.jqueryui.com/w/page/12138026/Position

  1. jQuery - 1.7.2, Jquery UI 1.8.22
  2. IE9, FF
  3. Windows 7 Enterprise
  4. Instruction on how to reproduce the issue:
    1. Create an example of Use position utility with flip functionality like this one http://jsfiddle.net/DV6j7/1/
    2. Use for example vertical scrollbar to scroll down
    3. See that the positionable elements are not flipped down and hidden out from the view port
  5. Flipping from the right side and from the bottom side correctly consider the scrolling, but left and top flipping does not work.
  1. Working demo/test: http://jsfiddle.net/DV6j7/1/

Change History (4)

comment:1 Changed 7 years ago by roman.chizhevsky

To help with fix, you probably should know that there is a relevant ticket already fixed before: #7211 http://bugs.jqueryui.com/ticket/7211

Though, the change log shows that you fixed it only for the fit collision mode, not for flip: https://github.com/jquery/jquery-ui/commit/5e44b3ce8851f62a38ce9211c6721e1050e2dabf

Thank you, -Roman

comment:2 Changed 7 years ago by Scott González

Component: ui.coreui.position

comment:3 Changed 7 years ago by Scott González

Resolution: fixed
Status: newclosed

comment:4 Changed 7 years ago by Scott González

Position: Properly handle flip collision when the window is scrolled. Fixes #8481 - Position in flip mode does not take into consideration scrollTop and scrollLeft.

Changeset: 651460792b3aeb7f05cbad5dc8f7b2276bdd9b29

Note: See TracTickets for help on using tickets.