Skip to content

Fix review comments on Week Ahead articles: CSP, duplicate h1, localization, broken translations#1088

Merged
pethers merged 2 commits intonews/week-ahead-2026-03-12-fdefb7d9076a6614from
copilot/sub-pr-1087
Mar 12, 2026
Merged

Fix review comments on Week Ahead articles: CSP, duplicate h1, localization, broken translations#1088
pethers merged 2 commits intonews/week-ahead-2026-03-12-fdefb7d9076a6614from
copilot/sub-pr-1087

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

Addresses 10 review comments from PR #1087 across all 14 week-ahead HTML articles.

Security

  • Removed 'unsafe-inline' from style-src in CSP — external stylesheets don't need it
  • Removed inline onload handler on Google Fonts link; loads directly as rel="stylesheet"

Accessibility / SEO

  • Removed duplicate <h1> (plus stray logo and tagline duplicate) — single primary heading per page

Localization

  • Reading time: Localized twitter:data1 and article-meta for all non-EN files (e.g. 5 min läsning, 阅读时间:5分钟)
  • Policy significance: Replaced 6 distinct English trailing sentences with proper translations across all 14 languages (EU/foreign affairs, fiscal, defence, trade, environment, transport)
  • Broken partial translations: Fixed ~100+ lang="sv" spans where committee names were half-translated (e.g. 交通委员会s sammanträdeTrafikutskottets sammanträde, 质询ssvarInterpellationssvar)
  • Vote entry: Restored Swedish original Votering in lang="sv" spans linking to riksdagen.se

Data quality

  • De-duplicated "calendar, events, calendar, events" in meta keywords
  • Removed truncated Gymnasie- minister target line (incomplete source data)

All 14 files pass HTMLHint validation.


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

…ions, truncated minister name

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Update week ahead parliamentary preview for March 13-20, 2026 Fix review comments on Week Ahead articles: CSP, duplicate h1, localization, broken translations Mar 12, 2026
@pethers pethers marked this pull request as ready for review March 12, 2026 15:11
@pethers pethers merged commit d691913 into news/week-ahead-2026-03-12-fdefb7d9076a6614 Mar 12, 2026
@pethers pethers deleted the copilot/sub-pr-1087 branch March 12, 2026 15:11
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