Skip to content

Enhance Spinner Component with Loading Text and Customization Options#2

Open
ruimarcosjoao wants to merge 7 commits into
allipiopereira:mainfrom
ruimarcosjoao:new-features
Open

Enhance Spinner Component with Loading Text and Customization Options#2
ruimarcosjoao wants to merge 7 commits into
allipiopereira:mainfrom
ruimarcosjoao:new-features

Conversation

@ruimarcosjoao
Copy link
Copy Markdown

In this pull request, I have made significant enhancements to the Spinner component to improve its functionality and flexibility.

The key changes include:

Addition of defaultText: I introduced a new prop, defaultText, which allows the spinner to display a message while loading. This provides better feedback to users about ongoing processes.

Support for Default Text and Children: The component now supports displaying default text and custom children when not in a loading state. This allows for more personalized usage of the spinner in various contexts.

Improved Logic for Loading States: I refactored the component's logic to handle different loading states more clearly, ensuring a consistent and predictable behavior.

These improvements aim to enhance the user experience by providing clear visual feedback during loading operations and allowing for greater customization of the spinner's appearance and behavior.

I believe these updates will make the Spinner component more versatile and user-friendly. Thank you for considering this pull request!

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
shadcn-spinner ❌ Failed (Inspect) Nov 13, 2024 4:33pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant