Skip to content
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(OnyxFlyoutMenu)!: Replace VNode dependency with slot parameters #2123

Merged
merged 5 commits into from
Nov 20, 2024

Conversation

JoCa96
Copy link
Collaborator

@JoCa96 JoCa96 commented Nov 20, 2024

Relates to #1595

In preparation to support nesting.

  • Added component tests for the flyout menu
  • Replaced the VNodes unwrapping with slot parameters to avoid reliance on Vue internals
    • feat(OnyxNavButton, OnyxUserMenu, OnyxFlyoutMenu): button slot now provides a trigger parameter which must be bound to the interactive element
    • feat(OnyxFlyoutMenu): Renamed "default" slot to "button"

Copy link

changeset-bot bot commented Nov 20, 2024

🦋 Changeset detected

Latest commit: 656a4ae

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
sit-onyx Major
demo-app Patch
playground Patch
@sit-onyx/chartjs-plugin Major
@sit-onyx/nuxt Major
@sit-onyx/vitepress-theme Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@JoCa96 JoCa96 marked this pull request as ready for review November 20, 2024 09:46
@JoCa96 JoCa96 requested a review from a team as a code owner November 20, 2024 09:46
… BREAKING CHANGE: button slot now provides a trigger parameter which must be bound to the interactive element

- feat(OnyxFlyoutMenu): BREAKING CHANGE: Renamed "default" slot to "button"
@JoCa96 JoCa96 force-pushed the joca96/1595-prep-chore-use-slot-props-user-menu-flyout branch from 31c5f43 to 656a4ae Compare November 20, 2024 11:44
@JoCa96 JoCa96 changed the title chore(OnyxFlyoutMenu): Replace VNode dependency with slot parameters chore(OnyxFlyoutMenu)!: Replace VNode dependency with slot parameters Nov 20, 2024
@larsrickert larsrickert self-assigned this Nov 20, 2024
@JoCa96 JoCa96 merged commit 4a4c414 into main Nov 20, 2024
19 checks passed
@JoCa96 JoCa96 deleted the joca96/1595-prep-chore-use-slot-props-user-menu-flyout branch November 20, 2024 16:59
larsrickert added a commit that referenced this pull request Nov 21, 2024
Relates to #2123

Currently, the Storybook example for the OnyxFlyoutMenu is broken due to
#2123
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants