Skip to content
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

Merged
merged 11 commits into from
Sep 19, 2024

Conversation

kr-matthews
Copy link
Contributor

@kr-matthews kr-matthews commented Aug 27, 2024

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?

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
@kr-matthews kr-matthews self-assigned this Aug 27, 2024
Copy link
Member

@gregorbg gregorbg left a 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.

@kr-matthews kr-matthews force-pushed the task/competitions-ui-filter-changes branch from 6288128 to 7cc35fe Compare September 12, 2024 02:55
@kr-matthews
Copy link
Contributor Author

@gregorbg Any insight on the drop-down icon issue? From the linked issue:

The dropdown is 'clearable', so when something is selected, there should be an "x" icon to clear it. The component does this via class="dropdown icon clear". For some reason, that's not working, and the dropdown arrow icon is still showing up. Possibly we're missing semantic-ui-css, or need to update semantic-ui-react? I'm stuck here.

@gregorbg
Copy link
Member

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.

@kr-matthews
Copy link
Contributor Author

I have now manually ported them over and pushed a commit, which should fix the isse.

Running locally, I see nothing (as opposed to the "x" I should, or the "v" drop-down arrow I was before).

image

I have been having trouble with docker recently, so maybe it's just my setup.

@gregorbg
Copy link
Member

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)

@gregorbg gregorbg merged commit 61d34a4 into thewca:main Sep 19, 2024
2 checks passed
@kr-matthews kr-matthews deleted the task/competitions-ui-filter-changes branch September 19, 2024 04:10
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.

2 participants