Skip to content

Add attribute param to textarea in radios #3740

Merged
SriHV merged 5 commits intomainfrom
fix/set-textarea-within-radio-to-disabled
Oct 3, 2025
Merged

Add attribute param to textarea in radios #3740
SriHV merged 5 commits intomainfrom
fix/set-textarea-within-radio-to-disabled

Conversation

@SriHV
Copy link
Copy Markdown
Contributor

@SriHV SriHV commented Sep 29, 2025

What is the context of this PR?

  • As per the ONSDESYS-365, the disabled: true prop couldn't be applied to nested textarea within radio component. To fix this , I've added the attributes param to the textaarea in radios component

  • Also added a test to check this

How to review this PR

  • Go to any radios example where a textarea is used within others param and add "attributes": {"disabled": true} to check that the textarea is disabled

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@SriHV SriHV requested a review from a team as a code owner September 29, 2025 12:24
@netlify
Copy link
Copy Markdown

netlify bot commented Sep 29, 2025

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit 7d2f99a
🔍 Latest deploy log https://app.netlify.com/projects/ons-design-system-preview/deploys/68de7746b01b2f0008ed2497
😎 Deploy Preview https://deploy-preview-3740--ons-design-system-preview.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.

@SriHV SriHV self-assigned this Sep 29, 2025
@SriHV SriHV added the Bug Something isn't working label Sep 29, 2025
Co-authored-by: rmccar <42928680+rmccar@users.noreply.github.com>
@SriHV SriHV merged commit ab68ad0 into main Oct 3, 2025
14 checks passed
@SriHV SriHV deleted the fix/set-textarea-within-radio-to-disabled branch October 3, 2025 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants