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

Fix the filters title covering other labels #354

Merged
merged 2 commits into from
Oct 29, 2024

Conversation

zanderle
Copy link
Collaborator

@zanderle zanderle commented Aug 2, 2024

Fix #293

There's still a slight difference. The only way to avoid that completely, would be to remove the default checkbox, and style our own. But I'm not really keen on that solution.

BTW this styling/positioning will probably be further improved in #292 , so this is just a quick solution to make it look better.

@zanderle zanderle requested review from humitos and a team as code owners August 2, 2024 14:20
@zanderle zanderle requested a review from agjohnson August 2, 2024 14:20
Copy link
Contributor

@agjohnson agjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is totally reasonable. Honestly, I'm not sure these elements really needed to be absolutely positioned from the start, but you work with what you have...

Looks good 👍

@humitos
Copy link
Member

humitos commented Aug 5, 2024

In my tests, the checkbox is still covered by the "Filters" border/padding or similar in Chrome. I'm having the same behavior reported in the issue. @zanderle can you share a screenshot on how it behaves for you?

@humitos
Copy link
Member

humitos commented Oct 8, 2024

@zanderle were you able to take a look at this?

@zanderle zanderle force-pushed the zanderle/fix-search-filters-styling branch from 5c3dbec to 59fee41 Compare October 18, 2024 09:24
@zanderle
Copy link
Collaborator Author

Should be ok now. Sorry for the delay.

Screenshot 2024-10-18 at 11 23 32

@humitos humitos merged commit 6c86cb0 into main Oct 29, 2024
4 checks passed
@humitos humitos deleted the zanderle/fix-search-filters-styling branch October 29, 2024 09:46
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.

Search: style difference between Firefox and Chrome
3 participants