refactor(icons): replace LoadingOutlined with SigNoz Loader#10839
refactor(icons): replace LoadingOutlined with SigNoz Loader#10839yadavshubham01 wants to merge 8 commits intoSigNoz:mainfrom
Conversation
|
Welcome to the SigNoz community! Thank you for your first pull request and making this project better. 🤗 |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 444f01f. Configure here.
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
@YounixM need review |
H4ad
changed the title
H4ad
added
frontend
refactored
safe-to-test
H4ad
left a comment
H4ad
left a comment
There was a problem hiding this comment.
Thanks for the PR, would you mind taking a look at the failling tests and format/lint issues?
Sure |
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
primus-bot
Bot
removed
the
safe-to-test
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
@yadavshubham01 Only one issue in the lint, and then we will be good to merge |
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
primus-bot
Bot
removed
the
safe-to-test
|
@H4ad done |
H4ad
added
the
safe-to-test
2 participants
Pull Request
📄 Summary
This PR addresses issue #10346.
Replaced the LoadingOutlined ant-icon with SigNoz Loader / LoaderCircle icons
Updated high-frequency usage areas incrementally (starting with this PR)
Ensured consistency with the current design system and icon standards
This is part of a series of PRs where high-frequency icons are being updated step-by-step to avoid large changes in a single PR.
Changes Made
Replaced LoadingOutlined → Loader / LoaderCircle
Updated relevant components without altering functionality
Screenshots / Screen Recordings (if applicable)
Screen.Recording.2026-04-06.110901.mp4
Issues closed by this PR
Related #10346
✅ Change Type
Select all that apply
🧪 Testing Strategy
📝 Changelog
📋 Checklist
Note
Low Risk
Low risk visual-only change: replaces loading indicators across many UI surfaces without altering data flow or business logic. Main regression risk is minor styling/alignment differences or incorrect icon import in a few components.
Overview
Replaces Ant Design’s
LoadingOutlinedspinner icon with SigNoz loader icons (@signozhq/iconsLoader/LoaderCircle) across multiple high-traffic UI loading states (tables, modals, selects, dashboards, onboarding, and misc. status indicators).This updates the shared
Spinnercomponent and numerousSpin/button icon usages to render the new loader withanimate-spin, aiming for consistent iconography while keeping behavior unchanged.Reviewed by Cursor Bugbot for commit 444f01f. Bugbot is set up for automated code reviews on this repo. Configure here.