fix(ui): standardize loading indicators on MCP Registry, Teams, API T…#2950
Merged
crivetimihai merged 2 commits intoIBM:mainfrom Feb 15, 2026
Merged
Conversation
26c391e to
2c497ea
Compare
Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk>
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
2c497ea to
9082596
Compare
crivetimihai
approved these changes
Feb 15, 2026
Member
crivetimihai
left a comment
There was a problem hiding this comment.
LGTM — nice cleanup, thanks @omorros!
All four loading indicators now match the standard pattern used across the rest of the admin UI (flex items-center justify-center p-4, h-8 w-8 SVG spinner, ml-2, text-indigo-600 dark:text-indigo-400).
I rebased onto main and also applied the same fix to the Users section loading indicator (line ~6088), which had the same p-8/ml-3 inconsistency. Now all list-level loading indicators are fully consistent.
suciu-daniel
pushed a commit
that referenced
this pull request
Feb 16, 2026
#2950) * fix(ui): standardize loading indicators across admin pages (#2946) Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk> * fix(ui): standardize Users loading indicator to match pattern Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> --------- Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk> Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
vishu-bh
pushed a commit
that referenced
this pull request
Feb 18, 2026
#2950) * fix(ui): standardize loading indicators across admin pages (#2946) Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk> * fix(ui): standardize Users loading indicator to match pattern Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> --------- Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk> Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> Co-authored-by: Mihai Criveti <crivetimihai@gmail.com> Signed-off-by: Vishu Bhatnagar <vishu.bhatnagar@ibm.com>
kcostell06
pushed a commit
to kcostell06/mcp-context-forge
that referenced
this pull request
Feb 24, 2026
IBM#2950) * fix(ui): standardize loading indicators across admin pages (IBM#2946) Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk> * fix(ui): standardize Users loading indicator to match pattern Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> --------- Signed-off-by: Oriol Morros Vilaseca <OM368@student.aru.ac.uk> Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 Related Issue
Closes #2946
📝 Summary
Standardized the loading indicators on MCP Registry, Teams, API Tokens, and System Logs pages to use the same indigo SVG
spinner pattern used by Gateways, Tools, and other pages.
Changes in
mcpgateway/templates/admin.html:p-8/ml-3→p-4/ml-2)🏷️ Type of Change
🧪 Verification
make lintmake testmake coverage✅ Checklist
make black isort pre-commit)📓 Notes (optional)
This is an HTML-only change in a single template file. No Python code was modified, so lint/test/coverage are not affected.
Tested locally via Docker Compose by hard-reloading each affected page and verifying the loading spinner matches the
standard pattern on Gateways/Tools pages.