Skip to content

Commit c6ca712

Browse files
committed
fix: <main> element on all pages
this is an accessibility best practice, we had <div id=main> but needed to use either <main> or role=main, using the element works
1 parent 606746b commit c6ca712

File tree

6 files changed

+11
-11
lines changed

6 files changed

+11
-11
lines changed

_layouts/color-test.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
{% include header.html %}
22
{% include search_results.html %}
33

4-
<div class="CT-body" id="main">
4+
<main class="CT-body" id="main">
55
{{ content }}
6-
</div>
6+
</main>
77

88
{% include footer.html %}

_layouts/day-menu.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
{% include search_results.html %}
33

44
{% assign menu = site.data[page.menu] %}
5-
<div id="main">{% comment %} #main element needed for skip-menu link but we can't add it inside the loop {% endcomment %}</div>
5+
<main id="main">{% comment %} #main element needed for skip-menu link but we can't add it inside the loop {% endcomment %}</main>
66
{% for meal in menu %}
77
<section>
88
<div class="container">

_layouts/day-schedule.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{% include header.html %}
22
{% include search_results.html %}
33

4-
<div class="container" id="main">
4+
<main class="container" id="main">
55
<div class="row">
66
<div class="col-12">
77
<h1>{% assign day = page.day | slice: -1 %}
@@ -24,6 +24,6 @@ <h1>{% assign day = page.day | slice: -1 %}
2424
{% endif %}
2525
</div>
2626
</div>
27-
</div>
27+
</main>
2828

2929
{% include footer.html %}

_layouts/default.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{% include header.html %}
22
{% include search_results.html %}
3-
<div id="main" class="content">
3+
<main id="main" class="content">
44
{{ content }}
5-
</div>
5+
</main>
66
{% include footer.html %}

_layouts/presentation.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
{% endif %}
1818

1919
<script src="{{ "/assets/js/placeholder.js" | relative_url }}"></script>
20-
<div class="container presentation" id="main">
20+
<main class="container presentation" id="main">
2121
<div class="row">
2222
<div class="col-12 col-sm-8 col-md-9">
2323

@@ -204,6 +204,6 @@ <h2>Location</h2>
204204
{% endif %}
205205
</div>
206206
</div>
207-
</div>
207+
</main>
208208

209209
{% include footer.html %}

_layouts/secondary-nav.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@
3333
</ul>
3434
</nav>
3535
</div>
36-
<div id="main" class="col-12 col-md-9 col-lg-10">
36+
<main id="main" class="col-12 col-md-9 col-lg-10">
3737
{{ content }}
38-
</div>
38+
</main>
3939
</div>
4040
</div>
4141
</section>

0 commit comments

Comments
 (0)