Skip to content

Commit 9101d22

Browse files
authored
doc: fix site sidebar logo position (#5990)
* Fix doc site sidebar logo position * Add padding to logo
1 parent ebb8390 commit 9101d22

File tree

2 files changed

+7
-14
lines changed

2 files changed

+7
-14
lines changed

docs/source/_static/theme_overrides.css

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,16 +45,9 @@ code {
4545
.navbar-brand {
4646
height: auto;
4747
width: auto;
48+
padding: 0 2em;
4849
}
4950

50-
a.navbar-brand img {
51-
height: auto;
52-
width: auto;
53-
max-height: 15vh;
54-
max-width: 100%;
55-
}
56-
57-
5851
/* This is the bootstrap CSS style for "table-striped". Since the theme does
5952
not yet provide an easy way to configure this globaly, it easier to simply
6053
include this snippet here than updating each table in all rst files to
@@ -69,12 +62,12 @@ add ":class: table-striped" */
6962
custimized template, there is more content above the navigation, i.e.
7063
larger logo: if we don't decrease the max-height, it will overlap with
7164
the footer.
72-
Details: min(15vh, 110px) for the logo size, 8rem for search box etc*/
65+
Details: 8rem for search box etc*/
7366

7467
@media (min-width:720px) {
7568
@supports (position:-webkit-sticky) or (position:sticky) {
7669
.bd-links {
77-
max-height: calc(100vh - min(15vh, 110px) - 8rem)
70+
max-height: calc(100vh - 8rem)
7871
}
7972
}
8073
}

docs/source/_templates/docs-sidebar.html

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@
1313
{{ generate_nav_html("sidebar", startdepth=0, maxdepth=4, collapse=False, includehidden=True, titles_only=True) }}
1414
{% endif %}
1515
</div>
16-
</nav>
1716

17+
<a class="navbar-brand" href="{{ pathto(master_doc) }}">
18+
<img src="{{ pathto('_static/images/DataFusion-Logo-Background-White.png', 1) }}" class="logo" alt="logo">
19+
</a>
20+
</nav>
1821

19-
<a class="navbar-brand" href="{{ pathto(master_doc) }}">
20-
<img src="{{ pathto('_static/images/DataFusion-Logo-Background-White.png', 1) }}" class="logo" alt="logo">
21-
</a>

0 commit comments

Comments
 (0)