-
-
Notifications
You must be signed in to change notification settings - Fork 367
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
chore: Add toolbar
example
#970
Conversation
This pull request is being automatically deployed with Vercel (learn more). ariakit – ./🔍 Inspect: https://vercel.com/ariakit/ariakit/3MKRjMnyBiBXkLDrSrx5YpW5dxn9 [Deployment for ccd0a51 failed] reakit – ./🔍 Inspect: https://vercel.com/ariakit/reakit/5MgQe1RVfgyn4HHk63yDyb6n9kTR [Deployment for ccd0a51 canceled] |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
Codecov Report
@@ Coverage Diff @@
## v2 #970 +/- ##
===========================================
+ Coverage 42.37% 62.35% +19.97%
===========================================
Files 155 160 +5
Lines 4113 4133 +20
Branches 1160 1159 -1
===========================================
+ Hits 1743 2577 +834
+ Misses 2369 1555 -814
Partials 1 1
Continue to review full report at Codecov.
|
hey @diegohaz is this how you were thinking? |
} | ||
|
||
.button { | ||
@apply |
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.
is there a way to extend the button?
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.
What do you mean by extending the button? If we have to override a lot of styles, it's okay not to import the base button example and add everything here.
} | ||
|
||
.button svg { | ||
@apply sm:block hidden; |
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.
here i am removing the icon when the screen reaches the width of sm. This so as not to break the playground
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.
That makes sense. We can also allow horizontal scrolling on the playground preview in such cases.
Underline | ||
</ToolbarItem> | ||
<ToolbarSeparator className="toolbar-separator" /> | ||
{/* @ts-ignore */} |
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.
sorry about that, but i couldn't resolve it.
const menu = useMenuState({ orientation: "horizontal" }); | ||
return ( | ||
<> | ||
{/* @ts-ignore */} |
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.
sorry about that, but i couldn't resolve it.
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.
What's the error message? Just let the CI throw it so we can find a solution.
Let's work on a single example per PR. You can open another PR for the |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Adds a "toolbar" example #939.
Toolbar
Toolbar with menu
e-2f496bddc296.png">