-
Notifications
You must be signed in to change notification settings - Fork 347
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
fix: extra padding on external results page #2409
Conversation
@kunatastic @gigincg test failed, the filter and the buttons are still close |
test approved |
@kunatastic please do work on the merge conflict |
@khavinshankar @nihal467 fixed the merge conflict. Apologies for long delay |
❌ Deploy Preview Failed. CC: @rithviknishad
|
test appproved |
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.
please do make the recommended changes
className={`btn btn-primary ${ | ||
downloadLoading ? "pointer-events-none" : "" | ||
}`} |
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.
className={`btn btn-primary ${ | |
downloadLoading ? "pointer-events-none" : "" | |
}`} | |
className={clsx("btn btn-primary", downloadLoading && "pointer-events-none")} |
can you please use clsx here, it improves the readability and don't forget to import clsx if not already done so
@kunatastic check the conflict file ! |
@nihal467 this pr is good for testing |
test approved |
@kunatastic there are some minor changes, can you please address them |
Co-authored-by: Khavin Shankar <khavinshankar@gmail.com>
Co-authored-by: Khavin Shankar <khavinshankar@gmail.com>
Co-authored-by: Khavin Shankar <khavinshankar@gmail.com>
@kunatastic please do work on the merge conflict |
fixes #2369
Updated UI
![image](https://user-images.githubusercontent.com/61267953/168965117-ac298a65-fbf1-44b8-9d5d-31e6dac99b4d.png)