Skip to content

IBX-11506: Use design-system checkbox/radio/toggle components#74

Open
tischsoic wants to merge 15 commits intomainfrom
IBX-11506-use-ds-checkbox-component
Open

IBX-11506: Use design-system checkbox/radio/toggle components#74
tischsoic wants to merge 15 commits intomainfrom
IBX-11506-use-ds-checkbox-component

Conversation

@tischsoic
Copy link
Copy Markdown

🎫 Issue IBX-11506

Description:

For QA:

Documentation:

BREAKING CHANGE: Removed reliance on ids-btn--disabled CSS class

Button component:
- Removed ids-btn--disabled class from button template
- Now relies solely on native disabled attribute for styling
- Updated tests to verify disabled attribute instead of class

Link component:
- Added disabled property to Link component
- Uses aria-disabled=true attribute for disabled link-buttons
- Added test coverage for disabled state
- Links styled as buttons now properly support disabled state

Why this change:
- More semantic HTML - uses native browser attributes
- Better accessibility with native disabled/aria-disabled support
- Simpler component logic - no class state management needed
- CSS already supported attribute selectors

Migration guide:
- Remove any JavaScript selecting .ids-btn--disabled
- Use [disabled] or [aria-disabled] selectors instead
- Components automatically handle disabled state via props
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.

1 participant