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

Vertically center search box #213

Closed
wants to merge 1 commit into from
Closed

Conversation

jesobreira
Copy link
Contributor

I don't know if it's my OCD but this was taking my attention every time I visited php.net:

captura de tela 2018-03-29 as 00 53 52

This very small PR moves the search field closer to the middle.

captura de tela 2018-03-29 as 00 54 12

@@ -182,7 +182,7 @@ textarea {
.navbar-search {
position: relative;
float: left;
margin-top: .625rem;
margin-top: .770rem;
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be better to use a vertical centring technique, instead of arbitrarily adjusting one margin, to ensure consistency across all devices and screen sizes.

Copy link

Choose a reason for hiding this comment

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

@Bilge is there a place one can view browser stats for php.net so we know the most recent method we could use?

@php-pulls
Copy link

Comment on behalf of petk at php.net:

Thank @jesobreira for your contribution. Applied via d26b9a7

@php-pulls php-pulls closed this Oct 17, 2018
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.

None yet

4 participants