Fix dashboard data display: correct CSV column mappings, improve chart data processing, add politician-dashboard i18n by Copilot · Pull Request #1510 · Hack23/riksdagsmonitor

Copilot · 2026-04-01T21:43:38Z

Main page displayed absurd numbers (e.g., "7,832,870 Parliament seats" for S, "Total Seats: 15,311,496 of 349") because coalition-loader.ts read cumulative historical columns instead of current counts. Coalition dashboard charts used artificial clamping and raw values instead of normalized real data. Politician dashboard existed in English only.

Coalition loader — correct CSV column mappings

total_active_parliament → currently_active_members (S: 7832870 → 109, M: 3633784 → 76, etc.)
total_active_government → counted from view_riksdagen_goverment_role_member_sample.csv where active='t' (M=13, KD=6, L=4)
current_party_assignments (517K for S) → total_documents (26K) as the displayed metric
Data loading now tries local /cia-data/ before falling back to remote GitHub
CSV parsing now reuses shared parseCSV from src/browser/shared/data-loader.ts (PapaParse + CSP-safe fallback) — eliminates duplicated parsing logic and ensures consistent RFC4180 quoted field handling across all dashboards

Coalition dashboard — real data processing

Behavioral patterns: Removed Math.max(75, Math.min(100, ...)) clamping that masked real consistency differences between parties
Anomaly data: Computes weighted average deviation per party from classification breakdown CSV instead of using raw avg_rebellions directly as deviation score

Politician dashboard — 13 language variants

Created politician-dashboard_{sv,da,no,fi,de,fr,es,nl,ar,he,ja,ko,zh}.html with translated UI, lang attributes, RTL for ar/he, hreflang tags
Updated vite.config.js with all 14 entry points
Logo links point to language-specific homepages (e.g. index_sv.html)
Accessibility strings (skip-link, theme-toggle aria-label/title/data-labels, logo aria-label) fully translated in all 13 variants
Chart canvas aria-label values (career trajectory, productivity vs influence, experience distribution) translated in all 13 variants
Fixed mojibake encoding issue in politician-dashboard_es.html

Tests

Updated tests/coalition-loader.test.js column references from total_active_parliament → currently_active_members
All 4870 tests pass

…rs instead of total_active_parliament) Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6fdfd87b-2398-4cc4-9b22-7e18fb8468f7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

…ehavioral score clamping Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6fdfd87b-2398-4cc4-9b22-7e18fb8468f7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

…vite config Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6fdfd87b-2398-4cc4-9b22-7e18fb8468f7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

github-actions · 2026-04-01T21:44:13Z

🏷️ Automatic Labeling Summary

This PR has been automatically labeled based on the files changed and PR metadata.

Applied Labels: dashboard,html-css,javascript,performance,testing,size-xl

Label Categories

🗳️ Content: news, dashboard, visualization, intelligence
💻 Technology: html-css, javascript, workflow, security
📊 Data: cia-data, riksdag-data, data-pipeline, schema
🌍 I18n: i18n, translation, rtl
🔒 ISMS: isms, iso-27001, nist-csf, cis-controls
🏗️ Infrastructure: ci-cd, deployment, performance, monitoring
🔄 Quality: testing, accessibility, documentation, refactor
🤖 AI: agent, skill, agentic-workflow

For more information, see .github/labeler.yml.

github-actions · 2026-04-01T21:44:47Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot

Pull request overview

This PR fixes incorrect “absurd” coalition seat/document numbers by updating CSV column mappings and adds more realistic coalition dashboard data processing, while expanding the politician dashboard to 14 language variants with Vite entry points and hreflang support.

Changes:

Correct coalition loader CSV mappings (use currently_active_members, count active government roles, show total_documents), and prefer local /cia-data before remote GitHub.
Update coalition dashboard chart logic to remove consistency clamping and compute per-party anomaly deviation via weighted averages from the classification breakdown.
Add 13 localized politician-dashboard_*.html pages, wire them into vite.config.js, and add hreflang alternates to the English politician dashboard.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 15 comments.

Show a summary per file

