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 Button / ButtonGroup #30

Open
emilyjablonski opened this issue Sep 7, 2023 · 3 comments
Open

Align Button / ButtonGroup #30

emilyjablonski opened this issue Sep 7, 2023 · 3 comments

Comments

@emilyjablonski
Copy link

emilyjablonski commented Sep 7, 2023

DUIC has a local version of Button and ButtonGroup.

There are quite a few changes here especially to ButtonGroup in order for a ButtonGroup to be able to support the new behavior we see on the Doorway home page in the unit type selection where there is one button at a time active with the button group that can be selected. There is a lot of added business logic to the components that shouldn't be in the component.

We either need (1) a custom component that only exists in Doorway that is a new form component that handles a selectable button group, or if we think it's generalizable we could add it to UIC - since we are working on overhauling forms, I'd recommend just a local Doorway component, but still getting them aligned on our Button elsewhere.

Their changes in the SCSS file for Button can for the most part be handled with existing overrides. We would need a small PR to UIC to add a few new overrides that aren’t possible now.

@slowbot
Copy link

slowbot commented Sep 12, 2023

@emilyjablonski Do you think it makes sense to migrate the "selectable button group" to be a style of checkbox or radio depending on the selection of one or more options?

@emilyjablonski
Copy link
Author

From an accessibility perspective, a stylized radio would be perfect here

@emilyjablonski
Copy link
Author

Note: We're working on Seeds button uptake in core so we should put this on hold until that has been released in Doorway in terms of adding overriddes to support Doorway styles. We'll still to figure out a path forward with the ButtonGroup. Given the timeline on forms I'd recommend a local component in Doorway public.

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

No branches or pull requests

2 participants