Show loading screen while fetching sites in Pull an existing site modal - #2237
Conversation
📊 Performance Test ResultsComparing 046d1df vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
katinthehatsite
left a comment
There was a problem hiding this comment.
Thanks for fixing this, this worked as expected for me 👍
ivan-ottinger
left a comment
There was a problem hiding this comment.
Thanks for the fix, Roberto! The loading message is displaying as expected and the experience is more clear now. 👍🏼
I have noticed one issue - the scrollbars are displayed on the side like this:
Some local sites already exist:
CleanShot.2025-12-11.at.17.07.59.mp4
No local sites:
CleanShot.2025-12-11.at.17.12.29.mp4
|
Thanks @katinthehatsite and @ivan-ottinger for your reviews!
Well spotted! That was an existing issue, but I have fixed it as part of f5594d4. Now the list of sites is no longer rendered while sites are loading. I have also refactored the component a bit for better readability. |
Thank you for addressing that, Roberto! I can confirm the loading is now working correctly! 👌🏼 |

Related issues
Proposed Changes
Loading…messageCleanShot.2025-12-11.at.14.37.39.mp4
Testing Instructions
npm startAdd siteandPull an existing siteLoading…message, if you cannot see the message, tryCMD+Rto refresh dataPre-merge Checklist