-
-
Notifications
You must be signed in to change notification settings - Fork 7.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Track frequently used emojis in web UI #5275
Conversation
Gargron
commented
Oct 8, 2017
•
edited
Loading
edited
- Click in emoji picker as well as selection of autosuggest increase counter for that emoji
- Settings reducer gets some new events that track whether the settings are "dirty" (unsaved changes)
- Settings save now debounced by 5 seconds (saves API calls)
- Fix [emojos] emoji picker "frequently used" should update with used shortcodes #5203
- Fix Emojos in the new picker don't have tooltips #5144
ca1e98b
to
ac87d88
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything LGTM except for a couple things:
- Frequently used emoji don't seem to survive a refresh? At least when I test locally
- I feel we should cut these off at 8 or 16 so that these don't grow without bound (see screenshot)
@nolanlawson They grow to 4 lines, I took over that default from emoi-mart - do you think it should be two lines only? |
Also, save is debounced by 5 seconds, so yeah if you immediately refresh they won't save, but I doubt that will happen in practice |
I feel like 2 or maybe 3 lines is best but I'm not a heavy emoji user. Maybe good to ask others? |
I still managed to confirm that tooltips work though 👍 |
This is more of a general emoji-mart problem, it does need to be fixed, though separately. |
yeah just saying I can't test any further because my picker now always crashes. this is stored server-side? i hope i can find some "phpmyadmin" that works with postgre :S |
good, managed to find and delete the JSON object in web_settings using rubymine. Other than that, seems to work okay! Shortcodes from the suggestions bar are added correctly |
* Track frequently used emojis in web UI * Persist emoji usage, but debounce commits to the settings API * Fix mastodon#5144 - Add tooltips to picker * Display only 2 lines of frequently used emojis
* Track frequently used emojis in web UI * Persist emoji usage, but debounce commits to the settings API * Fix mastodon#5144 - Add tooltips to picker * Display only 2 lines of frequently used emojis