This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 974
Clicking "copy to clipboard" buttons need an animation #6297
Labels
accessibility
polish
Nice to have — usually related to front-end/visual tasks.
QA/checked-Linux
QA/checked-macOS
QA/checked-Win32
QA/checked-Win64
QA/test-plan-specified
release-notes/include
Milestone
Comments
bsclifton
added
accessibility
impact/low
polish
Nice to have — usually related to front-end/visual tasks.
labels
Dec 19, 2016
@bsclifton Is this reasonable to implement? The string would appear quickly and fade away. |
@bradleyrichter that looks perfect 😄 Having a div popup there and fade should be easy; would be a great chance to create a re-usable "fading alert" component (or similar) |
michalbe
added a commit
to michalbe/browser-laptop
that referenced
this issue
Feb 11, 2017
michalbe
added a commit
to michalbe/browser-laptop
that referenced
this issue
Feb 11, 2017
michalbe
added a commit
to michalbe/browser-laptop
that referenced
this issue
Feb 13, 2017
This was referenced Feb 26, 2017
This was referenced Mar 3, 2017
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
accessibility
polish
Nice to have — usually related to front-end/visual tasks.
QA/checked-Linux
QA/checked-macOS
QA/checked-Win32
QA/checked-Win64
QA/test-plan-specified
release-notes/include
Test Plan:
about:brave
Copy to clipboard
copy to clipboard
button now shows confirmation label when data is copied.Original issue details
This is a follow up for #5790
The copy to clipboard button works great; however, there are no UI behaviors reinforcing the fact that the button was clicked. For example, we could show the word "copied" and fade it in and out. Something to confirm to the user it was copied
For now, I believe this only exists here... but the behavior could be incorporated into our
about:styles
page (and it could be wrapped with a React component)cc: @bradleyrichter
The text was updated successfully, but these errors were encountered: