Skip to content

fix(a11y): localize skip-link/theme-toggle and mark English blocks in 11 multilingual news articles#841

Merged
pethers merged 2 commits intoevening-analysis-2026-03-05-13bbbf9821635a4dfrom
copilot/sub-pr-836
Mar 5, 2026
Merged

fix(a11y): localize skip-link/theme-toggle and mark English blocks in 11 multilingual news articles#841
pethers merged 2 commits intoevening-analysis-2026-03-05-13bbbf9821635a4dfrom
copilot/sub-pr-836

Conversation

Copy link
Contributor

Copilot AI commented Mar 5, 2026

Accessibility issues across the 2026-03-05 evening analysis articles: English UI strings on non-English pages, missing lang="en" on inline English content, and missing dir="ltr" on English blocks inside RTL pages (Arabic, Hebrew).

Skip-link & theme-toggle localization (DA, FI, ES)

UI chrome was using English defaults on Danish, Finnish, and Spanish pages:

<!-- before (all three languages) -->
<a class="skip-link" href="#main-content">Skip to main content</a>
<button aria-label="Switch to dark theme" data-label-dark="Switch to light theme" ...>

<!-- after — example: Danish -->
<a class="skip-link" href="#main-content">Spring til hovedindhold</a>
<button aria-label="Skift til mørkt tema" data-label-dark="Skift til lyst tema" ...>

Spanish also gets aria-label="Versiones de idioma" on the language-switcher <nav>.

lang="en" on inline English content

  • FR, DA, FI, ES, NO — body is largely English on a non-English page; added lang="en" to every English <p>, the English <h2>, and both <ul> lists (context-box + watch-list).
  • DE, ZH, JA — two paragraphs and both <ul> lists were missing the attribute despite the rest of the article already being marked up.
  • HE, AR (RTL) — same two paragraphs and both <ul> lists needed lang="en" dir="ltr" so bidirectional layout and screen-reader language switching behave correctly.
  • KO — context-box and watch-list <ul> were missing lang="en".

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ocks in 11 news articles

- DA: Localize skip-link ('Spring til hovedindhold'), theme toggle to Danish; add lang='en' to all English paragraphs/headings/lists
- FI: Localize skip-link ('Siirry suoraan pääsisältöön'), theme toggle to Finnish; add lang='en' to all English blocks
- ES: Localize skip-link ('Saltar al contenido principal'), theme toggle, and language-switcher nav aria-label to Spanish; add lang='en' to English blocks
- FR: Add lang='en' to all English paragraphs, headings and lists
- NO: Add lang='en' to all English paragraphs, headings and lists
- DE: Add lang='en' to 2 missing paragraphs and context-box/watch-list <ul>
- ZH: Add lang='en' to 2 missing paragraphs and context-box/watch-list <ul>
- JA: Add lang='en' to 2 missing paragraphs and context-box/watch-list <ul>
- HE: Add lang='en' dir='ltr' to 2 missing paragraphs and context-box/watch-list <ul>
- AR: Add lang='en' dir='ltr' to 2 missing paragraphs and context-box/watch-list <ul>
- KO: Add lang='en' to context-box and watch-list <ul>
All 11 files pass HTMLHint validation.

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix details in evening analysis report for March 5, 2026 fix(a11y): localize skip-link/theme-toggle and mark English blocks in 11 multilingual news articles Mar 5, 2026
@pethers pethers marked this pull request as ready for review March 5, 2026 22:59
@pethers pethers merged commit 7731aea into evening-analysis-2026-03-05-13bbbf9821635a4d Mar 5, 2026
@pethers pethers deleted the copilot/sub-pr-836 branch March 5, 2026 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants