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

no rounding of search and version selector #667

Closed
wants to merge 2 commits into from
Closed

Conversation

KristofferC
Copy link
Member

@KristofferC KristofferC commented Mar 11, 2018

I must admit that I find the rounding of the search and version selector quite ugly. This changes it to be just rectangular.

After:

screen shot 2018-03-09 at 22 25 57

@mortenpi
Copy link
Member

I'm basically neutral on the matter. This choice is from the original Sphinx theme. I am slightly sceptical of making these small subjective changes since you can go back and forth indefinitely. But since there seems to be support for this I propose let's leave this open for a week or two and, if there are no dissenting opinions, merge then.

The slight rounding of the selector you have seems to be platform dependent. For reference, this is what is looks like on Ubuntu / Gnome:

rounding

@KristofferC
Copy link
Member Author

Alright, I'll just configure these small things in my own css from now on. I think we can close this.

@mortenpi mortenpi mentioned this pull request May 10, 2018
@mortenpi mortenpi deleted the kc/square branch September 12, 2022 07:57
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.

None yet

2 participants