Content-Length: 360590 | pFad | https://github.com/mastodon/mastodon/pull/5275

AE Track frequently used emojis in web UI by Gargron · Pull Request #5275 · mastodon/mastodon · GitHub
Skip to content
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

Merged
merged 4 commits into from
Oct 8, 2017

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Oct 8, 2017

@Gargron Gargron added the ui Front-end, design label Oct 8, 2017
@Gargron Gargron mentioned this pull request Oct 8, 2017
14 tasks
@Gargron Gargron force-pushed the feature-frequently-used-emojis branch from ca1e98b to ac87d88 Compare October 8, 2017 18:33
Copy link
Contributor

@nolanlawson nolanlawson left a 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:

  1. Frequently used emoji don't seem to survive a refresh? At least when I test locally
  2. I feel we should cut these off at 8 or 16 so that these don't grow without bound (see screenshot)

@nolanlawson
Copy link
Contributor

Seems they grow without bound:

screenshot 2017-10-08 11 43 30

@Gargron
Copy link
Member Author

Gargron commented Oct 8, 2017

@nolanlawson They grow to 4 lines, I took over that default from emoi-mart - do you think it should be two lines only?

@Gargron
Copy link
Member Author

Gargron commented Oct 8, 2017

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

@nolanlawson
Copy link
Contributor

I feel like 2 or maybe 3 lines is best but I'm not a heavy emoji user. Maybe good to ask others?

@Gargron Gargron merged commit 488584b into master Oct 8, 2017
@Gargron Gargron deleted the feature-frequently-used-emojis branch October 8, 2017 19:47
@MightyPork
Copy link
Contributor

I was trying to test this, but after deleting a custom emoji that was in my recents, the picker does not open anymore.

screenshot_20171008_221440

screenshot_20171008_221424

@MightyPork
Copy link
Contributor

I still managed to confirm that tooltips work though 👍

@Gargron
Copy link
Member Author

Gargron commented Oct 8, 2017

This is more of a general emoji-mart problem, it does need to be fixed, though separately.

@MightyPork
Copy link
Contributor

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

@MightyPork
Copy link
Contributor

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

rutan pushed a commit to rutan/mastodon that referenced this pull request Oct 11, 2017
* 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
cobodo pushed a commit to cobodo/mastodon that referenced this pull request Oct 20, 2017
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui Front-end, design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/mastodon/mastodon/pull/5275

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy