Skip to content

Fix CSP, accessibility, and standards compliance in 2026-03-13 committee-reports articles#1106

Merged
pethers merged 2 commits intomain-62a4e8d6e78c93d5from
copilot/sub-pr-1103
Mar 13, 2026
Merged

Fix CSP, accessibility, and standards compliance in 2026-03-13 committee-reports articles#1106
pethers merged 2 commits intomain-62a4e8d6e78c93d5from
copilot/sub-pr-1103

Conversation

Copy link
Contributor

Copilot AI commented Mar 13, 2026

Addresses all 16 review comments from automated PR review on #1103, aligning both EN and SV article pages with established patterns in other news articles.

Security (CSP hardening)

  • Remove 'unsafe-inline' from both script-src and style-src directives
  • Replace inline theme-init IIFE with external ../js/theme-init.js
  • Remove Google Fonts onload="this.media='all'" inline handler; use standard <link rel="stylesheet">

Accessibility / SEO

  • Remove duplicate <h1> and tagline block — single <h1> in hero-header-text
  • Fix truncated BreadcrumbList JSON-LD name at position 3 to use full article title
  • Fix <time datetime> from YYYY-MM-DD HH:MM:SSYYYY-MM-DDTHH:MM:SS (valid ISO-8601)

Correctness

  • Replace ../scripts/back-to-top.ts with compiled ../js/back-to-top.js (browsers can't execute TS)
  • Deduplicate keywords in both <meta> tag and Schema.org structured data

All changes mirror the pattern used by the reference 2026-03-12-committee-reports-en.html. HTMLHint: 0 errors.


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

…cles

- Remove 'unsafe-inline' from CSP script-src and style-src (EN + SV)
- Replace inline theme init script with external ../js/theme-init.js (EN + SV)
- Remove Google Fonts inline onload handler, use regular stylesheet link (EN + SV)
- Remove duplicate h1 and tagline block, keep single h1 in hero-header-text (EN + SV)
- Fix truncated BreadcrumbList name to use full article title (EN + SV)
- Fix time datetime format from space-separated to ISO-8601 with T separator (EN + SV)
- Replace ../scripts/back-to-top.ts with compiled ../js/back-to-top.js (EN + SV)
- Deduplicate keywords in meta content and Schema.org keywords (EN + SV)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Add 2026-03-13 committee-reports articles (EN, SV) Fix CSP, accessibility, and standards compliance in 2026-03-13 committee-reports articles Mar 13, 2026
Copilot AI requested a review from pethers March 13, 2026 06:24
@pethers pethers marked this pull request as ready for review March 13, 2026 06:39
@pethers pethers merged commit d9d74b9 into main-62a4e8d6e78c93d5 Mar 13, 2026
@pethers pethers deleted the copilot/sub-pr-1103 branch March 13, 2026 06:39
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