Skip to main content

Search and Top Navigation

#9991 closed bug (fixed)

Opened April 23, 2014 06:13PM UTC

Closed May 06, 2014 01:07PM UTC

Menu: Scroll on cursor down doesn't fully show the newly focused item

Reported by: jzaefferer Owned by:
Priority: major Milestone: 1.11.0
Component: ui.menu Version: git (not yet released)
Keywords: regression Cc:
Blocked by: Blocking:
Description

This works fine in 1.10.4 with the old menu style: http://view.jqueryui.com/1.10.4/demos/autocomplete/maxheight.html

Its broken in master: http://view.jqueryui.com/master/demos/autocomplete/maxheight.html

To reproduce, type "a", then cursor down 6 times.

Can also be seen on the scrollable menu on the visual test page for menu: http://view.jqueryui.com/master/tests/visual/menu/menu.html

This was introduced with the new menu style: https://github.com/jquery/jquery-ui/commit/9910e938aad1090339a2c7f60693093ee18aba82

Attachments (0)
Change History (3)

Changed April 24, 2014 11:17AM UTC by jzaefferer comment:1

milestone: none1.11.0
priority: minormajor
status: newopen
version: 1.10.4git

This also affect selectmenu.

Changed April 24, 2014 05:56PM UTC by jzaefferer comment:2

keywords: → regression

Changed May 06, 2014 01:07PM UTC by Jörn Zaefferer comment:3

resolution: → fixed
status: openclosed

Menu: Use item.outerHeight() in _scrollIntoView

Fixes #9991

Changeset: b22280385c05eaf10f4d480c546906b85aa011e1