Skip to content

refactor: content tab styling - #7091

Merged
mergify[bot] merged 19 commits into
mainfrom
cal/content-tab-restyling
Aug 13, 2026
Merged

refactor: content tab styling#7091
mergify[bot] merged 19 commits into
mainfrom
cal/content-tab-restyling

Conversation

@IMB11

@IMB11 IMB11 commented Aug 10, 2026

Copy link
Copy Markdown
Member

+ some external file improvements

@IMB11
IMB11 requested a review from a team August 10, 2026 16:40
@modrinth-bot

modrinth-bot commented Aug 10, 2026

Copy link
Copy Markdown
Member

Pull request changelog

App

Added

  • Added detailed Content tab filters for authors, categories, enabled state, updates, warnings, open-source projects, external files, and shared content.
  • External files now show embedded names, versions, and icons in the Content tab.
  • Added total playtime to server instance headers, replacing the "Linked to server" text which is no longer needed due to the content tab changes.

Changed

  • Redesigned the Content tab modpack card to separate managed modpack, server, and shared-instance content.
  • Moved shared-instance update notifications and actions into the Content tab's managed-content card.
  • Replaced the Content tab's cycling sort control with a menu for alphabetical and date-added sorting.

Deprecated

Removed

Fixed

  • Fixed exported modpacks requiring client-only content on servers and server-only content on clients.
  • Fixed the Discover page hiding compatible resource packs and vanilla shaders when browsing content for an instance.
  • Fixed server project instances which used vanilla appearing as unlinked in Installation settings.

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

  • Added Content tab filters for authors, categories, enabled state, updates, warnings, open-source projects, and external files.

Changed

  • Redesigned the Content tab's modpack card
  • Replaced the Content tab's cycling sort control with a menu for alphabetical and date-added sorting.

Deprecated

Removed

Fixed

Security

@modrinth-bot

modrinth-bot commented Aug 10, 2026

Copy link
Copy Markdown
Member

@IMB11
IMB11 marked this pull request as draft August 10, 2026 17:24
@IMB11
IMB11 marked this pull request as ready for review August 11, 2026 13:07

@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

@@ -0,0 +1,185 @@
<script setup lang="ts">

@tdgao tdgao Aug 11, 2026

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.

Rename these to remove the prefix. I think we should make it a standard
Non-blocking though

@mergify

mergify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 7 minutes 24 seconds in the queue, including 6 minutes 35 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Aug 13, 2026
@mergify
mergify Bot merged commit 06b7c44 into main Aug 13, 2026
21 checks passed
@mergify
mergify Bot deleted the cal/content-tab-restyling branch August 13, 2026 14:55
@mergify mergify Bot removed the queued label Aug 13, 2026
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.

3 participants