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

131 select input v-2 #132

Merged
merged 9 commits into from
May 24, 2023
Merged

131 select input v-2 #132

merged 9 commits into from
May 24, 2023

Conversation

tishoyanchev
Copy link
Contributor

@tishoyanchev tishoyanchev commented May 12, 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 ---

The Select Component version 2

v-1: without divider
v-2: divider inside dropdown-menu
v-3: checkbox
v-4: when menu is at the bottom
v-5: multi-select?

📦 Published PR as canary version: 18.0.16--canary.132.f8f375feb66e3cc99fbb5916f51c732aa6f2c244.0

✨ Test out this PR locally via:

npm install @infineon/infineon-design-system-stencil@18.0.16--canary.132.f8f375feb66e3cc99fbb5916f51c732aa6f2c244.0
# or 
yarn add @infineon/infineon-design-system-stencil@18.0.16--canary.132.f8f375feb66e3cc99fbb5916f51c732aa6f2c244.0

@tishoyanchev tishoyanchev linked an issue May 12, 2023 that may be closed by this pull request
@github-actions
Copy link
Contributor

github-actions bot commented May 12, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-05-24 09:20 UTC

PR Preview URL for Storybook: https://Infineon.github.io/infineon-design-system-stencil/pr-preview/pr-132/

@tishoyanchev tishoyanchev changed the title 131 select input v-1 131 select input v-2 May 12, 2023
width: 203px;
height: 1px;

background: #EEEDED;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use color from tokens

})

export class DropdownDivider {
@Element() el;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not used?

@@ -102,6 +102,7 @@ export class Dropdown {
}

addEventListeners() {
console.log('here')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove console.log

@tishoyanchev tishoyanchev merged commit 1b0733d into master May 24, 2023
@github-actions
Copy link
Contributor

🚀 PR was released in v18.0.19 🚀

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

Successfully merging this pull request may close these issues.

select input
2 participants