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

Align style of Select components #7277

Open
ferhoyos opened this issue Apr 15, 2024 · 5 comments
Open

Align style of Select components #7277

ferhoyos opened this issue Apr 15, 2024 · 5 comments
Assignees
Labels
enhancement This is the preferred way to describe new end-to-end features. good first issue Needs a contributor. Good starter issue

Comments

@ferhoyos
Copy link
Contributor

What do you want to improve?

Currently there are two types of Select components within Kiali:

  • Standard Selects used mainly in Wizards

image

  • Patternfly Selects used in the rest of the Kiali application:

image

It is a UX good practice to apply the same style for the same type of component, Select in this case.

What is the current behavior?

Two different Select component styles used in Kiali.

What is the new behavior?

Use the same Select component in Kiali. IMO the best choice is to follow Patternfly's style like the most of the UI elements in Kiali.

@ferhoyos ferhoyos added the enhancement This is the preferred way to describe new end-to-end features. label Apr 15, 2024
@jshaughn jshaughn added backlog Triaged Issue added to backlog good first issue Needs a contributor. Good starter issue labels Apr 15, 2024
@TomasMatarazzo
Copy link

can i work on this? thx

@aljesusg
Copy link
Collaborator

Of course. I assigned this to you !

Welcome to Kiali community
Thanks @TomasMatarazzo

@ferhoyos
Copy link
Contributor Author

ferhoyos commented Apr 15, 2024

Thanks @TomasMatarazzo! The idea is to replace all <FormSelect> (https://www.patternfly.org/components/forms/form-select, which embeds browser native select) to <Select> components (https://www.patternfly.org/components/menus/select). There are some examples of <Select> within the Kiali application you can take as reference.

Previously I would like to confirm with @fkargbo that this is the correct path to follow. Maybe for forms it is better to use native selects for some reason I ignore.

@jshaughn
Copy link
Collaborator

@TomasMatarazzo , Thanks! If you prefer, you can do this incrementally with multiple PRs. Please feel free to ask for guidance/review.

@jshaughn jshaughn removed the backlog Triaged Issue added to backlog label Apr 15, 2024
@ferhoyos
Copy link
Contributor Author

ferhoyos commented Apr 19, 2024

Confirmed with @fkargbo that it is recommended to replace FormSelect components to unify UX style. @TomasMatarazzo, I suggest to create a first PR replacing one of the FormSelect for guidance/review. Then, you can continue with the rest of them. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is the preferred way to describe new end-to-end features. good first issue Needs a contributor. Good starter issue
Projects
Status: 🏗 In progress
Development

No branches or pull requests

4 participants