You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I believe the code above was added to make sure e.g. (3) Item 1, My Item 2, Another Item 3 was not scrolled out of view when selecting/deselecting or navigating down the list of items in the pull down menu using ARROW DOWN. But changing the position of the cursor is just bad practice, and as demonstrated it doesn't play well when search is implemented.
See the following example: https://jsfiddle.net/tk5yp12x/
(Code also attached at the end).
This makes it difficult to adding additional letters to the search string.
The root of the problem is this:
Fit.UI/Controls/DropDown/DropDown.js
Line 2138 in 1408781
I believe the code above was added to make sure e.g.
(3) Item 1, My Item 2, Another Item 3
was not scrolled out of view when selecting/deselecting or navigating down the list of items in the pull down menu using ARROW DOWN. But changing the position of the cursor is just bad practice, and as demonstrated it doesn't play well when search is implemented.Notice that
Fit.Dom.SetCaretPosition(...)
is used in two places in DropDown.js. It's also used here:https://github.com/Jemt/Fit.UI/blob/14087818019a6859a25fb5e542d6e2e0ad398304/Controls/DropDown/DropDown.js#L2979C4-L2979C28
Code from JSFiddle:
The text was updated successfully, but these errors were encountered: