Skip to content

fix: Chromium patch to avoid showing scrollbar too early#1978

Merged
ghiscoding merged 1 commit into
masterfrom
bugfix/chromium-scrollbar
May 15, 2025
Merged

fix: Chromium patch to avoid showing scrollbar too early#1978
ghiscoding merged 1 commit into
masterfrom
bugfix/chromium-scrollbar

Conversation

@ghiscoding

@ghiscoding ghiscoding commented May 15, 2025

Copy link
Copy Markdown
Owner
  • another trial at fixing or I should patching the issue I first tried to fix in fix: incorrect grid container box size calculation border not showing #1961, this new patch seems to work a lot better though and is much more safe which is just to add 0.2px to the grid container but only after the grid columns autosize is called.
  • this is just a patch because in the end, I'm pretty sure that it's a Chromium bug because this issue never shows in Firefox and I would guess that it's caused by a floating precision which could differ on different computers. There are some open Chromium bugs for similar things but it could take a very long time before any fix comes in, so let's patch it since we can

image

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@codecov

codecov Bot commented May 15, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.0%. Comparing base (32da143) to head (a9c45f7).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1978   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         195      195           
  Lines       35920    35921    +1     
  Branches    10647    10648    +1     
=======================================
+ Hits        35920    35921    +1     
Flag Coverage Δ
angular 100.0% <ø> (ø)
universal 100.0% <100.0%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ghiscoding ghiscoding merged commit 0744e75 into master May 15, 2025
13 checks passed
@ghiscoding ghiscoding deleted the bugfix/chromium-scrollbar branch May 15, 2025 05:09
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.

1 participant