File	Description
vite.config.js	Adds 13 new politician dashboard HTML entry points for multi-language builds.
tests/coalition-loader.test.js	Updates test fixtures/assertions to use `currently_active_members` instead of the old column.
src/browser/dashboards/coalition-loader.ts	Fixes seat/document mapping, adds local-first CSV loading, and counts active government roles.
src/browser/dashboards/coalition-dashboard.ts	Removes behavioral clamping and computes anomaly deviation as weighted averages per party.
politician-dashboard.html	Adds hreflang alternates for the localized politician dashboards.
politician-dashboard_sv.html	New Swedish politician dashboard variant.
politician-dashboard_da.html	New Danish politician dashboard variant.
politician-dashboard_no.html	New Norwegian Bokmål politician dashboard variant.
politician-dashboard_fi.html	New Finnish politician dashboard variant.
politician-dashboard_de.html	New German politician dashboard variant.
politician-dashboard_fr.html	New French politician dashboard variant.
politician-dashboard_es.html	New Spanish politician dashboard variant.
politician-dashboard_nl.html	New Dutch politician dashboard variant.
politician-dashboard_ar.html	New Arabic (RTL) politician dashboard variant.
politician-dashboard_he.html	New Hebrew (RTL) politician dashboard variant.
politician-dashboard_ja.html	New Japanese politician dashboard variant.
politician-dashboard_ko.html	New Korean politician dashboard variant.
politician-dashboard_zh.html	New Chinese politician dashboard variant.

Copilot · 2026-04-01T21:50:48Z

politician-dashboard_es.html

+
+<!-- Productivity vs Influence -->
+<div class="chart-card chart-card-spacing">
+<h2>�� Productividad vs. Influencia</h2>


The section heading has mojibake characters ("��") which indicates a text encoding/copy-paste issue. This should be the intended icon/text (it looks like the 💼 emoji was replaced) so it renders correctly for users.

Suggested change

<h2>�� Productividad vs. Influencia</h2>

<h2>💼 Productividad vs. Influencia</h2>

Copilot · 2026-04-01T21:50:49Z

politician-dashboard_sv.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">


This page is Swedish, but the skip-link text and theme-toggle aria-label/title/data-labels are still English. For accessibility (screen readers) and consistency with other localized pages (e.g. index_sv.html), these strings should be translated for sv (and ideally the other politician-dashboard_* localized pages too).

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<a href="#main-content" class="skip-link">Hoppa till huvudinnehåll</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Byt till mörkt tema"

title="Byt till mörkt tema"

data-label-dark="Byt till ljust tema"

data-label-light="Byt till mörkt tema">

Copilot · 2026-04-01T21:50:49Z

politician-dashboard_ar.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Arabic homepage. This breaks language-consistent navigation; update it to point to index_ar.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_ar.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:49Z

politician-dashboard_da.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Danish homepage. This breaks language-consistent navigation; update it to point to index_da.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_da.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:50Z

politician-dashboard_de.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the German homepage. This breaks language-consistent navigation; update it to point to index_de.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_de.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:51Z

politician-dashboard_ko.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Korean homepage. This breaks language-consistent navigation; update it to point to index_ko.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_ko.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:51Z

politician-dashboard_nl.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Dutch homepage. This breaks language-consistent navigation; update it to point to index_nl.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_nl.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:52Z

politician-dashboard_no.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Norwegian (Bokmål) homepage. This breaks language-consistent navigation; update it to point to index_no.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_no.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:52Z

politician-dashboard_sv.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Swedish homepage. This breaks language-consistent navigation; update it to point to index_sv.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_sv.html" aria-label="Riksdagsmonitor Home">

Copilot · 2026-04-01T21:50:52Z

politician-dashboard_zh.html

+<header>
+<nav>
+<div class="logo">
+<a href="index.html" aria-label="Riksdagsmonitor Home">


The logo link points to index.html (English) instead of the Chinese homepage. This breaks language-consistent navigation; update it to point to index_zh.html (matching the rest of the nav).

Suggested change

<a href="index.html" aria-label="Riksdagsmonitor Home">

