Skip to content

Accessibility tweaks #4206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Oct 7, 2025
Merged

Accessibility tweaks #4206

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6e4c577
Some quick accessibility wins
katshann-raspberry Oct 6, 2025
f1a6166
Accessibility tweak
katshann-raspberry Oct 6, 2025
12bcd32
Update nav header to h2. CSS tweak to retain size.
katshann-raspberry Oct 7, 2025
93c3553
Further CSS tweaks
katshann-raspberry Oct 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions jekyll-assets/_layouts/boxes.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,12 @@
<div class="toptitle">
<h1><a href="/documentation/"><b>Raspberry Pi</b> Documentation</a></h1>
<div id="search-container">
<div id="docsearch"></div>
<div id="docsearch" role="search"></div>
</div>
{% include tabs.html %}
</div>

<div id="container">
<div id="container" role="main">
<section id="box-content">
{% for item in site.data.index.tabs %}
{% if page.dir == item.path or page.dir == "/" and item.default_tab and item.default_tab == "yes" %}
Expand Down
12 changes: 6 additions & 6 deletions jekyll-assets/_layouts/docs.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,19 +11,19 @@
<div id="toc-container">
<div id="toc-inner">
<div id="docs-header">
<h1 id="docs-header-title">
<h2 id="docs-header-title">
<a href="/documentation">
Documentation
</a>
</h1>
</h2>
<label class="mobile-menu-toggle" for="mobile-toggle">
<div class="mobile-menu-toggle-inner"></div>
</label>
</div>
<div id="docsearch">
<div id="docsearch" role="search">
</div>
</div>
<div class="toc">
<div class="toc" role="navigation">
{% for subdir in site.data.nav %}
<ul class="sectlevel1">
<input class="toc-toggle-box" type="checkbox" id="{{ subdir.path }}" {% if page.url contains subdir.path %}checked{% endif %} />
Expand Down Expand Up @@ -154,14 +154,14 @@ <h1 id="docs-header-title">
</div>
<div id="docs-container">
<div id="main-window">
<section id="content">
<section id="content" role="main">
<h1>{{ page.sub_title | markdownify | remove: '<p>' | remove: '</p>'}}</h1>
{{ content }}
</section>

<div id="on-this-page">
<div id="on-this-page-inner">
<h5>On this page</h5>
<h4>On this page</h4>
<div id="tocbot" class="js-toc"></div>
</div>
</div>
Expand Down
14 changes: 10 additions & 4 deletions jekyll-assets/css/style.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
--accent: #cd2355;
--docsearch-primary-color: var(--accent);
--docsearch-logo-color: var(--red-tint);
--docsearch-muted-color: #333;
--copy-button-bg: #f6f6f6;
--copy-button-text: #444;
--textcolor: black;
Expand Down Expand Up @@ -56,6 +57,7 @@
--accent: #d75a64;
--docsearch-primary-color: var(--accent);
--docsearch-logo-color: var(--red-tint);
--docsearch-muted-color: #333;
--copy-button-bg: #707070;
--copy-button-text: #CCC;
--textcolor: white;
Expand Down Expand Up @@ -102,6 +104,10 @@ h1 {
background: transparent;
}

h2 {
Copy link
Contributor

@mudge mudge Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we want to be a bit defensive here, is it worth scoping to #docs-header or even putting this on #docs-header-title directly?

Copy link
Contributor Author

@katshann-raspberry katshann-raspberry Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've put it on docs-header. I didn't want to be too specific as there is an overriding 1.9em on the more specific item that I want to apply, but my browser was multiplying this with a 1.5em from somewhere. I think I needed the initial to clear that 1.5em.

Copy link
Contributor

@mudge mudge Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, em is a relative unit so it is affected by the parent font size, see https://developer.mozilla.org/en-US/docs/Learn_web_development/Core/Styling_basics/Values_and_units#numbers_lengths_and_percentages. (This is why we prefer using rem for the rest of the site.)

font-size: initial;
}

body, article, div, nav, h1, h2, h3, h4, p {
display: block;
font-family: 'Roboto', sans-serif;
Expand Down Expand Up @@ -754,7 +760,7 @@ h6 .anchor::before {

#content p {
font-size: 0.95em;
font-weight: 300;
font-weight: 400;
line-height: 1.5em;
margin-bottom: 15px;
}
Expand Down Expand Up @@ -926,7 +932,7 @@ span.mlabel {
}

#content td.content {
font-weight: 300;
font-weight: 400;
}

#content p > code,
Expand Down Expand Up @@ -954,7 +960,7 @@ td div.listingblock div.content code {
div.imageblock div.title {
color: var(--subtle-text);
font-style: italic;
font-weight: 300;
font-weight: 400;
font-size: 0.8em;
margin-top: -15px;
margin-bottom: 30px;
Expand Down Expand Up @@ -1212,7 +1218,7 @@ div.memitem {

table.params,
p.returns {
font-weight: 300;
font-weight: 400;
}

table.params td {
Expand Down