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

UI combobox #48

Merged
merged 5 commits into from
Feb 22, 2024
Merged

UI combobox #48

merged 5 commits into from
Feb 22, 2024

Conversation

florianLSP
Copy link
Contributor

fix #38

Copy link
Contributor

@rezib rezib left a comment

Choose a reason for hiding this comment

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

Please see the comments below.

racksdb/web/ui/src/components/ComboBox.vue Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
@rezib rezib force-pushed the ui-combobox branch 2 times, most recently from 9d4c0f1 to 1331201 Compare January 10, 2024 14:49
Copy link
Contributor

@rezib rezib left a comment

Choose a reason for hiding this comment

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

Currently, the combobox does not have the focus when datacenters or infrastructures views are open:

image

Users have to click on the input field to get focus and start typing or selection with arrows. It would be nice to directly have the focus on the combobox to avoid this additional click.

Copy link
Contributor

@rezib rezib left a comment

Choose a reason for hiding this comment

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

Can you please look at the following requests?

racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
Copy link
Contributor

@rezib rezib left a comment

Choose a reason for hiding this comment

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

Please have a look at the two following comments.

racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
racksdb/web/ui/src/components/ComboBox.vue Outdated Show resolved Hide resolved
Introduce custom ComboxBox component, based on Headless UI Combobox, to
select either a datacenter or an infrastructure among a predefined list.
Add headlessui/vue and tailwindcss/forms dependencies required for
comboboxes. Also enable forms plugin in tailwind plugin.
florianLSP and others added 3 commits February 22, 2024 16:39
Use ComboBox component in Datacenters and Infrastructures views.

fix #38
When Combobox is loaded in Datacenters and Infrastructures views, the
focus is automatically set on the input so user can directly type or
select an entry with keyboard without requiring an initial click on the
Combobox.
Remove button to open/close combobox as the combobox is static and
cannot actually be closed.
@rezib rezib merged commit 4fea580 into main Feb 22, 2024
@rezib rezib added this to the v0.4.0 milestone Feb 22, 2024
@florianLSP florianLSP deleted the ui-combobox branch February 28, 2024 09:24
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.

Add combobox to select datacenter and infrastructure
2 participants