-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update heading and navbar font to Quicksand #15
Conversation
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.
doc/conf.py
Outdated
@@ -109,7 +109,7 @@ | |||
# Xanadu theme options (see theme.conf for more information). | |||
html_theme_options = { | |||
"navbar_name": "Xanadu Sphinx Theme", | |||
"navbar_logo_colour": "#2d7c7f", | |||
"navbar_logo_colour": "#f48fb1", |
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.
Good catch! 🎣
font-weight: 400; | ||
font-family: 'Roboto', sans-serif !important; | ||
font-weight: 500; | ||
font-family: 'Quicksand', sans-serif !important; |
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.
Why is Quicksand in quotation marks here and below?
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.
Not sure, I just copied what Google Fonts told me to do in their code snippets :)
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.
Fair enough although it would be nice to be consistent across the file! 🔗
At the moment it is 16px, I think I prefer it as it is in terms of how it is balanced across the rest of the page 🤔 I had a quick look at other OSS packages:
I think Torch only gets away with such large font for their navbar because their navbar is super thick |
Ah, good call on the OSS comparison. I agree this should be fine! 👌 |
No description provided.