fix: prevent onPress from being called when RadioGroup is read only

[yihuiliao]

Closes #9425

We do this in useToggle for Checkbox/CheckboxGroup

react-spectrum/packages/@react-aria/toggle/src/useToggle.ts

Line 96 in 5670b2a

isDisabled: isDisabled || isReadOnly

so the logic is similar

✅ Pull Request Checklist:

• Included link to corresponding React Spectrum GitHub Issue.
• Added/updated unit tests and storybook for this change (for new code or code which already has tests).
• Filled out test instructions.
• Updated documentation (if it already exists for this component).
• Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

[rspbot]

Build successful! 🎉

• View the storybook
• View the S2 storybook
• View the S2 docs
• View the React Aria docs

[snowystinger]

this is a little weird, and possibly a different issue, why are the group and the label both getting the readonly class?

[yihuiliao]

i just copied that from the previous test to confirm the ready only part so different issue?

[snowystinger]

huh... useToggle seems more like the bug, it was done to prevent us from allowing the value to be changed by our manual handling, it wasn't done to prevent onPress from being called.

I would argue that onPress should be called, interactions aren't prevented by readonly, for example, an input. You can click into those and select and copy text. You just can't change it. I'll ask a question in the Issue

[yihuiliao]

hm yeah i think that's a valid argument. fine to close this issue if we think it this is the correct behavior