-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
[bugfix] Fix slice width calculation in dashboard #5867
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #5867 +/- ##
==========================================
+ Coverage 63.73% 63.79% +0.05%
==========================================
Files 374 374
Lines 23325 23462 +137
Branches 2607 2607
==========================================
+ Hits 14867 14968 +101
- Misses 8445 8481 +36
Partials 13 13
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤦♂️ thanks for fixing this
(cherry picked from commit 166058d)
My previous fix (#5839) was incorrect.
It only works for slices that spans 1-2 columns, but compute less width than it should be for the slices that span >2 columns.
The previous formula did not take
GRID_GUTTER_SIZE
into account. A slice withn
columns also containsn-1
gutter.The formula for width is updated to
@john-bodley @graceguo-supercat