Skip to content

Conversation

SriHV
Copy link
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
Copy link

netlify bot commented Sep 29, 2025

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

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

2 participants