Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: custom loading icon not showing #468

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

eliezerbasubi
Copy link

@eliezerbasubi eliezerbasubi commented Jul 13, 2024

Description:

This PR addresses the issue of custom loading icons not displaying correctly. By introducing a LoadingWrapper component, we ensure that custom loading icons are properly wrapped with the necessary sonner-loading-wrapper class, allowing them to be displayed as intended.

What has been done:

  • Added LoadingWrapper component to handle loading icon wrapping
  • Updated existing loading icon references to use LoadingWrapper.

Summary

These changes improve the flexibility and usability of the loading icon feature, allowing for custom icons to be seamlessly integrated.

Screenshots/Videos:

Video of error

loading_icon_not_showing_before_implementation.mov

Video of fix

loading_icon_showing_after_resolving_error.mov

Copy link

vercel bot commented Jul 13, 2024

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

Name Status Preview Comments Updated (UTC)
sonner ✅ Ready (Inspect) Visit Preview 💬 1 unresolved Oct 28, 2024 1:18pm

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