#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
- jQuery - 1.7.2, Jquery UI 1.8.22
- IE9, FF
- Windows 7 Enterprise
- Instruction on how to reproduce the issue:
- Create an example of Use position utility with flip functionality like this one http://jsfiddle.net/DV6j7/1/
- Use for example vertical scrollbar to scroll down
- See that the positionable elements are not flipped down and hidden out from the view port
- Flipping from the right side and from the bottom side correctly consider the scrolling, but left and top flipping does not work.
- Working demo/test: http://jsfiddle.net/DV6j7/1/
Change History (4)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
Component: | ui.core → ui.position |
---|
comment:3 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:4 Changed 11 years ago by
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.
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