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

Hide Emoji list after leaving comment reaction #9563

Closed
TildaDares opened this issue Apr 26, 2021 · 5 comments · Fixed by #9564
Closed

Hide Emoji list after leaving comment reaction #9563

TildaDares opened this issue Apr 26, 2021 · 5 comments · Fixed by #9564
Labels
feature explains that the issue is to add a new feature

Comments

@TildaDares
Copy link
Member

Please describe the desired behavior.
After leaving a reaction on a comment, the emoji container should hide/close.

Additional context (optional)

Screen.Recording.2021-04-26.at.21.03.27.mov
@TildaDares TildaDares added the feature explains that the issue is to add a new feature label Apr 26, 2021
@govindgoel
Copy link
Member

@TildaDares this is great

@jywarren
Copy link
Member

Hi @TildaDares i really appreciate this one, but I am also unsure -- i often use multiple reactions and so i kind of like that it stays open? What do other people think? I'm not set either way.

@CQW-Code
Copy link

I sometimes do use multiple reactions so like that it stays open; I don't know if it's feasible but maybe a time delay on closing the reaction/emoji list? ( that may or may not be a plus for low vision and blind people who may use publiclab's site).

@jywarren
Copy link
Member

That's such a great idea, thanks @CQW-Code. I think that's a relatively easy change as well, what do you think @TildaDares?

Maybe we need to help the first-timer at #9574 add a setTimeout() delay?

@TildaDares
Copy link
Member Author

@jywarren The issue at #9574 is different from this one. The setTimeout() will be added to the PR for this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature explains that the issue is to add a new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants