[PR #178] [CLOSED] feat(ui): Finish porting radio button component #567

Closed
opened 2026-02-16 12:35:57 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/stoatchat/for-web/pull/178
Author: @ghost
Created: 4/20/2023
Status: Closed

Base: masterHead: master


📝 Commits (2)

  • 9de092d feat(ui): start working on radio inputs
  • 91e9b85 fix(ui): description using --accent-contrast when selected

📊 Changes

6 files changed (+2238 additions, -2911 deletions)

View changed files

components/ui/components/design/atoms/inputs/Radio.stories.tsx (+44 -0)
📝 components/ui/components/design/atoms/inputs/Radio.tsx (+37 -27)
packages/client/public/assets (+0 -1)
packages/client/public/assets (+1 -0)
📝 packages/showcase/componentData.ts (+1 -1)
📝 packages/showcase/src/stories.ts (+2 -1)
📝 pnpm-lock.yaml (+2153 -2881)

📄 Description

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended
  • These changes do not have any notable side effects on other Revolt projects

This PR finishes porting radio buttons from revoltchat/components to @revolt/ui

This should close #73

I haven't tested multiple radio components ye, so I don't know whether the disabled prop works properly.

Here is how it looks in the component showcase


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/stoatchat/for-web/pull/178 **Author:** [@ghost](https://github.com/ghost) **Created:** 4/20/2023 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`9de092d`](https://github.com/stoatchat/for-web/commit/9de092db6e1927829fc7baa338991e7761d722a1) feat(ui): start working on radio inputs - [`91e9b85`](https://github.com/stoatchat/for-web/commit/91e9b8512e6b849a3fbeb9bfe1956845b11d439a) fix(ui): description using --accent-contrast when selected ### 📊 Changes **6 files changed** (+2238 additions, -2911 deletions) <details> <summary>View changed files</summary> ➕ `components/ui/components/design/atoms/inputs/Radio.stories.tsx` (+44 -0) 📝 `components/ui/components/design/atoms/inputs/Radio.tsx` (+37 -27) ➖ `packages/client/public/assets` (+0 -1) ➕ `packages/client/public/assets` (+1 -0) 📝 `packages/showcase/componentData.ts` (+1 -1) 📝 `packages/showcase/src/stories.ts` (+2 -1) 📝 `pnpm-lock.yaml` (+2153 -2881) </details> ### 📄 Description ## Please make sure to check the following tasks before opening and submitting a PR * [x] I understand and have followed the [contribution guide](https://github.com/revoltchat/revolt/discussions/282) * [x] I have tested my changes locally and they are working as intended * [x] These changes do not have any notable side effects on other Revolt projects This PR finishes porting radio buttons from revoltchat/components to @revolt/ui This *should* close #73 I haven't tested multiple radio components ye, so I don't know whether the `disabled` prop works properly. [Here is how it looks in the component showcase](https://autumn.revolt.chat/attachments/U6WQjJb5Y0a-4FqO2FLjtN7H5DVTrKQ8FOQT-5ntr5/Kooha-2023-04-20-09-10-21.mp4) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 12:35:57 -05:00
yindo closed this issue 2026-02-16 12:35:57 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: stoatchat/for-web#567