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

fix: update Checkbox, Radio, and Toggle style mappings #408

Merged
merged 7 commits into from
Nov 15, 2023

Conversation

erautenberg
Copy link
Contributor

Description

  • Updates the default style property mappings of the Checkbox, Radio, and Toggle controls
  • Aliases Checkbox's "checkSrc,", "checkWidth," and "checkHeight" for "icon"
  • Deprecates Toggle's "knobX" and "knobXChecked" and aliases strokeWeight as strokeWidth
  • Updated Toggle's height and width calculation logic

References

Testing

Navigate to the Checkbox, Radio, and Toggle stories (and Small variants) and confirm all is displaying as expected.

Automation

Checklist

  • all commented code has been removed
  • any new console issues have been resolved
  • code linter and formatter has been run
  • test coverage meets repo requirements
  • PR name matches the expected semantic-commit syntax

@erautenberg erautenberg changed the title Fix/form component mappings fix: update Checkbox, Radio, and Toggle style mappings Nov 14, 2023
@svc-lightning-ui-components
Copy link
Collaborator

Test Execution Failed.

1 similar comment
@svc-lightning-ui-components
Copy link
Collaborator

Test Execution Failed.

@ImCoolNowRight ImCoolNowRight force-pushed the fix/form-component-mappings branch from e4c8ba4 to 62c12b2 Compare November 15, 2023 21:51
@svc-lightning-ui-components
Copy link
Collaborator

Test Execution Failed.

@ImCoolNowRight ImCoolNowRight merged commit 1e93576 into develop Nov 15, 2023
5 checks passed
@ImCoolNowRight ImCoolNowRight deleted the fix/form-component-mappings branch November 15, 2023 22:24
svc-lightning-ui-components pushed a commit that referenced this pull request Nov 22, 2023
# [@lightningjs/ui-components-v2.17.0](https://github.com/rdkcentral/Lightning-UI-Components/compare/@lightningjs/ui-components-v2.16.1...@lightningjs/ui-components-v2.17.0) (2023-11-22)

### Bug Fixes

* **Button:** support re patching prefix after undefined ([#407](#407)) ([e773c32](e773c32))
* **NavigationManager:** adjust isFullyOnScreen and isComponentOnScreen ([#417](#417)) ([32601bd](32601bd))
* remove applySmooth in button ([#409](#409)) ([d2c29dd](d2c29dd))
* **TabBar:** propogate key press events and emit tabChanged signal ([#406](#406)) ([9f06b81](9f06b81))
* **TitleRow:** alias rowMarginTop to titleMarginBottom ([#404](#404)) ([c2c3eb4](c2c3eb4))
* update Checkbox, Radio, and Toggle style mappings ([#408](#408)) ([1e93576](1e93576))

### Features

* **Button:** only repatch prefix/suffix row if props change ([#410](#410)) ([9297903](9297903))
@svc-lightning-ui-components
Copy link
Collaborator

🎉 This PR is included in version @lightningjs/ui-components-v2.17.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants