-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Better Debug for Storybook + Popper API Improvements #155
Better Debug for Storybook + Popper API Improvements #155
Conversation
Watermelon AI SummaryThis PR introduces enhancements to the Popper component within the Storybook environment, improving functionality and styling for better on-screen positioning, and adding a ScrollPlayground component for more effective debugging and testing of pop-up related features. GitHub PRs
ui is an open repo and Watermelon will serve it for free. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #155 +/- ##
=======================================
Coverage 90.62% 90.62%
=======================================
Files 5 5
Lines 32 32
Branches 8 8
=======================================
Hits 29 29
Misses 3 3 ☔ View full report in Codecov by Sentry. |
return <div> | ||
<Popper open={true} placement="bottom-start" popperName="dropdown" pop={<PopElem/>}> | ||
return <div className='relative'> | ||
<Popper open={false} placement="bottom-start" popperName="dropdown" pop={<PopElem/>}> |
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.
was the previous open
flag a mistake? i havent looked at dropdown component yet
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.
dropdown is still a test component - its not part of the library yet :D
@@ -0,0 +1,34 @@ | |||
import React, {useState} from 'react'; |
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.
this is for storybook ya?
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.
correct 👍
This PR improves popper component by adding the right combination of hooks to make popper work in generic scenarios
Also added a scroll container playground component that helps us test/debug popper related components