Skip to content

Sync CSS and require readability pages in sitemap check #180

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
lukefretwell merged 1 commit into from
Oct 25, 2026
Merged

Sync CSS and require readability pages in sitemap check #180

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
29 changes: 23 additions & 6 deletions public/css/scangov.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -498,15 +498,19 @@ button:not(.card *) {
}
}

/* Skip link — visible on focus for keyboard users (WCAG 2.4.1) */
/* Skip link — visible on focus for keyboard users (WCAG 2.4.1). Hidden by
Bootstrap's .visually-hidden-focusable until it takes focus; never pair it
with .visually-hidden, which stays clipped on focus. */
.skip-link {
position: absolute;
top: -100%;
left: 0;
}

.skip-link:focus {
top: 0;
left: 0;
z-index: 1100;
padding: 0.5rem 1rem;
background-color: var(--bs-body-bg);
color: var(--bs-link-color);
border: 1px solid var(--bs-border-color);
border-top: 0;
}

/* Navbar */
Expand Down Expand Up @@ -1933,6 +1937,19 @@ iframe {
border: 1px solid var(--bs-border-color) !important;
}

/* Video embeds (lite-youtube-embed, see the youtube shortcode). The
component caps itself at 720px; fill the column like the old iframe did. */
lite-youtube {
max-width: 100%;
}

/* The play button covers the whole poster and the component clips paint at
its edge, so the site's offset focus ring would be cut off. Draw it inset. */
lite-youtube > .lyt-playbtn:focus-visible {
outline: 3px solid #fff;
outline-offset: -5px;
}

/* News posts */

.post-content h2 {
Expand Down
29 changes: 23 additions & 6 deletions scripts/bundle.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -713,15 +713,19 @@ button:not(.card *) {
}
}

/* Skip link — visible on focus for keyboard users (WCAG 2.4.1) */
/* Skip link — visible on focus for keyboard users (WCAG 2.4.1). Hidden by
Bootstrap's .visually-hidden-focusable until it takes focus; never pair it
with .visually-hidden, which stays clipped on focus. */
.skip-link {
position: absolute;
top: -100%;
left: 0;
}

.skip-link:focus {
top: 0;
left: 0;
z-index: 1100;
padding: 0.5rem 1rem;
background-color: var(--bs-body-bg);
color: var(--bs-link-color);
border: 1px solid var(--bs-border-color);
border-top: 0;
}

/* Navbar */
Expand Down Expand Up @@ -2148,6 +2152,19 @@ iframe {
border: 1px solid var(--bs-border-color) !important;
}

/* Video embeds (lite-youtube-embed, see the youtube shortcode). The
component caps itself at 720px; fill the column like the old iframe did. */
lite-youtube {
max-width: 100%;
}

/* The play button covers the whole poster and the component clips paint at
its edge, so the site's offset focus ring would be cut off. Draw it inset. */
lite-youtube > .lyt-playbtn:focus-visible {
outline: 3px solid #fff;
outline-offset: -5px;
}

/* News posts */

.post-content h2 {
Expand Down
16 changes: 16 additions & 0 deletions scripts/check-sitemap-completeness.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -177,6 +177,22 @@ const sitemapPaths = new Set(
[...sitemapXml.matchAll(/<loc>(.*?)<\/loc>/g)].map((m) => new URL(m[1]).pathname)
);

// The readability pages must stay in the sitemap. The checks below only catch a
// built page that is missing, so a removed or renamed page would go unnoticed.
const REQUIRED_READABILITY_PATHS = [
'/readability/',
'/reading-level/',
'/automated-readability-index/',
'/flesch-kincaid/',
'/guidance/readability/',
];
const requiredMissing = REQUIRED_READABILITY_PATHS.filter((p) => !sitemapPaths.has(p));
if (requiredMissing.length > 0) {
console.error(`check-sitemap-completeness: ${requiredMissing.length} readability page(s) missing from sitemap.xml:\n`);
for (const p of requiredMissing) console.error(` ${p}`);
process.exit(1);
}

let missing, total;
try {
({ missing, total } = await checkViaNdjson(siteRoot, sitemapPaths));
Expand Down
Loading