-
Notifications
You must be signed in to change notification settings - Fork 5
Migrate pattern library site and tests to Tailwind v4 #2084
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
Conversation
| stickyHeaderHeight, | ||
| scrollEl.scrollTop, | ||
| 1, | ||
| 2, |
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.
Expected height is 50, actual height before was 49 but is now 48.
| // But when in a Panel, and the Panel is within a height-constrained | ||
| // element, the total Panel height will not exceed parent height (200px) | ||
| assert.equal( | ||
| assert.isBelow( |
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.
Actual value is now 198, which matches the test description of being below 200.
047a5a5 to
bd762b5
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2084 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 71 71
Lines 1364 1364
Branches 509 509
=========================================
Hits 1364 1364 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
bd762b5 to
499d04b
Compare
Migrate the pattern library website and tests to use Tailwind v4. The npm package remains usable with either Tailwind v3 or v4 downstream. The changes here are similar to those in h (hypothesis/h#9877).
499d04b to
a30d0a7
Compare
Migrate the pattern library website and tests to use Tailwind v4. The npm package remains usable with either Tailwind v3 or v4 downstream. The changes here are similar to those in h (hypothesis/h#9877).