Enable setting the first option in the list as active.#1796
Conversation
|
When the user types into the search bar, the first item is always highlighted. When the user then uses backspace (for example, to correct a typo), the same item stays highlighted even though it may not be the top-most item anymore. This is arguably confusing UX. For example, the user types "Anna", highlighting the person named Anna. Then the user uses backspace to correct a typo. When the user pressed backspace twice (resulting in "An" as a search query), Anna remains highlighted even though there could be many more persons before (for example "Anastasia"). This option opts-in to always highlight the first element in the search results, independent of whether the user used backspace or not. |
|
This pull request is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days |
|
Could the maintainers please indicate whether they intend to review/merge this, or should I close this PR? This is still a concern from my point of view. Thanks! |
risadams
left a comment
There was a problem hiding this comment.
dist files should not be modified here. they will be reset during a build
| } | ||
| } | ||
| // activate | ||
| self.hasOptions = results.items.length > 0 || ( has_create_option && self.settings.showAddOptionOnCreate ) || self.settings.setFirstOptionActive; |
There was a problem hiding this comment.
TODO: We should look at a cleanup of these if checks into something that is both easier to understand as a dev, and avoids edge cases
CHORE (testing, documentation): Do you have any updates to unit tests or an example that can show the expected behavior?
…-active Copying from #1796 to resolve conflicts
|
copied and merged in #1874 |
|
Thank you a lot @risadams ! |
No description provided.