Skip to main content

Search and Top Navigation

#7331 closed bug (worksforme)

Opened May 06, 2011 12:17AM UTC

Closed May 07, 2011 03:29AM UTC

Last modified May 11, 2011 12:05PM UTC

Suggestions scrolling minor fix

Reported by: michal.pokrywka Owned by:
Priority: minor Milestone: 1.9.0
Component: ui.autocomplete Version: 1.8.12
Keywords: Cc:
Blocked by: Blocking:
Description

Autocomplete with scrollable results has visual imperfection when using keyboard.

To reproduce: set menu (ul) max height to integer multiply of item (li a) height, add border to menu, scroll suggestions using keyboard up and down.

Test:

http://jsfiddle.net/Q5VmF/2/

Proposed fix included in jsfiddle

Attachments (0)
Change History (5)

Changed May 06, 2011 12:28PM UTC by scottgonzalez comment:1

status: newopen

Thanks, would you mind sending a pull request with this patch? You would need to apply the patch to jquery.ui.menu.js in master.

Changed May 06, 2011 08:46PM UTC by michal.pokrywka comment:2

Looks like it is fixed in master,

I modifed jsfiddle http://jsfiddle.net/Q5VmF/3/

to load jqueryui from github and "glitch" is no longer present.

Consider this report as invalid.

Changed May 07, 2011 03:29AM UTC by rdworth comment:3

resolution: → worksforme
status: openclosed

Changed May 11, 2011 09:17AM UTC by wuzz comment:4

Replying to [comment:2 michal.pokrywka]:

Looks like it is fixed in master, I modifed jsfiddle http://jsfiddle.net/Q5VmF/3/ to load jqueryui from github and "glitch" is no longer present. Consider this report as invalid.

try this with jq1.6 or 1.6.1 - scrolling to active item does not work anymore. works fine in 1.5.

Changed May 11, 2011 12:05PM UTC by scottgonzalez comment:5

Replying to [comment:4 wuzz]:

try this with jq1.6 or 1.6.1 - scrolling to active item does not work anymore. works fine in 1.5.

That's a completely unrelated problem. Please file a new ticket. Thanks.