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.

Sign up for GitHub

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

Jump to bottom

Emoji list is wrong when cleaning filter #185

Closed
chamerling opened this issue Aug 27, 2020 · 1 comment
Closed

Emoji list is wrong when cleaning filter #185

chamerling opened this issue Aug 27, 2020 · 1 comment
Assignees
@RomaricMourgues
Labels
bug Something isn't working

Comments

@chamerling
Copy link
Contributor

chamerling commented Aug 27, 2020 •
edited by BenoitTallandier
Loading

Describe the bug

After the user cleans the emojione filter, the emojis are not all displayed

To Reproduce

  1. Open the emojione reaction
  2. Type some text to search an emoji
  3. Clear the text
  4. All the emojis are not displayed

Expected behavior

Display all the emojis when clearing text

Screenshots

emojione

Desktop (please complete the following information):

  • OS: OS X
  • Browser Chrome 84

user story

As a user, when I open emoji picker (edit message reaction for example), I can search "clap" and when I remove "clap" from the search, I can see all emoji as possible.
@BenoitTallandier BenoitTallandier added the bug Something isn't working label Sep 2, 2020
@RomaricMourgues RomaricMourgues mentioned this issue Sep 28, 2020
Merged
@RomaricMourgues
Copy link
Contributor

Notes for QA: now we are using emoji-mart emoji picker.

@hantt12 hantt12 closed this as completed Oct 1, 2020
@snyk-bot snyk-bot mentioned this issue Oct 20, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RomaricMourgues RomaricMourgues

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@chamerling @RomaricMourgues @BenoitTallandier @stephanevieira75 @hantt12