When the list os sorted using another column, you don’t  need to
compare that column’s value with the searched term

Related to #808

When closing the app/game tab with a zone revealed the client will
crash. This was due to the zones not being deleted.

The problem was that back when I added #336 I made it so that if YOU
didnt recive the PT of the token, then it would be fetched from your
tokens.xml, the issue is that we needed to send that value to the
server. The server sees it as a 0/0

This fix will send the PT from the tokens.xml if available.

Tokens from decks will now show in the token dlg window

Adjusted proportions to make the table fit better.

Needs to be checked on UHD screens.

+ Removed unused buttons
+ remade button art
+ fixed crash with font colors/replays
+ removed unused timestamp label in replay

Felt like areas different to the current zone should be segregated.

It is awkward to have to use precise punctuation when searching for
cards. Planeswalkers and legendary creatures often have "," in the name
and you have to enter it.

This commit means you no longer need to do that.

+ Can use " " in place of "-" (example: Wilt-Leaf Liege)
+ Can use "" in place of non word chars (Example: Hero's Downfall and
Ajani, Caller of the Pride)

+ Ctrl + N

Was an issue with double clicking the icon when the app was full screen/
fullscreen + minimized.

Now works as expected.

Previously would not open the client if the window was minimized.
Added the same popup fix for mentions

+ will now only popup with a mention alert when you are not on the same
tab/window.
+ will also no longer popup by mentioning yourself

Workaround the two remaining issues, mostly fixing the “load deck”
screen in tab_game:
 * Only preload up to the first 300 cards of a deck;
 * don’t even bother trying to load card images if they’ll be shown
with width <= 10

Trying a 17k cards deck like the one proposed in #67 is still a no go,
mostly due to the cpu time used to create all the objects. Anyway, i
tested decks up to 8k cards and they works fine now (if you like to
wait while trice is loading them)..

Fix selection problem when sorting the deck list

if you have the setting to NOT play cards to the stack enabled, it meant
that any spells (like Lightning Bolt) would be cast to the stack but no
arrow would show. This was an edge case I missed.

Missed from #879

Before:

After:

I think the majority of players use this feature for
goldfishing/practicing. Defaulting to one player for convenience.

Set to false by default, let me know if you think it should be true.

As all cards are played to the left of the screen this feels more
comfortable to use as you dont need to keep looking at different areas
of the screen.

Will auto rearrange when changed during game-play.

Before:

After:

Previously if you middle click on a counter (life/mana) and then click
away, depending on the button clicked, the counter would +/-.

I have added a fix to make sure the mouse is over the counter to change
it.

Previously had - numbers at the top and + at the bottom.
This feels like it makes more sense.

I noticed that sometimes I would have lots of icons in the sys tray. I
would have to mouse over them to make then be removed. This also happens
with some other programs too. I have added some code to hide() the sys
icon when the client is close, it seems to be helping with the issue.
Hard to reproduce, might also only be a windows issue.

Avoid error “QIODevice::read: device not open”

The check was inverted. LoadStatus::Ok = 0, so only when
db->loadCardDatabase() returns 0 we know the card database path is
correct

Users can now target from their hand. This will cast the card and point
to the target.