Skip to content

fix(ui): move warm-up to startup - #1073

Open
saadndm wants to merge 1 commit into
Polyfrost:v1from
saadndm:fix/warm-up
Open

fix(ui): move warm-up to startup#1073
saadndm wants to merge 1 commit into
Polyfrost:v1from
saadndm:fix/warm-up

Conversation

@saadndm

@saadndm saadndm commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Warm-up

  • Moved from world loading screen to the MC startup loading screen so that OC can be opened in titlescreen smoothly
  • Removed background class preloading (required classes already loaded in warmup, and didnt work anyways because it targeted wrong modid)
  • Removes search focus warmup (benchmark shows it has no stutter and previous one didnt work anyways)
  • Holds startup loading screen until warm-up finishes (15s timeout)

Ui State and cleanup

  • Fixes sometimes opening the Mods page scrolled to the bottom by resetting scroll position after warm-up
  • Clears navigation history after warm-up
  • Centralized error logging and cleanup in ComposePreloader.failStartup()

Font loading

  • Registered skia font listener before MC's initial resource reload, and moved it to a worker so that it doesn't use the synchronous fallback

Icons

  • Fixes icon preload cache keys not matching paths, and made it actually reuse cached file bytes

@saadndm
saadndm marked this pull request as draft September 10, 2026 07:34
@saadndm
saadndm marked this pull request as ready for review September 10, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant