Skip to content

Add unified Deck Browser for home and deck editor#10636

Open
Madwand99 wants to merge 3 commits intoCard-Forge:masterfrom
Madwand99:AddDeckBrowser
Open

Add unified Deck Browser for home and deck editor#10636
Madwand99 wants to merge 3 commits intoCard-Forge:masterfrom
Madwand99:AddDeckBrowser

Conversation

@Madwand99
Copy link
Copy Markdown
Contributor

@Madwand99 Madwand99 commented May 9, 2026

This branch adds a shared Deck Browser experience for the desktop Home tab and Deck Editor.

Major changes:

  • Replaces the Home tab deck list with a navigable Deck Browser that supports folders, .., virtual folders, generated deck options, net decks, archive decks, recursive search, refresh, and richer deck columns.
  • Adds a Deck Browser tab to the Deck Editor and removes the old Commander/Oathbreaker/Brawl/Tiny Leaders deck tabs.
  • Hooks Deck Editor browser navigation into the current deck save/load flow, including save target tracking by current folder.
  • Reorganizes generated deck options into navigable virtual folders, with separate icons for folders, net folders, generated folders, and generated deck choices.
  • Improves selection behavior so folder rows cannot be chosen as playable decks, restored/selected decks are highlighted and scrolled into view, and .. navigation preserves useful selection context.
  • Adds refresh behavior for normal folders, net decks, and archive decks.
  • Adds recursive Deck Browser search that runs on Enter and restores normal folder contents when canceled.
  • Consolidates generated deck proxy behavior into GeneratedDeckProxy.
  • Consolidates net archive deck storage/loading behavior through NetDeckStorageBase, reducing repeated archive-specific code.

Validation:

mvn -pl forge-gui-desktop -am -DskipTests compile passes.

DeckBrowser4 DeckBrowser3 DeckBrowser2 DeckBrowser1

@tool4ever
Copy link
Copy Markdown
Contributor

well this is huge...

can you spin off the NetDeckStorageBase refactor into its own PR for a first size reduction?

@Madwand99
Copy link
Copy Markdown
Contributor Author

well this is huge...

can you spin off the NetDeckStorageBase refactor into its own PR for a first size reduction?

Yeah it's kept me pretty busy. And sure, will try that and see how it looks.

@MostCromulent
Copy link
Copy Markdown
Contributor

@Madwand99 currently the "format" of a deck (constructed/commander/oathbreaker) etc is determined by which deck browser panel you have open at the time the deck is created, so you have to be in the commander tab when you create your deck or else it doesn't get a command zone.

How does this work in your PR if we're collapsing those different browser panels into a general purpose one? i.e. how do you assign your deck to commander format here?

(FWIW I think the current "open format tab first, then create deck" approach is highly unintuitive hence #10517 - so I've got nothing against a unified approach, just want to make sure we don't lose functionality along the way).

@Madwand99
Copy link
Copy Markdown
Contributor Author

Madwand99 commented May 10, 2026

How does this work in your PR if we're collapsing those different browser panels into a general purpose one? i.e. how do you assign your deck to commander format here?

@MostCromulent In the editor, the new Deck Browser is not just a passive file list. Navigating into a real format folder updates the Current Deck section’s active deck type. So entering the commander folder switches the editor to Commander handling, and new/saved decks from there get the command zone as expected. The browser is general-purpose UI, but the selected real folder still provides the format context.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants