Skip to content

refactor: settings cleanup - #7968

Open
IMB11 wants to merge 2 commits into
mainfrom
cal/settings-layout
Open

IMB11 wants to merge 2 commits into
mainfrom
cal/settings-layout

Conversation

@IMB11

@IMB11 IMB11 commented Oct 14, 2026

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

  • Reorganized the Settings modal into Display, Account, Instance defaults, and System sections.
    • Default Java and memory, Window, and Launch hooks settings now each have their own tab under Instance defaults.
    • Moved Java installations into the Java and memory tab.
    • Renamed the Synced settings tab to Syncing.
    • Split the Resource management tab into Resources and Troubleshooting tabs.
    • Renamed Purge cache to Clear cache on the Troubleshooting tab.
  • Reorganized the instance settings modal into Instance and Game sections.
    • Java and memory, Window, Launch hooks, and Syncing are now separate tabs that replace the Sync overrides tab.
    • Each Game tab has a Manage default settings button that opens the matching tab in the Settings modal.
  • Refreshed the sidebar, backgrounds, and scrollbars in the Settings, instance settings, and server settings modals.
  • Settings modals that open on a specific tab now scroll the sidebar to show that tab.

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@IMB11
IMB11 force-pushed the cal/settings-layout branch from ee40d97 to 752d638 Compare October 14, 2026 16:35
@github-actions

Copy link
Copy Markdown
Contributor

@IMB11

IMB11 commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

@Mergifyio queue

@mergify

mergify Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Merge Queue Status

  • 🟠 Waiting for queue conditions
  • ⏳ Enter queue
  • ⏳ Run checks
  • ⏳ Merge
Waiting for any of
  • check-neutral = Lint and Test
  • check-skipped = Lint and Test
  • check-success = Lint and Test
All conditions
  • any of [🔀 queue conditions]:
    • all of [📌 queue conditions of queue rule default]:
      • any of [🛡 GitHub branch protection]:
        • check-neutral = Lint and Test
        • check-skipped = Lint and Test
        • check-success = Lint and Test
      • github-review-approved [🛡 GitHub branch protection]
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of [📌 queue -> configuration change requirements]:
    • -mergify-configuration-changed
    • check-success = @mergify/Configuration changed
    • check-success = @mergify/Configuration has been deleted
  • any of [📌 queue requirement]:
    • check-neutral = @mergify/Mergify Merge Protections
    • check-skipped = @mergify/Mergify Merge Protections
    • check-success = @mergify/Mergify Merge Protections

@tdgao tdgao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

This branch has not been deployed

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants