Skip to content

Commit

Permalink
fix: remove Site navigation header (#1615)
Browse files Browse the repository at this point in the history
* fix: remove Site navigation header

* test: edit the test suit
  • Loading branch information
12rambau authored Jan 4, 2024
1 parent 4aa82e2 commit e0b39b8
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 22 deletions.
8 changes: 4 additions & 4 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,3 @@
}
}
}

// Don't display the `site navigation` in the header menu
.bd-header .navbar-nav > p.sidebar-header-items__title {
display: none;
}
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
{# Displays links to the top-level TOCtree elements, in the header navbar. #}
<nav class="navbar-nav">
<p class="sidebar-header-items__title"
role="heading"
aria-level="1"
aria-label="{{ _('Site Navigation') }}">
{{ _("Site Navigation") }}
</p>
<ul class="bd-navbar-elements navbar-nav">
{{ generate_header_nav_html(n_links_before_dropdown=theme_header_links_before_dropdown, dropdown_text=theme_header_dropdown_text) }}
</ul>
Expand Down
1 change: 0 additions & 1 deletion tests/test_build.py
Original file line number Diff line number Diff line change
Expand Up @@ -964,7 +964,6 @@ def test_translations(sphinx_build_factory) -> None:
index = sphinx_build.html_tree("section1/index.html")

sidebar_primary = index.select(".bd-sidebar-primary")[0]
assert "Navigation du site" in str(sidebar_primary)
assert "Navigation de la section" in str(sidebar_primary)

sidebar_secondary = index.select(".bd-sidebar-secondary")[0]
Expand Down
3 changes: 0 additions & 3 deletions tests/test_build/navbar_ix.html
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
<div class="me-auto navbar-header-items__center">
<div class="navbar-item">
<nav class="navbar-nav">
<p aria-label="Site Navigation" aria-level="1" class="sidebar-header-items__title" role="heading">
Site Navigation
</p>
<ul class="bd-navbar-elements navbar-nav">
<li class="nav-item">
<a class="nav-link nav-internal" href="page1.html">
Expand Down
3 changes: 0 additions & 3 deletions tests/test_build/sidebar_subpage.html
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,6 @@
<div class="sidebar-header-items__center">
<div class="navbar-item">
<nav class="navbar-nav">
<p aria-label="Site Navigation" aria-level="1" class="sidebar-header-items__title" role="heading">
Site Navigation
</p>
<ul class="bd-navbar-elements navbar-nav">
<li class="nav-item">
<a class="nav-link nav-internal" href="../page1.html">
Expand Down

0 comments on commit e0b39b8

Please sign in to comment.