Kicker:
Dash:
Settings:
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
two small feedback pieces:
Agreeing with Marco re "Most used", also because I can see "Popular" being an i18n challenge. In other languages the word that most closely maps to Popular might have an even more different sentiment from actually-most-used.
I'm kind of confused by the settings UI. Show ... first? As opposed to what? The bottom?
"most used" is not what it does - it is a combination of most frequently and most recently used.
As for 'first', you are right. The source of the word was the possibility to show an infinite list.
The combo box now has:
Recently used |
Often used |
Checkboxes are "recent"/"often used"
And the same for the text in the menu
p.s. As for the default behaviour, I would not make it the default just yet. We need to test whether some things regarding scoring should be changed.