AEFeinstein/mtg-familiar

Rework Deck Sort for Lands

AEFeinstein opened this issue · 0 comments

The way Familiar sorts cards by supertype is by running through the following list, top to bottom, and if the type matches, it puts that card in that bucket. So a bunch of things come before lands, which affects Artifact Lands, and also Dryad Arbor and Urza's Saga.

<string-array name="card_types_extra">
  <item>@string/card_type_creature</item>
  <item>@string/card_type_planeswalker</item>
  <item>@string/card_type_instant</item>
  <item>@string/card_type_sorcery</item>
  <item>@string/card_type_artifact</item>
  <item>@string/card_type_enchantment</item>
  <item>@string/card_type_land</item>
  <item>@string/card_type_conspiracy</item>
  <item>@string/card_type_phenomenon</item>
  <item>@string/card_type_plane</item>
  <item>@string/card_type_scheme</item>
  <item>@string/card_type_vanguard</item>
</string-array>

CardComparatorSupertype should be reworked to always sort cards card_type_land in that category, regardless of other types.