Skip to content

Conversation

@mainman879
Copy link
Contributor

No description provided.

@mainman879
Copy link
Contributor Author

Question for ordering cards when spaces are involved: Do we order it letter by letter (thereby ignoring the spaces), or word by word? The reason I ask is because of:

  • Bloodcrazed Socialite
  • Bloodsoaked Reveler
  • Bloodtithe Harvester
  • Bloodvial Purveyor
  • Blood Petal Celebrant

The Blood Petal Celebrant has a space but the others dont, so if we go letter by letter it goes between bloodcrazed and bloodsoaked, and if we go word by word it goes before all of them.

@tooomm tooomm mentioned this pull request Nov 6, 2021
@ebbit1q
Copy link
Member

ebbit1q commented Nov 6, 2021

space is a character, it is a lower value than others

@tooomm
Copy link
Contributor

tooomm commented Nov 6, 2021

To clarify on your statement, is A a lower value than B? Or the other way around?

@mainman879 mainman879 marked this pull request as ready for review November 12, 2021 00:44
@mainman879
Copy link
Contributor Author

mainman879 commented Nov 12, 2021

Known issue: There is no token image for the Bat tokens made by Timothar, Baron of Bats. It is a 1/1 Bat token with flying and another effect. I could not find an image of this token with the other tokens.

@ebbit1q
Copy link
Member

ebbit1q commented Nov 13, 2021

if no placeholder art can be found for the card you should remove the picURL alltogether

@ebbit1q
Copy link
Member

ebbit1q commented Nov 13, 2021

actually, timothar specifies that the card gains the ability, this means it is not part of the card's text box imo

/VOW^M/related^MV/token^Mk:norm ^dt>$p^M:'<,'>sort^M:'<,'>norm ^2f<d$^P^M
sed -i s/$(cat version.txt)/$(date --utc +%Y%m%d)/ tokens.xml
date --utc +%Y%m%d >version.txt
@ebbit1q ebbit1q merged commit 9d98508 into Cockatrice:master Nov 13, 2021
@mainman879 mainman879 deleted the VOW-Tokens branch November 22, 2021 21:04
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