[PR #3632] dbeaver/pro#6444 refactor: fix Radio component #4288

Closed
opened 2026-03-07 21:17:35 +03:00 by kerem · 0 comments
Owner

Original Pull Request: https://github.com/dbeaver/cloudbeaver/pull/3632

State: closed
Merged: Yes


removes RadioGroupContext(radio group/radio use their own provider) removes all the logic of state changing from Radio component since it's creating a wrong understanding of how these components should be used together with RadioGroup. Radio still can be controllable, but it should be used in most scenarios only with RadioGroup which is controlling the state. simplify styles by removing extra class

https://github.com/user-attachments/assets/702e2c25-a8a2-44fc-84d7-4fe48e6391dc

**Original Pull Request:** https://github.com/dbeaver/cloudbeaver/pull/3632 **State:** closed **Merged:** Yes --- removes RadioGroupContext(radio group/radio use their own provider) removes all the logic of state changing from Radio component since it's creating a wrong understanding of how these components should be used together with RadioGroup. Radio still can be controllable, but it should be used in most scenarios only with RadioGroup which is controlling the state. simplify styles by removing extra class https://github.com/user-attachments/assets/702e2c25-a8a2-44fc-84d7-4fe48e6391dc
kerem 2026-03-07 21:17:35 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cloudbeaver#4288
No description provided.