ni / nimble

The NI Nimble Design System

https://nimble.ni.dev

MIT License

32 stars 8 forks source link

Remove workaround for browser scrolling bugs #2382

Open m-akinc opened 2 months ago

m-akinc commented 2 months ago

🧹 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.

rajsite commented 2 months ago

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 commented 2 months ago

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