-
Notifications
You must be signed in to change notification settings - Fork 205
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
docs: document menu-bar reverse-collapse #3188
Conversation
Items that don't fit into the current width of the menu bar collapse into an overflow menu at the end: | ||
Items that don't fit into the current width of the menu bar collapse into an overflow menu at the end. | ||
|
||
By default, collapsed items are removed from the end of the menu bar, but the component can be configured to remove items from the start instead. |
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.
Should it mention the API name?
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.
We don't usually mention API specifics in the documentation because the same documentation is shared by Flow, Lit, and React users while the API is different for all these component types. The component type-specific API can usually be discovered in the example source. In this case, there is no live example, so it's only available in the API documentation.
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.
Right. Do we also need to add the since...
tag here?
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.
* Add Flow React adapter integration guide * docs: document menu-bar reverse-collapse (#3188) * chore: replace `@vaadin/hilla-core` with `@vaadin/hilla-frontend` (#3178) * chore: replace @vaadin/hilla-frontend with @vaadin/hilla-frontend Follow-up for vaadin/hilla#2052 Fixes #3177 * upgrade to alpha4 --------- Co-authored-by: Anton Platonov <[email protected]> * chore: update vaadin.version and springboot (#3202) * split Java example into more steps * chore: update to 24.4 alpha4 * First full pass at editing. * Second full pass at editing. --------- Co-authored-by: Tomi Virkki <[email protected]> Co-authored-by: Soroosh Taefi <[email protected]> Co-authored-by: Zhe Sun <[email protected]> Co-authored-by: Mikhail Shabarov <[email protected]> Co-authored-by: Russell J.T. Dyer <[email protected]>
Part of vaadin/platform#5018