<a href="index_zh.html" aria-label="Riksdagsmonitor Home">

pethers · 2026-04-01T21:54:16Z

@copilot apply changes based on the comments in this thread

…c homepages, fix mojibake in es variant Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/53b64446-a448-44a3-9186-6b1706116c8e Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot · 2026-04-01T22:00:58Z

@copilot apply changes based on the comments in this thread

Applied all review feedback in 6ed24bc:

Fixed logo links in all 13 politician-dashboard language variants to point to language-specific homepages (e.g., index_sv.html instead of index.html)
Fixed mojibake encoding issue in politician-dashboard_es.html (restored 💼 emoji)

github-actions · 2026-04-01T22:01:42Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot

Pull request overview

Copilot reviewed 18 out of 18 changed files in this pull request and generated 14 comments.

Copilot · 2026-04-01T22:09:29Z

src/browser/dashboards/coalition-loader.ts

+    const csvText = await fetchCSVRetry(CONFIG.governmentRoles);
+    const data = parseCSV(csvText);
+    const counts: Record<string, number> = {};
+    data.forEach((row) => {
+      if (row.active === 't' && row.party) {
+        const party = row.party as string;
+        counts[party] = (counts[party] || 0) + 1;


loadGovernmentRoles() relies on parseCSV(), but parseCSV() splits on commas and skips rows when the column count mismatches. The government role member CSV contains quoted fields with commas (e.g. "Gymnasie-, högskole- och forskningsminister"), so active rows will be dropped and governmentCounts will be undercounted. Use a CSV parser that handles RFC4180 quoting (e.g. PapaParse like other dashboards) or reuse the robust line parser from scripts/government-role-validator.ts for this loader.

Copilot · 2026-04-01T22:09:29Z

politician-dashboard_sv.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">
+  <span class="theme-icon" aria-hidden="true">🌙</span>
+</button>
+
+<header>
+<nav>
+<div class="logo">
+<a href="index_sv.html" aria-label="Riksdagsmonitor Home">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). For consistency with index_sv.html and to keep screen-reader output in Swedish, translate these attributes/text in this variant (and keep the semantics the same).

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_sv.html" aria-label="Riksdagsmonitor Home">

<a href="#main-content" class="skip-link">Hoppa till huvudinnehåll</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Byt till mörkt tema"

title="Byt till mörkt tema"

data-label-dark="Byt till ljust tema"

data-label-light="Byt till mörkt tema">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_sv.html" aria-label="Riksdagsmonitor hem">

Copilot · 2026-04-01T22:09:30Z

politician-dashboard_da.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">
+  <span class="theme-icon" aria-hidden="true">🌙</span>
+</button>
+
+<header>
+<nav>
+<div class="logo">
+<a href="index_da.html" aria-label="Riksdagsmonitor Home">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Danish (consistent with index_da.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_da.html" aria-label="Riksdagsmonitor Home">

<a href="#main-content" class="skip-link">Spring til hovedindhold</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Skift til mørkt tema"

title="Skift til mørkt tema"

data-label-dark="Skift til lyst tema"

data-label-light="Skift til mørkt tema">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_da.html" aria-label="Riksdagsmonitor forside">

Copilot · 2026-04-01T22:09:30Z

politician-dashboard_no.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">
+  <span class="theme-icon" aria-hidden="true">🌙</span>
+</button>
+
+<header>
+<nav>
+<div class="logo">
+<a href="index_no.html" aria-label="Riksdagsmonitor Home">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Bokmål (consistent with index_no.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_no.html" aria-label="Riksdagsmonitor Home">

<a href="#main-content" class="skip-link">Hopp til hovedinnhold</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Bytt til mørkt tema"

title="Bytt til mørkt tema"

data-label-dark="Bytt til lyst tema"

data-label-light="Bytt til mørkt tema">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_no.html" aria-label="Riksdagsmonitor forside">

Copilot · 2026-04-01T22:09:30Z

politician-dashboard_fi.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">
+  <span class="theme-icon" aria-hidden="true">🌙</span>
+</button>
+
+<header>
+<nav>
+<div class="logo">
+<a href="index_fi.html" aria-label="Riksdagsmonitor Home">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Finnish (consistent with index_fi.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_fi.html" aria-label="Riksdagsmonitor Home">

<a href="#main-content" class="skip-link">Siirry suoraan sisältöön</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Vaihda tummaan teemaan"

title="Vaihda tummaan teemaan"

data-label-dark="Vaihda vaaleaan teemaan"

data-label-light="Vaihda tummaan teemaan">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_fi.html" aria-label="Riksdagsmonitor etusivu">

Copilot · 2026-04-01T22:09:31Z

politician-dashboard_ar.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Arabic (consistent with index_ar.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<a href="#main-content" class="skip-link">تخطي إلى المحتوى الرئيسي</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="التبديل إلى الوضع الداكن"

title="التبديل إلى الوضع الداكن"

data-label-dark="التبديل إلى الوضع الفاتح"

data-label-light="التبديل إلى الوضع الداكن">

Copilot · 2026-04-01T22:09:32Z

politician-dashboard_he.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Hebrew (consistent with index_he.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<a href="#main-content" class="skip-link">דלג לתוכן הראשי</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="החלפה למצב כהה"

title="החלפה למצב כהה"

data-label-dark="החלפה למצב בהיר"

data-label-light="החלפה למצב כהה">

Copilot · 2026-04-01T22:09:32Z

politician-dashboard_ja.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Japanese (consistent with index_ja.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<a href="#main-content" class="skip-link">メインコンテンツへスキップ</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="ダークテーマに切り替える"

title="ダークテーマに切り替える"

data-label-dark="ライトテーマに切り替える"

data-label-light="ダークテーマに切り替える">

Copilot · 2026-04-01T22:09:32Z

politician-dashboard_ko.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Korean (consistent with index_ko.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<a href="#main-content" class="skip-link">본문으로 바로 가기</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="다크 테마로 전환"

title="다크 테마로 전환"

data-label-dark="라이트 테마로 전환"

data-label-light="다크 테마로 전환">

Copilot · 2026-04-01T22:09:33Z

politician-dashboard_zh.html

+<a href="#main-content" class="skip-link">Skip to main content</a>
+<button id="theme-toggle" class="theme-toggle-btn" type="button"
+        aria-pressed="false"
+        aria-label="Switch to dark theme"
+        title="Switch to dark theme"
+        data-label-dark="Switch to light theme"
+        data-label-light="Switch to dark theme">
+  <span class="theme-icon" aria-hidden="true">🌙</span>
+</button>
+
+<header>
+<nav>
+<div class="logo">
+<a href="index_zh.html" aria-label="Riksdagsmonitor Home">


This localized dashboard still uses English accessibility strings (skip link text and theme-toggle aria-label/title/data-label-*). Translate these attributes/text to Chinese (consistent with index_zh.html) so screen readers and tooltips are in the page language.

Suggested change

<a href="#main-content" class="skip-link">Skip to main content</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="Switch to dark theme"

title="Switch to dark theme"

data-label-dark="Switch to light theme"

data-label-light="Switch to dark theme">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_zh.html" aria-label="Riksdagsmonitor Home">

<a href="#main-content" class="skip-link">跳转到主要内容</a>

<button id="theme-toggle" class="theme-toggle-btn" type="button"

aria-pressed="false"

aria-label="切换到深色主题"

title="切换到深色主题"

data-label-dark="切换到浅色主题"

data-label-light="切换到深色主题">

<span class="theme-icon" aria-hidden="true">🌙</span>

</button>

<header>

<nav>

<div class="logo">

<a href="index_zh.html" aria-label="Riksdagsmonitor 首页">

pethers · 2026-04-01T22:12:21Z

@copilot apply changes based on the comments in this thread

… fix coalition-loader CSV parsing with PapaParse Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/c8c70386-51fc-4a74-a79b-e64167b7e724 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot · 2026-04-01T22:25:56Z

@copilot apply changes based on the comments in this thread

Applied all review feedback in bdbe9e0:

Translated skip-link text, theme-toggle aria-label/title/data-label-*, and logo aria-label in all 13 localized politician-dashboard files (sv, da, no, fi, de, fr, es, nl, ar, he, ja, ko, zh)
Fixed parseCSV() in coalition-loader.ts to use PapaParse (with naive fallback) for proper RFC4180 quoted field handling — government role titles with commas (e.g. "Gymnasie-, högskole- och forskningsminister") are now parsed correctly

github-actions · 2026-04-01T22:26:26Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot

Pull request overview

Copilot reviewed 18 out of 18 changed files in this pull request and generated 14 comments.

Comments suppressed due to low confidence (1)

tests/coalition-loader.test.js:33

The CSV parsing tests are currently “simulating parseCSV logic” with a naive split(','), so they won’t catch regressions in the real implementation (now using PapaParse for RFC4180 quoted fields). Add an assertion covering a quoted field containing a comma (and ideally exercise the actual coalition-loader parsing path) to validate the bugfix described in the PR.

    it('should parse valid CSV with headers and data rows', () => {
      const csvText = `party,active,currently_active_members
M,t,68
SD,t,73
S,t,106`;

      // Simulate parseCSV logic
      const lines = csvText.trim().split('\n');
      const headers = lines[0].split(',').map(h => h.trim());
      const data = [];

Copilot · 2026-04-01T22:37:15Z

src/browser/dashboards/coalition-loader.ts

 function parseCSV(csvText: string): CSVRow[] {
+  const Papa = (globalThis as Record<string, unknown>).Papa as {
+    parse: (input: string, config: Record<string, unknown>) => { data: CSVRow[] };
+  } | undefined;
+
+  if (Papa) {
+    const parsed = Papa.parse(csvText, { header: true, skipEmptyLines: true });
+    return parsed.data;


parseCSV() duplicates CSV parsing logic that already exists in src/browser/shared/data-loader.ts (PapaParse + CSP-safe fallback). Consider reusing the shared parser/loader here to avoid divergence (e.g., quoted fields, trimming rules) across dashboards and reduce maintenance overhead.

Copilot · 2026-04-01T22:37:15Z

politician-dashboard_sv.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 Karriärbaneanalys</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Swedish.

Copilot · 2026-04-01T22:37:15Z

politician-dashboard_da.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 Karriereforløbsanalyse</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Danish.

Copilot · 2026-04-01T22:37:16Z

politician-dashboard_no.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 Karriereløpsanalyse</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Norwegian Bokmål.

Copilot · 2026-04-01T22:37:16Z

politician-dashboard_fi.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 Urapolkuanalyysi</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Finnish.

Copilot · 2026-04-01T22:37:17Z

politician-dashboard_ar.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 تحليل المسار المهني</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Arabic.

Copilot · 2026-04-01T22:37:18Z

politician-dashboard_he.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 ניתוח מסלול קריירה</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Hebrew.

Copilot · 2026-04-01T22:37:18Z

politician-dashboard_ja.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 キャリア軌跡分析</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Japanese.

Copilot · 2026-04-01T22:37:18Z

politician-dashboard_ko.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 경력 궤적 분석</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Korean.

Copilot · 2026-04-01T22:37:19Z

politician-dashboard_zh.html

+<div class="chart-card chart-card-spacing">
+<h2>📈 职业轨迹分析</h2>
+<div class="chart-wrapper">
+<canvas id="career-trajectory-chart" role="img" aria-label="Career trajectory timeline chart showing MP career progression"></canvas>


This page’s chart canvas aria-label values are still in English (and the other chart aria-labels in this file appear to be as well). For accessibility and consistency with the translated UI, translate these aria-label strings to Chinese.

pethers · 2026-04-01T22:46:31Z

@copilot apply changes based on the comments in this thread

…s in all 13 politician-dashboard variants Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/5adeca4e-9e3c-465c-ac64-91996b4f7e06 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

github-actions · 2026-04-01T22:59:15Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot · 2026-04-01T23:01:09Z

@copilot apply changes based on the comments in this thread

Applied all review feedback in bc787bb:

Replaced duplicated parseCSV() in coalition-loader.ts with import { parseCSV } from '../shared/data-loader.js' — eliminates code divergence across dashboards
Translated all 3 chart canvas aria-label values in all 13 politician-dashboard language files (sv, da, no, fi, de, fr, es, nl, ar, he, ja, ko, zh)

All 4870 tests pass.

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI and others added 3 commits April 1, 2026 21:17

Fix coalition-loader: use correct CSV columns (currently_active_membe…

3d80970

…rs instead of total_active_parliament) Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6fdfd87b-2398-4cc4-9b22-7e18fb8468f7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Fix coalition-dashboard: improve anomaly data processing and remove b…

5005a1e

…ehavioral score clamping Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6fdfd87b-2398-4cc4-9b22-7e18fb8468f7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Add politician-dashboard language variants (13 languages) and update …

9ca4786

…vite config Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6fdfd87b-2398-4cc4-9b22-7e18fb8468f7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot AI assigned Copilot and pethers Apr 1, 2026

Copilot created this pull request from a session on behalf of pethers April 1, 2026 21:43 View session

github-actions bot added dashboard Interactive dashboards (Chart.js/D3.js) html-css HTML/CSS changes javascript JavaScript code changes performance Performance optimization testing Test coverage size-xl Extra large change (> 1000 lines) labels Apr 1, 2026

pethers marked this pull request as ready for review April 1, 2026 21:44

Copilot AI review requested due to automatic review settings April 1, 2026 21:44

Copilot started reviewing on behalf of pethers April 1, 2026 21:46 View session

Copilot AI reviewed Apr 1, 2026

View reviewed changes

Copilot started work on behalf of pethers April 1, 2026 21:54 View session

Fix politician-dashboard i18n: correct logo links to language-specifi…

6ed24bc

…c homepages, fix mojibake in es variant Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/53b64446-a448-44a3-9186-6b1706116c8e Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot finished work on behalf of pethers April 1, 2026 22:01

Copilot AI requested a review from pethers April 1, 2026 22:01

pethers requested a review from Copilot April 1, 2026 22:03

Copilot started reviewing on behalf of pethers April 1, 2026 22:04 View session

Copilot AI reviewed Apr 1, 2026

View reviewed changes

Copilot started work on behalf of pethers April 1, 2026 22:12 View session

Copilot finished work on behalf of pethers April 1, 2026 22:27

pethers requested a review from Copilot April 1, 2026 22:31

Copilot started reviewing on behalf of pethers April 1, 2026 22:32 View session

Copilot AI reviewed Apr 1, 2026

View reviewed changes

Copilot started work on behalf of pethers April 1, 2026 22:46 View session

Copilot finished work on behalf of pethers April 1, 2026 23:04

pethers requested a review from Copilot April 1, 2026 23:06

Copilot AI reviewed Apr 1, 2026

View reviewed changes

pethers merged commit 3fbfaf8 into main Apr 1, 2026
26 checks passed

pethers deleted the copilot/improve-dashboards-for-csv-imports branch April 1, 2026 23:15

	<h2>�� Productividad vs. Influencia</h2>
	<h2>💼 Productividad vs. Influencia</h2>

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_ar.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_da.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_de.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_ko.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_nl.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_no.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_sv.html" aria-label="Riksdagsmonitor Home">

	<a href="index.html" aria-label="Riksdagsmonitor Home">
	<a href="index_zh.html" aria-label="Riksdagsmonitor Home">

Conversation

Copilot AI commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coalition loader — correct CSV column mappings

Coalition dashboard — real data processing

Politician dashboard — 13 language variants

Tests

Uh oh!

github-actions bot commented Apr 1, 2026

🏷️ Automatic Labeling Summary

Label Categories

Uh oh!

github-actions bot commented Apr 1, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

pethers commented Apr 1, 2026

Uh oh!

Copilot AI commented Apr 1, 2026

Uh oh!

github-actions bot commented Apr 1, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 1, 2026

Copilot AI commented Apr 1, 2026 •

edited

Loading