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

ResizeObserver: Add workaround for odd grid dimensions and zoom levels leading to "flickering" scrollbars #2389

Merged
merged 1 commit into from
May 6, 2021

Conversation

nstepien
Copy link
Contributor

@nstepien nstepien commented May 6, 2021

No description provided.

@nstepien nstepien requested a review from amanmahajan7 May 6, 2021 15:51
@nstepien nstepien self-assigned this May 6, 2021
@amanmahajan7 amanmahajan7 merged commit 59333b5 into canary May 6, 2021
@amanmahajan7 amanmahajan7 deleted the flicker branch May 6, 2021 16:01
gernotkogler pushed a commit to Garaio-REM/react-data-grid that referenced this pull request May 13, 2021
@ekeuus
Copy link
Contributor

ekeuus commented Aug 17, 2021

We are still seeing this in production for Windows users.

Should the fix rather be to round the width down/up always to prevent the flickering? The modulo of 0.5 seems kind of strange and highly specific imo.

I am unable to reproduce it even in a windows VM so I do not know the exact steps to repro. If either of you knows how to reproduce it code-wise, I would happily write tests and fix the issue.

@nstepien @amanmahajan7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants