Skip to content

Add X button to clear the challenge search. #4255

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

Conversation

narekcat
Copy link
Contributor

I added X button to clear the challenge search as described in this issue: #3622.

@skyhit
Copy link
Contributor

skyhit commented Apr 16, 2020

@narekcat tests failed, please check

@skyhit
Copy link
Contributor

skyhit commented Apr 16, 2020

@narekcat for these .snap are they purposely to be changed? if not, why commit in this PR?

@narekcat
Copy link
Contributor Author

@skyhit They purposed to be changed, because to add X button I added span tag in ChallengeSearchBar.jsx file and new property to ChallengeFilters.jsx file. But tests still fails, because classNames has auto generated values. Now working on it.

@narekcat narekcat force-pushed the hot-fix-x-button-to-clear-search branch from 33670fd to a615a6c Compare April 16, 2020 07:48
@narekcat
Copy link
Contributor Author

@skyhit I fixed tests issues.

@skyhit skyhit merged commit cffea43 into topcoder-platform:hot-fix-x-button-to-clear-search Apr 16, 2020
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