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

Adjust the double-click blocking time to make it a little less noticeable #7444

Closed
jonboiser opened this issue Aug 5, 2020 · 1 comment · Fixed by #11719
Closed

Adjust the double-click blocking time to make it a little less noticeable #7444

jonboiser opened this issue Aug 5, 2020 · 1 comment · Fixed by #11719
Assignees
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) P2 - normal Priority: Nice to have TAG: performance User-facing performance TAG: ux update Improved user-facing feature

Comments

@jonboiser
Copy link
Contributor

jonboiser commented Aug 5, 2020

Observed behavior

The fix for #3914 adds a perceivable delay on the Learn page between when the page loads (for fast connections at least) and when the user can interact with things.

While it was meant to prevent double-clicking the same space held by a content card in the the Channels navigation flow, this delay applies globally, e.g. clicking the navigation tabs and other links.

Expected behavior

Personally, I would prefer to not have this delay at all, but if we need to keep it, I would like to have the double-click blocking last for a shorter duration and to be limited only to the area of the content cards.

User-facing consequences

Errors and logs

Steps to reproduce

Context

@indirectlylit indirectlylit added P2 - normal Priority: Nice to have TAG: performance User-facing performance TAG: ux update Improved user-facing feature labels Aug 6, 2020
@indirectlylit indirectlylit added this to the upcoming patch milestone Aug 6, 2020
@jonboiser jonboiser added the APP: Learn Re: Learn App (content, quizzes, lessons, etc.) label Feb 20, 2021
@rtibbles rtibbles self-assigned this Jan 11, 2024
@marcellamaki
Copy link
Member

Closed with #11719

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) P2 - normal Priority: Nice to have TAG: performance User-facing performance TAG: ux update Improved user-facing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants