Fix `ButtonGroup` styles for disabled buttons w/ `Tooltip`

Review changes
Download
Patches
Plain diff

Open Fix `ButtonGroup` styles for disabled buttons w/ `Tooltip`

Overview 0
Commits 1
Pipelines 0
Changes 2

Open Warren Gifford requested to merge lrh/fix-button-group-tooltips into main 2 years ago

Overview 0
Commits 1
Pipelines 0
Changes 2

Created by: lrhacker

In order for tooltips to work properly on disabled buttons, we need to wrap those buttons with additional elements. This wound up breaking styles when trying to use disabled buttons with tooltips inside of a button group.

This change adjusts the styles for button groups so that they no longer assume the element with the .btn class name will be a direct child. I also moved the ButtonGroup into its own dedicated story and included examples with tooltips to help with testing.

Test plan

Validate visual styles in Storybook. Confirm no visual regressions with ButtonGroups used in the application.

App preview:

Web
Storybook

Check out the client app preview documentation to learn more.

Merge request reports

Activity

Filter activity

Approvals
Assignees & reviewers
Comments (from bots)
Comments (from users)
Commits & branches
Edits
Labels
Lock status
Mentions
Merge request status
Tracking

Please register or sign in to reply

0 Assignees

0 Reviewers

None

Request review from

Labels

cla-signed team/frontend-platform

Select labels

Manage project labels

Milestone

None

Time tracking

No estimate or time spent

Participants

There are currently no pipelines.

To run a merge request pipeline, the jobs in the CI/CD configuration file must be configured to run in merge request pipelines and you must have sufficient permissions in the source project.