Skip to content

fix(button-group): remove element selector restrictions - #12291

Merged
kodiakhq[bot] merged 2 commits into
carbon-design-system:mainfrom
marcelojcs:fix/button-group/adjust-element-selector
Jul 24, 2025
Merged

fix(button-group): remove element selector restrictions#12291
kodiakhq[bot] merged 2 commits into
carbon-design-system:mainfrom
marcelojcs:fix/button-group/adjust-element-selector

Conversation

@marcelojcs

Copy link
Copy Markdown
Contributor

Related Ticket(s)

jira

Description

{{Add a human-readable description / detail summary of what the PR is changing and any details around how and why}}

{{If applicable, include a screenshot indicating an example or examples of what the PR is changing in the application}}

Changelog

PR to remove the restrictions on the method that defines the child count on button-group
The restrictions were filtering out valid components and causing the button group to misbehave.

Before:
image
(buttons on top of each other)

After:
image
(Buttons side by side)

@marcelojcs
marcelojcs requested a review from a team as a code owner July 17, 2025 20:20
@marcelojcs
marcelojcs requested review from Valentin-Sorin-Nicolae and bruno-amorim and removed request for a team July 17, 2025 20:20
@netlify

netlify Bot commented Jul 17, 2025

Copy link
Copy Markdown

Deploy Preview for incomparable-wisp-8d6771 ready!

Name Link
🔨 Latest commit 71d9bd5
🔍 Latest deploy log https://app.netlify.com/projects/incomparable-wisp-8d6771/deploys/6882915eb7b87e0008cbcce4
😎 Deploy Preview https://deploy-preview-12291--incomparable-wisp-8d6771.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jul 17, 2025

Copy link
Copy Markdown

Deploy Preview for ibm-dotcom-web-components-react-wrap ready!

Name Link
🔨 Latest commit 71d9bd5
🔍 Latest deploy log https://app.netlify.com/projects/ibm-dotcom-web-components-react-wrap/deploys/6882915eb7b87e0008cbcce2
😎 Deploy Preview https://deploy-preview-12291--ibm-dotcom-web-components-react-wrap.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jul 17, 2025

Copy link
Copy Markdown

Deploy Preview for ibm-dotcom-web-components ready!

Name Link
🔨 Latest commit 71d9bd5
🔍 Latest deploy log https://app.netlify.com/projects/ibm-dotcom-web-components/deploys/6882915ebb67360008d82d6a
😎 Deploy Preview https://deploy-preview-12291--ibm-dotcom-web-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bruno-amorim bruno-amorim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Working fine to me too! Nice job!

@marcelojcs marcelojcs added the Ready to merge Label for the pull requests that are ready to merge label Jul 24, 2025
@kodiakhq
kodiakhq Bot merged commit 7206b39 into carbon-design-system:main Jul 24, 2025
20 of 22 checks passed
@marcelojcs
marcelojcs deleted the fix/button-group/adjust-element-selector branch August 13, 2025 11:37
@github-actions

Copy link
Copy Markdown
Contributor

Hey there! This issue/pull request was referenced in recently released v2.29.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready to merge Label for the pull requests that are ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants