Skip to content

Commit 00c87e8

Browse files
authored
fix(blade): radio helptext being negative (#2175)
* fix(blade): radio helptext being negative * chore: update snaos * Create poor-days-dress.md * chore: update checkbox logic * Create loud-shrimps-smoke.md * chore: update
1 parent 72a68ba commit 00c87e8

File tree

4 files changed

+10
-24
lines changed

4 files changed

+10
-24
lines changed

.changeset/loud-shrimps-smoke.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@razorpay/blade": patch
3+
---
4+
5+
fix(blade): radio helptext being negative

packages/blade/src/components/Checkbox/Checkbox.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -193,9 +193,7 @@ const _Checkbox: React.ForwardRefRenderFunction<BladeElementRef, CheckboxProps>
193193
large: 'large',
194194
} as const;
195195

196-
// only show error when the self validation is set to error
197-
// Since we don't want to show errorText inside the group
198-
const showSupportingText = validationState !== 'error' && helpText;
196+
const showSupportingText = helpText;
199197

200198
const handleChange: OnChange = ({ isChecked, event, value }) => {
201199
if (isChecked) {

packages/blade/src/components/Radio/Radio.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ const _Radio: React.ForwardRefRenderFunction<BladeElementRef, RadioProps> = (
7474
const _isDisabled = isDisabled ?? groupProps?.isDisabled;
7575
const _isRequired = groupProps?.isRequired || groupProps?.necessityIndicator === 'required';
7676
const name = groupProps?.name;
77-
const showHelpText = !hasError && helpText;
77+
const showHelpText = helpText;
7878
const isReactNative = getPlatformType() === 'react-native';
7979
const _size = groupProps.size ?? size;
8080

@@ -131,7 +131,7 @@ const _Radio: React.ForwardRefRenderFunction<BladeElementRef, RadioProps> = (
131131
</BaseBox>
132132
{showHelpText && (
133133
<BaseBox marginLeft={helpTextLeftSpacing}>
134-
<SelectorSupportText size={_size} isNegative={true} id={ids?.helpTextId}>
134+
<SelectorSupportText size={_size} id={ids?.helpTextId}>
135135
{helpText}
136136
</SelectorSupportText>
137137
</BaseBox>

packages/blade/src/components/Radio/__tests__/__snapshots__/Radio.ssr.test.tsx.snap

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

3-
exports[`<Radio /> should render with help text 1`] = `"<div id="root"><div data-blade-component="base-box" class="BaseBox-bmPWx"><div role="radiogroup" aria-labelledby="radio-group-undefined-label" data-blade-component="radio-group" class="BaseBox-bmPWx fnwmJW"><span style="width:auto;flex-shrink:0;margin-right:0px" id="radio-group-undefined-label" data-blade-component="form-label"><div data-blade-component="base-box" class="BaseBox-bmPWx fZlcZT"><div data-blade-component="base-box" class="BaseBox-bmPWx ikPXAQ"><p class="StyledBaseText-dVBfTO kCEKIT" data-blade-component="text">Select fruit</p><div data-blade-component="visually-hidden" class="VisuallyHiddenweb__StyledVisuallyHidden-g3hls3-0 Fuolv"><p class="StyledBaseText-dVBfTO fAQkza" data-blade-component="text">,Select One</p></div></div></div></span><div data-blade-component="base-box" class="BaseBox-bmPWx"><div data-blade-component="base-box" class="BaseBox-bmPWx fnwmJW"><div data-blade-component="base-box" class="BaseBox-bmPWx fdoqPr"><div data-blade-component="base-box" class="BaseBox-bmPWx"><label data-blade-component="radio-label" class="SelectorLabelweb__StyledSelectorLabel-sc-1yoi09a-0 ehaOvX"><div data-blade-component="base-box" class="BaseBox-bmPWx fnwmJW"><div data-blade-component="base-box" class="BaseBox-bmPWx fAEwZa"><input type="radio" name="radio-group-undefined" role="radio" aria-required="false" aria-invalid="false" aria-disabled="false" aria-checked="false" aria-describedby="radio-undefined-helptext-undefined" class="SelectorInputweb__StyledInput-sc-1bnkrae-0 jObyqS" value="apple"/><div data-blade-component="radio-icon-wrapper" class="RadioIconWrapperweb__RadioIconWrapper-sc-1memw5n-0 kbZLYq"></div><div data-blade-component="base-box" class="BaseBox-bmPWx CzMgf"></div><p class="StyledBaseText-dVBfTO cYIHIH" data-blade-component="text">Apple</p></div><div data-blade-component="base-box" class="BaseBox-bmPWx ewfYUx"><div id="radio-undefined-helptext-undefined" data-blade-component="box" class="BaseBox-bmPWx kTMxBD"><span class="StyledBaseText-dVBfTO gvbiie" data-blade-component="text">Apple Help</span></div></div></div></label></div></div><div data-blade-component="base-box" class="BaseBox-bmPWx hlbQIL"><div data-blade-component="base-box" class="BaseBox-bmPWx"><label data-blade-component="radio-label" class="SelectorLabelweb__StyledSelectorLabel-sc-1yoi09a-0 ehaOvX"><div data-blade-component="base-box" class="BaseBox-bmPWx fnwmJW"><div data-blade-component="base-box" class="BaseBox-bmPWx fAEwZa"><input type="radio" name="radio-group-undefined" role="radio" aria-required="false" aria-invalid="false" aria-disabled="false" aria-checked="false" aria-describedby="radio-undefined-helptext-undefined" class="SelectorInputweb__StyledInput-sc-1bnkrae-0 jObyqS" value="mango"/><div data-blade-component="radio-icon-wrapper" class="RadioIconWrapperweb__RadioIconWrapper-sc-1memw5n-0 kbZLYq"></div><div data-blade-component="base-box" class="BaseBox-bmPWx CzMgf"></div><p class="StyledBaseText-dVBfTO cYIHIH" data-blade-component="text">Mango</p></div></div></label></div></div></div><div data-blade-component="base-box" class="BaseBox-bmPWx kzOBQv"><span style="word-break:break-all" data-blade-component="base-box" class="BaseBox-bmPWx fAEwZa"><span class="StyledBaseText-dVBfTO csXxAx" data-blade-component="text">Select One</span></span></div></div></div></div></div>"`;
3+
exports[`<Radio /> should render with help text 1`] = `"<div id="root"><div data-blade-component="base-box" class="BaseBox-bmPWx"><div role="radiogroup" aria-labelledby="radio-group-undefined-label" data-blade-component="radio-group" class="BaseBox-bmPWx fnwmJW"><span style="width:auto;flex-shrink:0;margin-right:0px" id="radio-group-undefined-label" data-blade-component="form-label"><div data-blade-component="base-box" class="BaseBox-bmPWx fZlcZT"><div data-blade-component="base-box" class="BaseBox-bmPWx ikPXAQ"><p class="StyledBaseText-dVBfTO kCEKIT" data-blade-component="text">Select fruit</p><div data-blade-component="visually-hidden" class="VisuallyHiddenweb__StyledVisuallyHidden-g3hls3-0 Fuolv"><p class="StyledBaseText-dVBfTO fAQkza" data-blade-component="text">,Select One</p></div></div></div></span><div data-blade-component="base-box" class="BaseBox-bmPWx"><div data-blade-component="base-box" class="BaseBox-bmPWx fnwmJW"><div data-blade-component="base-box" class="BaseBox-bmPWx fdoqPr"><div data-blade-component="base-box" class="BaseBox-bmPWx"><label data-blade-component="radio-label" class="SelectorLabelweb__StyledSelectorLabel-sc-1yoi09a-0 ehaOvX"><div data-blade-component="base-box" class="BaseBox-bmPWx fnwmJW"><div data-blade-component="base-box" class="BaseBox-bmPWx fAEwZa"><input type="radio" name="radio-group-undefined" role="radio" aria-required="false" aria-invalid="false" aria-disabled="false" aria-checked="false" aria-describedby="radio-undefined-helptext-undefined" class="SelectorInputweb__StyledInput-sc-1bnkrae-0 jObyqS" value="apple"/><div data-blade-component="radio-icon-wrapper" class="RadioIconWrapperweb__RadioIconWrapper-sc-1memw5n-0 kbZLYq"></div><div data-blade-component="base-box" class="BaseBox-bmPWx CzMgf"></div><p class="StyledBaseText-dVBfTO cYIHIH" data-blade-component="text">Apple</p></div><div data-blade-component="base-box" class="BaseBox-bmPWx ewfYUx"><div id="radio-undefined-helptext-undefined" data-blade-component="box" class="BaseBox-bmPWx kTMxBD"><span class="StyledBaseText-dVBfTO csXxAx" data-blade-component="text">Apple Help</span></div></div></div></label></div></div><div data-blade-component="base-box" class="BaseBox-bmPWx hlbQIL"><div data-blade-component="base-box" class="BaseBox-bmPWx"><label data-blade-component="radio-label" class="SelectorLabelweb__StyledSelectorLabel-sc-1yoi09a-0 ehaOvX"><div data-blade-component="base-box" class="BaseBox-bmPWx fnwmJW"><div data-blade-component="base-box" class="BaseBox-bmPWx fAEwZa"><input type="radio" name="radio-group-undefined" role="radio" aria-required="false" aria-invalid="false" aria-disabled="false" aria-checked="false" aria-describedby="radio-undefined-helptext-undefined" class="SelectorInputweb__StyledInput-sc-1bnkrae-0 jObyqS" value="mango"/><div data-blade-component="radio-icon-wrapper" class="RadioIconWrapperweb__RadioIconWrapper-sc-1memw5n-0 kbZLYq"></div><div data-blade-component="base-box" class="BaseBox-bmPWx CzMgf"></div><p class="StyledBaseText-dVBfTO cYIHIH" data-blade-component="text">Mango</p></div></div></label></div></div></div><div data-blade-component="base-box" class="BaseBox-bmPWx kzOBQv"><span style="word-break:break-all" data-blade-component="base-box" class="BaseBox-bmPWx fAEwZa"><span class="StyledBaseText-dVBfTO csXxAx" data-blade-component="text">Select One</span></span></div></div></div></div></div>"`;
44

55
exports[`<Radio /> should render with help text 2`] = `
66
.c10.c10.c10.c10.c10 {
@@ -169,23 +169,6 @@ exports[`<Radio /> should render with help text 2`] = `
169169
}
170170
171171
.c15.c15.c15.c15.c15 {
172-
color: hsla(4,74%,49%,1);
173-
font-family: "Inter","Inter Fallback Arial",Arial;
174-
font-size: 0.6875rem;
175-
font-weight: 400;
176-
font-style: italic;
177-
-webkit-text-decoration-line: none;
178-
text-decoration-line: none;
179-
line-height: 1rem;
180-
-webkit-letter-spacing: 0px;
181-
-moz-letter-spacing: 0px;
182-
-ms-letter-spacing: 0px;
183-
letter-spacing: 0px;
184-
margin: 0;
185-
padding: 0;
186-
}
187-
188-
.c18.c18.c18.c18.c18 {
189172
color: hsla(211,22%,56%,1);
190173
font-family: "Inter","Inter Fallback Arial",Arial;
191174
font-size: 0.6875rem;
@@ -440,7 +423,7 @@ exports[`<Radio /> should render with help text 2`] = `
440423
style="word-break:break-all"
441424
>
442425
<span
443-
class="c18"
426+
class="c15"
444427
data-blade-component="text"
445428
>
446429
Select One

0 commit comments

Comments
 (0)