jorisvandenbossche commented on code in PR #38148:
URL: https://github.com/apache/arrow/pull/38148#discussion_r1350024167
##########
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:
How did you decide on the min width of 950? From doing a quick experiment
with the "Responsive design mode" in Firefox, I already see it wraparound
(causing a very large header) at a width of around 1380 px
--
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]
