-
Notifications
You must be signed in to change notification settings - Fork 45
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
FORMS-12667 Update type property on replacing checkboxgroup to radio group #1176
Conversation
Lighthouse scores (desktop)
|
Lighthouse scores (mobile)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
Lighthouse scores (desktop)
|
Lighthouse scores (mobile)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1176 +/- ##
=========================================
Coverage 80.75% 80.75%
Complexity 773 773
=========================================
Files 91 91
Lines 2110 2110
Branches 285 285
=========================================
Hits 1704 1704
Misses 252 252
Partials 154 154 ☔ View full report in Codecov by Sentry. |
@@ -3,6 +3,7 @@ | |||
jcr:primaryType="nt:unstructured" | |||
jcr:title="Radio Button" | |||
fieldType="radio-group" | |||
type="number" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reasons for adding type to OOTB component ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In enum we are providing enum="[0,1]
, this is similar to checkbox group but there we have mentioned number
type [1]
[1]:
Please add JIRA id to the commit message |
ui.tests/test-module/specs/replaceaction/replacelayout-container.authoring.spec.js
Outdated
Show resolved
Hide resolved
25efeba
to
4a0d5c8
Compare
Lighthouse scores (mobile)
|
Lighthouse scores (desktop)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
…group (#1176) * Update type property on replacing checkboxgroup to radio group * FORMS-12667 review comment --------- Co-authored-by: demaurya <demaurya@adobe.com>
…group (#1176) * Update type property on replacing checkboxgroup to radio group * FORMS-12667 review comment --------- Co-authored-by: demaurya <demaurya@adobe.com>
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: