Skip to content

feat: shared loading state + cleanup loading state management #5835

Open
IMB11 wants to merge 4 commits intomainfrom
cal/standardize-loading-state-management-app-and-site
Open

feat: shared loading state + cleanup loading state management #5835
IMB11 wants to merge 4 commits intomainfrom
cal/standardize-loading-state-management-app-and-site

Conversation

@IMB11
Copy link
Copy Markdown
Member

@IMB11 IMB11 commented Apr 17, 2026

  • Ensures loading state is consistent across app and website.
    • Server panel now respects the "wait-before-paint" rules we have in the app (page is not shown to the user until the data is loaded)
    • More aggressively prefetching data for instances, this solves performance problems on the worlds tab completely.
    • There are some further improvements to be done for other website pages, like users, orgs and collections but they should be done when we deduplicate those pages into the cross platform page system.

@IMB11 IMB11 requested a review from a team April 17, 2026 15:41
@IMB11 IMB11 added website Relates to Modrinth.com web frontend app Relates to Modrinth App frontend Involves work from the frontend team labels Apr 17, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 17, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
storybook d45ffa5 Commit Preview URL

Branch Preview URL
Apr 17 2026, 05:15 PM

@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented Apr 17, 2026

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

  • Performance improvements to the worlds tab for instances - worlds and servers now load pretty much immediately compared to one or two seconds like it was before.

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

  • Switching between tabs on your server panel should feel much more responsive.
  • The "Manage servers" page should load immediately in the Modrinth App, and feel much more responsive on the website.

Security

@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented Apr 17, 2026

@IMB11 IMB11 requested a review from aecsocket April 17, 2026 16:43
@IMB11 IMB11 force-pushed the cal/standardize-loading-state-management-app-and-site branch from 7b6c553 to 71665e6 Compare April 17, 2026 17:09
Copy link
Copy Markdown
Contributor

@aecsocket aecsocket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Backend lgtm

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

Labels

app Relates to Modrinth App frontend Involves work from the frontend team website Relates to Modrinth.com web frontend

Development

Successfully merging this pull request may close these issues.

3 participants