Skip to content

Commit 4cc679a

Browse files
fix: changed some nav landmarks to div and div to nav for better accessibility
risk: low JIRA: DP-3086
1 parent 5ad5041 commit 4cc679a

File tree

3 files changed

+44
-40
lines changed

3 files changed

+44
-40
lines changed
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
<nav class="gd-docs-header-nav__center gd-header" aria-label="Main">
2+
<input type="checkbox" role="button" id="gd-header-mobile-menu-trigger-input" name="gd-header-mobile-menu-trigger-input" class="gd-header-mobile gd-header-mobile-menu-trigger-input" aria-label="Open navigation menu" aria-controls="gd-header-mobile-menu" aria-expanded="false">
3+
<label tabindex="0" for="gd-header-mobile-menu-trigger-input" class="gd-header-mobile gd-header-mobile-menu-trigger"><span></span></label>
4+
<ul class="gd-docs-header-nav__menu gd-header-links">
5+
<li class="gd-docs-header-nav__menuitem gd-header-link-item gd-header-link-item-mobile-only">
6+
<a href="https://www.gooddata.com/learn/" class="gd-docs-header-nav__menulink gd-header-link gd-header-link__main">Learn</a>
7+
</li>
8+
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
9+
<a href="https://university.gooddata.com/" class="gd-docs-header-nav__menulink gd-header-link">University</a>
10+
</li>
11+
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
12+
<a href="https://community.gooddata.com/" class="gd-docs-header-nav__menulink gd-header-link">Community</a>
13+
</li>
14+
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
15+
<a href="https://www.gooddata.com/docs/cloud/" class="gd-docs-header-nav__menulink gd-header-link gd-header-link__active">Documentation</a>
16+
</li>
17+
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
18+
<a href="https://support.gooddata.com/hc/en-us" class="gd-docs-header-nav__menulink gd-header-link">Support</a>
19+
</li>
20+
<li class="gd-docs-header-nav__menuitem gd-header-link-item gd-header-link-item-mobile-only">
21+
<a href="https://www.gooddata.com/" class="button">Go to GoodData.com</a>
22+
</li>
23+
</ul>
24+
</nav>
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
<nav class="gd-docs-header-nav__left" aria-label="Breadcrumbs">
2+
<ul class="gd-breadcrumb">
3+
<li class="gd-breadcrumb-item">
4+
<a href="https://www.gooddata.com/docs/">All Products</a>
5+
</li>
6+
<li class="gd-breadcrumb-item">
7+
<span> &nbsp;Python SDK</span>
8+
</li>
9+
</ul>
10+
</nav>

docs/layouts/partials/navbar.html

Lines changed: 10 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,18 @@
1+
{{/* (C) 2023 GoodData Corporation */}}
2+
13
{{ $cover := .HasShortcode "blocks/cover" }}
2-
<nav class="gd-docs-header-nav gd-docs-header-nav__first">
3-
<div class="gd-docs-header-nav__left">
4+
<div class="gd-docs-header-nav gd-docs-header-nav__first">
5+
<nav class="gd-docs-header-nav__left" aria-label="Homepage">
46
<a class="gd-docs-header-nav__logo gd-header-nav__logo" href="{{ (printf "%s%s/" .Site.Home.Permalink "latest" ) | safeURL }}" >
57
{{ with resources.Get "icons/GoodDataLearn.svg" }}{{ ( . | minify).Content | safeHTML }}{{ end }}
68
</a>
79
<span class="gd-header-nav__logo-text">Learn</span>
8-
</div>
9-
<div class="gd-docs-header-nav__center gd-header">
10-
<input type="checkbox" id="gd-header-mobile-menu-trigger-input" name="gd-header-mobile-menu-trigger-input" class="gd-header-mobile gd-header-mobile-menu-trigger-input">
11-
<label for="gd-header-mobile-menu-trigger-input" class="gd-header-mobile gd-header-mobile-menu-trigger"><span></span></label>
12-
<ul class="gd-docs-header-nav__menu gd-header-links">
13-
<li class="gd-docs-header-nav__menuitem gd-header-link-item gd-header-link-item-mobile-only">
14-
<a href="https://www.gooddata.com/learn/" class="gd-docs-header-nav__menulink gd-header-link gd-header-link__main">Learn</a>
15-
</li>
16-
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
17-
<a href="https://university.gooddata.com/" class="gd-docs-header-nav__menulink gd-header-link">University</a>
18-
</li>
19-
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
20-
<a href="https://community.gooddata.com/" class="gd-docs-header-nav__menulink gd-header-link">Community</a>
21-
</li>
22-
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
23-
<a href="https://www.gooddata.com/docs/cloud/" class="gd-docs-header-nav__menulink gd-header-link gd-header-link__active">Documentation</a>
24-
</li>
25-
<li class="gd-docs-header-nav__menuitem gd-header-link-item">
26-
<a href="https://support.gooddata.com/hc/en-us" class="gd-docs-header-nav__menulink gd-header-link">Support</a>
27-
</li>
28-
<li class="gd-docs-header-nav__menuitem gd-header-link-item gd-header-link-item-mobile-only">
29-
<a href="https://www.gooddata.com/" class="button">Go to GoodData.com</a>
30-
</li>
31-
</ul>
32-
</div>
10+
</nav>
11+
{{ partial "header-menu.html" . }}
3312
{{ partial "header-right.html" . }}
34-
</nav>
35-
<nav class="gd-docs-header-nav gd-docs-header-nav__second">
36-
<div class="gd-docs-header-nav__left">
37-
<ul class="gd-breadcrumb">
38-
<li class="gd-breadcrumb-item">
39-
<a href="https://www.gooddata.com/docs/">All Products</a>
40-
</li>
41-
<li class="gd-breadcrumb-item">
42-
<span> &nbsp;Python SDK</span>
43-
</li>
44-
</ul>
45-
</div>
13+
</div>
14+
<div class="gd-docs-header-nav gd-docs-header-nav__second">
15+
{{ partial "navbar-breadcrumb.html" . }}
4616
<div class="gd-docs-header-nav__center">
4717
<div id="adds-container" class="gd-docs-header-nav__search"></div>
4818
</div>
@@ -64,4 +34,4 @@
6434
{{ end }}
6535
</div>
6636
{{ end }}
67-
</nav>
37+
</div>

0 commit comments

Comments
 (0)