-
Notifications
You must be signed in to change notification settings - Fork 47
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
SuperSelect
- search position (HDS-4254)
#2612
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
packages/components/src/components/hds/form/super-select/single/base.hbs
Show resolved
Hide resolved
packages/components/src/components/hds/form/super-select/multiple/base.hbs
Outdated
Show resolved
Hide resolved
…add preliminary Showcase examples
cd56538
to
50080f9
Compare
…Multiple SuperSelect, clean up Showcase
SuperSelect
- search position (HDS-4254)SuperSelect
- search position (HDS-4254)
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.
🚢
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.
This seems to require a rebase to resolve some conflicts in yarn.lock
, otherwise it's good to be merged.
…add preliminary Showcase examples
…Multiple SuperSelect, clean up Showcase
…b.com/hashicorp/design-system into hds-4254-super-select-search-position # Conflicts: # yarn.lock
Resolved yarn.lock conflicts. |
The merge-base changed after approval.
The merge-base changed after approval.
@@ -52,7 +52,7 @@ | |||
"ember-focus-trap": "^1.1.0", | |||
"ember-get-config": "^2.1.1", | |||
"ember-modifier": "^4.1.0", | |||
"ember-power-select": "^8.2.0", | |||
"ember-power-select": "^8.6.0", |
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 had two patch releases since we started the work on this one, so we should bump this to latest 8.6.2
"ember-power-select": "^8.6.0", | |
"ember-power-select": "^8.6.2", |
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.
of course, this would require a small adjustment to the changelog entry and a yarn.lock
update
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.
on a second thought, we have a version bump in #2625 (review) as well, so you can get this in as is.
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.
So is this one ok to just merge now as is?
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.
Yes.
However, we're having issues with getting it approved. They automatically get dismissed after the main
was merged into it. Would you be able to force-push from local to see if we can overcome this?
The merge-base changed after approval.
# Conflicts: # showcase/package.json # yarn.lock
📌 Summary
If merged, this PR adds the
searchFieldPosition
toSuperSelect
enablingSuperSelect::Multiple
to display the search field within the before options area instead of in the trigger area.Related:
SuperSelect
- docs for search position (HDS-4254) #2620🛠️ Detailed description
searchFieldPosition
argument to SuperSelect components📸 Screenshots
🔗 External links
👀 Component checklist
💬 Please consider using conventional comments when reviewing this PR.