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

Copied notice on app passwords doesn't take reduced motion into account #6199

Closed
mary-ext opened this issue Nov 10, 2024 · 3 comments · Fixed by #6220
Closed

Copied notice on app passwords doesn't take reduced motion into account #6199

mary-ext opened this issue Nov 10, 2024 · 3 comments · Fixed by #6220
Labels
bug Something isn't working

Comments

@mary-ext
Copy link
Contributor

mary-ext commented Nov 10, 2024

Steps to Reproduce

  1. Enable reduced motion
  2. Open app passwords settings screen
  3. Create a new app password
  4. Copy the app password

The copied notice should remain on screen for a little bit, but it does not, it pops in for 0.75s and immediately disappears afterwards.

Attachments

No response

What platform(s) does this occur on?

Web (Desktop), Web (Mobile)

Device Info

Firefox 134 (nightly); Linux (Arch Linux; Linux aster 6.11.7-2-cachyos #1 SMP PREEMPT_DYNAMIC Fri, 08 Nov 2024 17:23:46 +0000 x86_64 GNU/Linux)

What version of the app are you using?

Build version: 1.94.0; Bundle info: dev (prod); Bundle date: 24111000; Platform: web; Platform version: undefined

Additional Information

No response

@mary-ext mary-ext added the bug Something isn't working label Nov 10, 2024
@khuddite
Copy link
Contributor

I will look into this issue

@khuddite
Copy link
Contributor

I am not able to reproduce the issue. I've tested it on my macOs with on/off reduced motion and it seemed the copied notice stayed for a few secs.

CleanShot.2024-11-10.at.15.51.25.mp4

@mary-ext
Copy link
Contributor Author

New app passwords screen, the one that's currently on main branch, https://main.bsky.dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants