deac/chosen
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix position issue when using up and down arrows to navigate the results.

Browse source
This commit is contained in:
Patrick Filler 2011-07-13 15:33:10 -04:00
parent ea87ad4df3
commit d5b62668ca
2 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chosen/chosen.jquery.js
View file

@ -280,7 +280,7 @@
maxHeight = parseInt(this.search_results.css("maxHeight"), 10); maxHeight = parseInt(this.search_results.css("maxHeight"), 10);
visible_top = this.search_results.scrollTop(); visible_top = this.search_results.scrollTop();
visible_bottom = maxHeight + visible_top; visible_bottom = maxHeight + visible_top;
high_top = this.result_highlight.position().top; high_top = this.result_highlight.position().top + this.search_results.scrollTop();
high_bottom = high_top + this.result_highlight.outerHeight(); high_bottom = high_top + this.result_highlight.outerHeight();
if (high_bottom >= visible_bottom) { if (high_bottom >= visible_bottom) {
return this.search_results.scrollTop((high_bottom - maxHeight) > 0 ? high_bottom - maxHeight : 0); return this.search_results.scrollTop((high_bottom - maxHeight) > 0 ? high_bottom - maxHeight : 0);

6
coffee/chosen.jquery.coffee
View file

@ -251,16 +251,12 @@ class Chosen
visible_top = @search_results.scrollTop() visible_top = @search_results.scrollTop()
visible_bottom = maxHeight + visible_top visible_bottom = maxHeight + visible_top
high_top = @result_highlight.position().top high_top = @result_highlight.position().top + @search_results.scrollTop()
high_bottom = high_top + @result_highlight.outerHeight() high_bottom = high_top + @result_highlight.outerHeight()
#console.log visible_top, visible_bottom, high_top, high_bottom
if high_bottom >= visible_bottom if high_bottom >= visible_bottom
#console.log "bottom is greater than bottom"
@search_results.scrollTop if (high_bottom - maxHeight) > 0 then (high_bottom - maxHeight) else 0 @search_results.scrollTop if (high_bottom - maxHeight) > 0 then (high_bottom - maxHeight) else 0
else if high_top < visible_top else if high_top < visible_top
#console.log "top is less than top"
@search_results.scrollTop high_top @search_results.scrollTop high_top
result_clear_highlight: -> result_clear_highlight: ->