AlenkaF commented on code in PR #38148:
URL: https://github.com/apache/arrow/pull/38148#discussion_r1350048127
##########
docs/source/_static/theme_overrides.css:
##########
@@ -21,12 +21,18 @@
/* Customizing with theme CSS variables */
:root {
- /* Change header hight to make the logo a bit larger */
- --pst-header-height: 6rem;
/* Make headings more bold */
--pst-font-weight-heading: 600;
}
+/* Change header hight to make the logo a bit larger */
+/* only on wider screens */
+@media only screen and (min-width: 950px){
Review Comment:
Ok, setting the minimum width of the screen to have the bigger header height
to 1380px gave me two layouts to compare as the limit for the double header is
a bit higher:
- just under 1380px and smaller header height:
- just over 1380px and bigger header height:
I agree the smaller header size is better in this case also (double navbar),
but the logo is too small to read it and that is not something I would keep.
I propose to have the bigger header height for 1400px but in the case of
default header height I would use a different logo: [Chevron-only
logo](https://arrow.apache.org/img/arrow-logo_chevrons_black-txt_white-bg.png)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
