Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

49 colors filter breaks single tournament view #50

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

doomdg
Copy link
Collaborator

@doomdg doomdg commented Aug 12, 2023

image

Fixed color filters. Removed commander filter with no conditions

@doomdg doomdg linked an issue Aug 12, 2023 that may be closed by this pull request
@JasonQiu21
Copy link
Collaborator

hitting reset resets color filters but not graphic

on commander view, color filter is maintained when reset button is clicked:

Peek.2023-08-14.12-40.mp4

additionally, colorID should not be sent to server:

image
image

@JasonQiu21
Copy link
Collaborator

screen recorded wrong window:

Peek.2023-08-14.12-40.mp4

note that any filters change resets the colorID filter.

@JasonQiu21 JasonQiu21 requested a review from g00gol October 18, 2023 00:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Colors Filter breaks Single Tournament View
2 participants