Skip to content

Conversation

BrunoQuaresma
Copy link
Collaborator

This component is necessary to move forward with the "Define a global button style" issue, as it requires buttons to support loading states with spinners. The Spinner component is heavily inspired by the Radix UI Spinner component and was developed with the help of v0.

Preview
Screen Recording 2025-01-02 at 14 37 18

@BrunoQuaresma BrunoQuaresma self-assigned this Jan 2, 2025
@jaaydenh
Copy link
Contributor

jaaydenh commented Jan 2, 2025

@BrunoQuaresma Im curious why not just use the Radix spinner component directly? For example, the Radix spinner has a bit of a better style because of the thicker lines and matches the Figma design a bit better.

@BrunoQuaresma
Copy link
Collaborator Author

@BrunoQuaresma Im curious why not just use the Radix spinner component directly? For example, the Radix spinner has a bit of a better style because of the thicker lines and matches the Figma design a bit better.

Because it’s not a primitive—we’d have to install the entire Radix UI package along with their setup.

@BrunoQuaresma BrunoQuaresma requested a review from jaaydenh January 2, 2025 18:25
Copy link
Contributor

@jaaydenh jaaydenh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

BrunoQuaresma and others added 2 commits January 2, 2025 15:52
Co-authored-by: Jaayden Halko <jaayden.halko@gmail.com>
@BrunoQuaresma BrunoQuaresma merged commit cee3656 into main Jan 2, 2025
29 checks passed
@BrunoQuaresma BrunoQuaresma deleted the bq/add-spinner-component branch January 2, 2025 19:10
@github-actions github-actions bot locked and limited conversation to collaborators Jan 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy