blakevanlan / KingdomCreator

Deck randomizer for Dominion
MIT License
44 stars 31 forks source link

Sort by cost is not working #145

Closed markbrockettrobson closed 1 year ago

markbrockettrobson commented 1 year ago
image
rPraml commented 1 year ago

Sort seems to be completely broken. Sot by set: Screenshot_20230129_175923_Chrome

gillesgros commented 1 year ago

for cost, just change line in src/components/SortableSupplyCards.vue

get supplyCardsWithBane() {
    //const cards =  SupplyCardSorter.sort(this.supplyCards.concat() as SupplyCard[], this.sortOption, this.$t.bind(this));
    const cards = this.supplyCards.concat();
    if (this.kingdom.supply.baneCard) {
      cards.push(this.kingdom.supply.baneCard);
    }
gillesgros commented 1 year ago

fixed on master and deplyed

markbrockettrobson commented 1 year ago

Nice work thank you