Skip to content

Fix visible background over border radius in search dropdown #90

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

Merged
merged 1 commit into from
Oct 9, 2017
Merged

Fix visible background over border radius in search dropdown #90

merged 1 commit into from
Oct 9, 2017

Conversation

imagentleman
Copy link
Contributor

Currently the algolia search results have a weird and subtle but eye-twitch-inducing artifact, where you can see the white background on the top round borders of the dropdown.

It looks like this:

before

This PR moves the background from the top container to each individual row that must be white in order to achieve the same effect.

It would look like this:

after

@reactjs-bot
Copy link

Deploy preview ready!

Built with commit 06a8b8e

https://deploy-preview-90--reactjs.netlify.com

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

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

This looks great on desktop! But it seems to cause horizontal overflow on the mobile layout. Can you fix that one too?

@imagentleman
Copy link
Contributor Author

@bvaughn , yep i'll be happy to fix any issues.

Just visited the autogenerated preview of the PR on my phone (https://deploy-preview-90--reactjs.netlify.com/) but at first glance couldn't see the horizontal overflow : S

Can you attach a screenshot or give me a few more details?

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

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

I think this might actually just be something Chrome is doing because now I'm able to repro it on the live site and I wasn't last night. Sorry.

I think this is a nice improvement. Let's move forward with it. 😄

@bvaughn bvaughn merged commit d8d8f5d into reactjs:master Oct 9, 2017
BetterZxx pushed a commit to BetterZxx/react.dev that referenced this pull request Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants