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.

Sign up for GitHub

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

Jump to bottom

Remove workaround for browser scrolling bugs #2382

Open
m-akinc opened this issue Sep 4, 2024 · 2 comments
Open

Remove workaround for browser scrolling bugs #2382

m-akinc opened this issue Sep 4, 2024 · 2 comments
Labels
blocked Blocked on a third-party issue tech debt

Comments

@m-akinc
Copy link
Contributor

m-akinc commented Sep 4, 2024 •
edited
Loading

🧹 Tech Debt

See this Firefox bug and this (unrelated) Chromium bug.

To work around these issues, we added code to the table Virtualizer to assume scrolling had ended after 300ms without a scroll event. We should remove this logic when both the above bugs are fixed.
@m-akinc m-akinc added tech debt triage New issue that needs to be reviewed blocked Blocked on a third-party issue and removed triage New issue that needs to be reviewed labels Sep 4, 2024
@rajsite
Copy link
Member

rajsite commented Sep 11, 2024 •
edited
Loading

We should remove this logic when both the above bugs are fixed.

@m-akinc can you link to the PR and / or line this was introduced? There are no backlinks, etc, for someone to quickly inspect

@rajsite
Copy link
Member

rajsite commented Sep 11, 2024

@m-akinc Also fyi in the chromium bug looks like they requested additional info (some system info): https://issues.chromium.org/issues/364568425#comment2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocked Blocked on a third-party issue tech debt
Projects
Nimble Design System Priorities
Status: Defined/Ready to Pickup
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rajsite @m-akinc