-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Allow to revert automated status #36854
Allow to revert automated status #36854
Conversation
Good idea 👍🏽 For the UI we could do it like this:
|
65de366
to
fec199c
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.
Nice! To make it a bit less jumpy as @nimishavijay noted, it would be good to add a little animation here. Best of the row you click "Reset" on moving up and fading away, sort of so it fades into the input field. :)
I'm not a frontender so can't help with that. The other option I would see is closing the modal which also happens when "Set status message" is used. |
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.
Nice as is now, the animation could be something for the future. Nice one @nickvergessen! :)
fec199c
to
b90e4c3
Compare
b90e4c3
to
aca4de9
Compare
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
…he store Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
b48d718
to
63dc8cb
Compare
Summary
TODO
Follow up
Checklist