-
Notifications
You must be signed in to change notification settings - Fork 176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New Competition Search UI | various improvements #9816
New Competition Search UI | various improvements #9816
Conversation
You can still use the url search params to do this.
the icon still won't show as "x" when it should
but keep All Years option at the top
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes fine per se. You should probably move WCA_EVENT_IDS
to filterUtils
to avoid the cyclic dependency as you suggested.
6288128
to
7cc35fe
Compare
@gregorbg Any insight on the drop-down icon issue? From the linked issue:
|
That was actually an interesting issue to debug. I checked that we are definitely using an up-to-date version of the stylesheets, but we're using Fomantic CSS (a maintained fork) whereas the clearable feature lives in Semantic CSS. The clearable feature must have been one of the last things they added around the time it was forked, so the CSS styles are missing in Fomantic stylesheets. I have now manually ported them over and pushed a commit, which should fix the isse. |
This most likely means that your browser is caching some stylesheets. Try hitting Ctrl+Shift+R (or the Mac equivalent, which I think is Cmd+Shift+R) |
Addresses a few issues from #9788 - see commit list.
The region dropdown bug isn't fully fixed - I left details about that on the linked issue.Edit: not sure how to best resolve the cyclical dependency. Should I move this (or all) filter constants to a new file?