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

added events on dropdown-menu #455

Merged
merged 7 commits into from
Sep 7, 2023
Merged

Conversation

tishoyanchev
Copy link
Contributor

@tishoyanchev tishoyanchev commented Sep 4, 2023

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

I have added events that emit the value of the selected option in the dropdown-menu.
The event is emitted from dropdown-item, captured in dropdown-menu, and then re-emitted again.

📦 Published PR as canary version: 20.16.3--canary.455.cf6a720b771d8f7b3b2a99ccd9941de15d3a4c6a.0

✨ Test out this PR locally via:

npm install @infineon/infineon-design-system-stencil@20.16.3--canary.455.cf6a720b771d8f7b3b2a99ccd9941de15d3a4c6a.0
# or 
yarn add @infineon/infineon-design-system-stencil@20.16.3--canary.455.cf6a720b771d8f7b3b2a99ccd9941de15d3a4c6a.0

@tishoyanchev tishoyanchev linked an issue Sep 4, 2023 that may be closed by this pull request
@tishoyanchev tishoyanchev added the patch patch version bump label Sep 4, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-09-07 10:28 UTC

@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2023

--STORYBOOK-PREVIEW--

@verena-ifx verena-ifx self-requested a review September 5, 2023 06:44
@verena-ifx
Copy link
Contributor

verena-ifx commented Sep 5, 2023

Could you change the story name, so the dropdown-menu doesnt appear anymore under "Not reviewed" on Storybook?
Or should I do it?
Another thing missing is the action on storybook, so the user can see which events get emitted and what they contain

@tishoyanchev
Copy link
Contributor Author

Could you change the story name, so the dropdown-menu doesnt appear anymore under "Not reviewed" on Storybook? Or should I do it? Another thing missing is the action on storybook, so the user can see which events get emitted and what they contain

You can fix those if you have the time so I can focus on the other high-priority issues.

@verena-ifx verena-ifx self-assigned this Sep 5, 2023
@verena-ifx
Copy link
Contributor

Should dropdown menu items generally only contain text? no value?

@verena-ifx
Copy link
Contributor

the dropdown event needs to be passed to the parent elements from the dropdown-item. Will add that logic as well

@verena-ifx verena-ifx merged commit e52a155 into master Sep 7, 2023
8 checks passed
@verena-ifx verena-ifx deleted the 454-dropdown-menu-item-emit-event branch September 7, 2023 10:26
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2023

🚀 PR was released in v20.16.3 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch patch version bump released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dropdown-menu-item emit event
2 participants