From 6ac4df4c1276c785b93384096d22bc0c2128c057 Mon Sep 17 00:00:00 2001 From: Sakari Malkki Date: Wed, 5 Nov 2025 15:59:28 +0200 Subject: [PATCH 1/5] Replaces credits field with more responsive caption field --- src/core/chartOptions/chartOptions.ts | 29 +---------- .../conversion/pxGrafDataConverter.test.ts | 48 +++++++++---------- src/core/highcharts/themes.ts | 3 +- .../chartstories/linechart.stories.tsx | 8 ++++ 4 files changed, 36 insertions(+), 52 deletions(-) diff --git a/src/core/chartOptions/chartOptions.ts b/src/core/chartOptions/chartOptions.ts index ec8bf13..7c81414 100644 --- a/src/core/chartOptions/chartOptions.ts +++ b/src/core/chartOptions/chartOptions.ts @@ -12,39 +12,13 @@ export const commonChartOptions = (view: View, locale: string, options?: IChartO const sourceText = Translations.source[locale]; let creditsText = `${sourceText}: ${view.sources.map(s => s[locale]).join(', ')}`; - let creditsConfig: any = { - enabled: true, - text: creditsText - }; - const hasLastUpdated = options?.showLastUpdated && view.lastUpdated; if (hasLastUpdated) { const lastUpdatedText = getFormattedLastUpdatedText(view.lastUpdated, locale); if (lastUpdatedText) { creditsText = `${lastUpdatedText}
${sourceText}: ${view.sources.map(s => s[locale]).join(', ')}`; - creditsConfig = { - enabled: true, - text: creditsText, - useHTML: true, - position: { - x: 5, - y: -30 // More space needed for two-line credits - } - }; } - } else { - // Single line credits, less spacing needed - creditsConfig = { - enabled: true, - text: creditsText, - position: { - align: 'left', - verticalAlign: 'bottom', - x: 5, - y: -10 // Less space needed for single line - } - }; } return { @@ -58,7 +32,8 @@ export const commonChartOptions = (view: View, locale: string, options?: IChartO }, title: { text: showTitles ? view.header[locale] : undefined }, subtitle: { text: view.subheaderValues.map(sv => sv[locale]).join(' | ') }, - credits: creditsConfig, + credits: { enabled: false }, + caption: { text: creditsText }, tooltip: { formatter: getToolTipFormatterFunction(view, locale) }, diff --git a/src/core/conversion/pxGrafDataConverter.test.ts b/src/core/conversion/pxGrafDataConverter.test.ts index a3db30b..cbef156 100644 --- a/src/core/conversion/pxGrafDataConverter.test.ts +++ b/src/core/conversion/pxGrafDataConverter.test.ts @@ -207,7 +207,7 @@ describe('PxGrafDataConverter tests, VerticalBarChart', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('column'); expect(result.title?.text).toBe('Lukumäärä 2015Q1 muuttujina Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('2015Q1 | Pääkaupunkiseutu (PKS) | Vapaarahoitteinen'); }); @@ -217,7 +217,7 @@ describe('PxGrafDataConverter tests, VerticalBarChart', () => { const result = convertPxGraphDataToChartOptions('sv', mockView); expect(result.chart?.type).toBe('column'); expect(result.title?.text).toBe('Antal 2015Q1 efter Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('2015Q1 | Huvudstadsregionen | Fri finansierad'); }); }); @@ -229,7 +229,7 @@ describe('PxGrafDataConverter tests, BasicHorizontalBarChart', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('bar'); expect(result.title?.text).toBe('Tiedot 2022Q4 muuttujina Tiedot, Alue, Huoneluku'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('Yksiöt | Lukumäärä'); }); @@ -239,7 +239,7 @@ describe('PxGrafDataConverter tests, BasicHorizontalBarChart', () => { const result = convertPxGraphDataToChartOptions('en', mockView); expect(result.chart?.type).toBe('bar'); expect(result.title?.text).toBe('Information 2022Q4 by Information, Region, Number of rooms'); - expect(result.credits?.text).toBe('Source: PxVisualizer-en'); + expect(result.caption?.text).toBe('Source: PxVisualizer-en'); expect(result.subtitle?.text).toBe('One-room flat | Number'); }); }); @@ -251,7 +251,7 @@ describe('PxGrafDataConverter tests, GroupHorizontalBarChart', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('bar'); expect(result.title?.text).toBe('Tiedot 2015Q1-2015Q2 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('Pääkaupunkiseutu (PKS) | Yksiöt | Lukumäärä'); }); @@ -261,7 +261,7 @@ describe('PxGrafDataConverter tests, GroupHorizontalBarChart', () => { const result = convertPxGraphDataToChartOptions('en', mockView); expect(result.chart?.type).toBe('bar'); expect(result.title?.text).toBe('Information 2015Q1-2015Q2 by Information, Region, Number of rooms, Type of funding'); - expect(result.credits?.text).toBe('Source: PxVisualizer-en'); + expect(result.caption?.text).toBe('Source: PxVisualizer-en'); expect(result.subtitle?.text).toBe('Greater Helsinki | One-room flat | Number'); }); }); @@ -273,7 +273,7 @@ describe('PxGrafDataConverter tests, PieBarChart', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('pie'); expect(result.title?.text).toBe('Lukumäärä, Yksiöt 2022Q4 muuttujana Alue'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe(''); }); @@ -283,7 +283,7 @@ describe('PxGrafDataConverter tests, PieBarChart', () => { const result = convertPxGraphDataToChartOptions('en', mockView); expect(result.chart?.type).toBe('pie'); expect(result.title?.text).toBe('Number, One-room flat 2022Q4 by Region'); - expect(result.credits?.text).toBe('Source: PxVisualizer-en'); + expect(result.caption?.text).toBe('Source: PxVisualizer-en'); expect(result.subtitle?.text).toBe(''); }); }); @@ -295,7 +295,7 @@ describe('PxGrafDataConverter tests, GroupVerticalBarChart', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('column'); expect(result.title?.text).toBe('Tiedot 2015Q1-2015Q2 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('Pääkaupunkiseutu (PKS) | Yksiöt | Lukumäärä'); }); @@ -305,7 +305,7 @@ describe('PxGrafDataConverter tests, GroupVerticalBarChart', () => { const result = convertPxGraphDataToChartOptions('sv', mockView); expect(result.chart?.type).toBe('column'); expect(result.title?.text).toBe('Uppgifter 2015Q1-2015Q2 efter Uppgifter, Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('Huvudstadsregionen | Enrumslägenhet | Antal'); }); }); @@ -318,7 +318,7 @@ describe('PxGrafDataConverter tests, StackedVerticalBarChart', () => { expect(result.chart?.type).toBe('column'); expect(result.plotOptions?.column?.stacking).toBe('normal'); expect(result.title?.text).toBe('Tiedot 2015Q1-2015Q2 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('Pääkaupunkiseutu (PKS) | Yksiöt | Lukumäärä'); }); @@ -329,7 +329,7 @@ describe('PxGrafDataConverter tests, StackedVerticalBarChart', () => { expect(result.chart?.type).toBe('column'); expect(result.plotOptions?.column?.stacking).toBe('normal'); expect(result.title?.text).toBe('Uppgifter 2015Q1-2015Q2 efter Uppgifter, Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('Huvudstadsregionen | Enrumslägenhet | Antal'); }); }); @@ -342,7 +342,7 @@ describe('PxGrafDataConverter tests, StackedHorizontalBarChart', () => { expect(result.chart?.type).toBe('bar'); expect(result.plotOptions?.bar?.stacking).toBe('normal'); expect(result.title?.text).toBe('Tiedot 2015Q1 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('2015Q1 | Pääkaupunkiseutu (PKS) | Yksiöt'); }); @@ -353,7 +353,7 @@ describe('PxGrafDataConverter tests, StackedHorizontalBarChart', () => { expect(result.chart?.type).toBe('bar'); expect(result.plotOptions?.bar?.stacking).toBe('normal'); expect(result.title?.text).toBe('Uppgifter 2015Q1 efter Uppgifter, Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('2015Q1 | Huvudstadsregionen | Enrumslägenhet'); }); }); @@ -371,7 +371,7 @@ describe('PxGrafDataConverter tests, PercentVerticalBarChart', () => { expect(result.chart?.type).toBe('column'); expect(result.plotOptions?.column?.stacking).toBe('percent'); expect(result.title?.text).toBe('Lukumäärä 2021Q4 muuttujina Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('2022Q1 | Yksiöt'); }); @@ -387,7 +387,7 @@ describe('PxGrafDataConverter tests, PercentVerticalBarChart', () => { expect(result.chart?.type).toBe('column'); expect(result.plotOptions?.column?.stacking).toBe('percent'); expect(result.title?.text).toBe('Antal 2021Q4 efter Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('2022Q1 | Enrumslägenhet'); }); }); @@ -405,7 +405,7 @@ describe('PxGrafDataConverter tests, PercentHorizontalBarChart', () => { expect(result.chart?.type).toBe('bar'); expect(result.plotOptions?.bar?.stacking).toBe('percent'); expect(result.title?.text).toBe('Lukumäärä 2022Q4 muuttujina Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('Kaksiot'); }); @@ -421,7 +421,7 @@ describe('PxGrafDataConverter tests, PercentHorizontalBarChart', () => { expect(result.chart?.type).toBe('bar'); expect(result.plotOptions?.bar?.stacking).toBe('percent'); expect(result.title?.text).toBe('Antal 2022Q4 efter Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('Tvårumslägenhet'); }); }); @@ -434,7 +434,7 @@ describe('PxGrafDataConverter tests, PyramidChart', () => { expect(result.chart?.type).toBe('bar'); expect(result.plotOptions?.series?.stacking).toBe('normal'); expect(result.title?.text).toBe('Väestö 31.12. 2020 muuttujina Alue, Ikä, Sukupuoli'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('KOKO MAA | 2020'); }); @@ -445,7 +445,7 @@ describe('PxGrafDataConverter tests, PyramidChart', () => { expect(result.chart?.type).toBe('bar'); expect(result.plotOptions?.series?.stacking).toBe('normal'); expect(result.title?.text).toBe('Befolkning 31.12. 2020 efter Område, Ålder, Kön'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('HELA LANDET | 2020'); }); }); @@ -457,7 +457,7 @@ describe('PxGrafDataConverter tests, ScatterPlot', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('scatter'); expect(result.title?.text).toBe('Tiedot 2015Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe('Pääkaupunkiseutu (PKS) | Yksiöt | Vapaarahoitteinen'); }); @@ -467,7 +467,7 @@ describe('PxGrafDataConverter tests, ScatterPlot', () => { const result = convertPxGraphDataToChartOptions('sv', mockView); expect(result.chart?.type).toBe('scatter'); expect(result.title?.text).toBe('Uppgifter 2015Q1-2022Q4 efter Uppgifter, Område, Antal rum, Finansieringssätt'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe('Huvudstadsregionen | Enrumslägenhet | Fri finansierad'); }); }); @@ -479,7 +479,7 @@ describe('PxGrafDataConverter tests, LineChart', () => { const result = convertPxGraphDataToChartOptions('fi', mockView); expect(result.chart?.type).toBe('line'); expect(result.title?.text).toBe('Neliövuokra (eur/m2), Yksiöt, Vapaarahoitteinen 2015Q1-2022Q4 muuttujana Alue'); - expect(result.credits?.text).toBe('Lähde: PxVisualizer-fi'); + expect(result.caption?.text).toBe('Lähde: PxVisualizer-fi'); expect(result.subtitle?.text).toBe(''); }); @@ -489,7 +489,7 @@ describe('PxGrafDataConverter tests, LineChart', () => { const result = convertPxGraphDataToChartOptions('sv', mockView); expect(result.chart?.type).toBe('line'); expect(result.title?.text).toBe('Kvadratmeterspris (eur/m2), Enrumslägenhet, Fri finansierad 2015Q1-2022Q4 efter Område'); - expect(result.credits?.text).toBe('Källa: PxVisualizer-sv'); + expect(result.caption?.text).toBe('Källa: PxVisualizer-sv'); expect(result.subtitle?.text).toBe(''); }); }); \ No newline at end of file diff --git a/src/core/highcharts/themes.ts b/src/core/highcharts/themes.ts index 56aa077..846c768 100644 --- a/src/core/highcharts/themes.ts +++ b/src/core/highcharts/themes.ts @@ -42,6 +42,7 @@ export const defaultTheme: (locale: string, fontFamily?: string) => Highcharts.O accessibility: { thousandsSep: Translations.thousandsSepAccessibility[locale], credits: Translations.credits[locale], + caption: Translations.credits[locale], chartTypes: { barMultiple: Translations.barMultiple[locale], barSingle: Translations.barSingle[locale], @@ -154,7 +155,7 @@ export const defaultTheme: (locale: string, fontFamily?: string) => Highcharts.O }, align: 'left', }, - credits: { + caption: { position: { align: "left", x: 5 diff --git a/src/stories/chartstories/linechart.stories.tsx b/src/stories/chartstories/linechart.stories.tsx index 428c19e..d185e7c 100644 --- a/src/stories/chartstories/linechart.stories.tsx +++ b/src/stories/chartstories/linechart.stories.tsx @@ -49,6 +49,14 @@ export const CombinationSeries = { args: LINE_CHART_WITH_COMBINATION_SERIES, } satisfies StoryObj; +export const CombinationSeriesWithLastUpdated = { + name: 'Combination series (with last updated)', + args: { + ...LINE_CHART_WITH_COMBINATION_SERIES, + showLastUpdated: true + } +} satisfies StoryObj; + export const NegativeValues = { name: 'Negative values', args: LINE_CHART_WITH_NEGATIVE_VALUES, From 1cda828e11f77b2508fb23b6e2543fbe34cc951b Mon Sep 17 00:00:00 2001 From: Sakari Malkki Date: Wed, 5 Nov 2025 16:29:55 +0200 Subject: [PATCH 2/5] Move html table title out of the table caption to a separate element --- .../__snapshots__/htmlTable.test.ts.snap | 100 ++++++++++-------- src/core/tables/htmlTable.ts | 16 ++- .../chart/__snapshots__/chart.test.tsx.snap | 96 ++++++++++------- .../__snapshots__/tableView.test.tsx.snap | 12 ++- .../components/globalStyle/globalStyle.ts | 2 +- 5 files changed, 133 insertions(+), 93 deletions(-) diff --git a/src/core/tables/__snapshots__/htmlTable.test.ts.snap b/src/core/tables/__snapshots__/htmlTable.test.ts.snap index 1bf7786..0bb1359 100644 --- a/src/core/tables/__snapshots__/htmlTable.test.ts.snap +++ b/src/core/tables/__snapshots__/htmlTable.test.ts.snap @@ -4,14 +4,16 @@ exports[`Html table render tests should match snapshot: Table with column variab " +  + Adoptiot 1987-1989 muuttujina Syntymävaltio, Ikä + 

  -  - Adoptiot 1987-1989 muuttujina Syntymävaltio, Ikä -     +  + Neliöhinta (EUR/m2), Helsinki 2020Q1-2023Q2* muuttujina Talotyyppi, Huoneluku + 
 + Rivitalot | Kaksiot + 

  -  - Neliöhinta (EUR/m2), Helsinki 2020Q1-2023Q2* muuttujina Talotyyppi, Huoneluku - 
 - Rivitalot | Kaksiot -     +  + Lukumäärä, Vantaa, Yksiöt, Vapaarahoitteinen 2022Q4 + 

  -  - Lukumäärä, Vantaa, Yksiöt, Vapaarahoitteinen 2022Q4 -     @@ -469,14 +475,16 @@ exports[`Html table render tests should match snapshot: Table with row and colum " +  + Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto + 

  -  - Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -     +  + Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto + 

  -  - Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -     +  + Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto + 

  -  - Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -     +  + Lukumäärä 2022Q1-2022Q4 muuttujina Alue, Huoneluku, Rahoitusmuoto + 

  -  - Lukumäärä 2022Q1-2022Q4 muuttujina Alue, Huoneluku, Rahoitusmuoto -     +  + Lukumäärä, Vantaa, Yksiöt, Vapaarahoitteinen 2022Q4 + 

  -  - Lukumäärä, Vantaa, Yksiöt, Vapaarahoitteinen 2022Q4 -     diff --git a/src/core/tables/htmlTable.ts b/src/core/tables/htmlTable.ts index db1c2aa..ccc9d14 100644 --- a/src/core/tables/htmlTable.ts +++ b/src/core/tables/htmlTable.ts @@ -15,15 +15,21 @@ export function renderHtmlTable(view: View, locale: string, options: IChartOptio const table = generateTable(view, locale); if (options.showTitles) { + const titleId = `${containerId}-title`; + const titleElement = document.createElement('p'); + titleElement.id = titleId; + titleElement.textContent = view.header[locale]; + titleElement.className = 'tableChart-title'; - const caption = document.createElement('caption'); - caption.textContent = view.header[locale]; if (view.subheaderValues.length > 0) { const subtitle: string = view.subheaderValues.map(value => value[locale]).join(' | '); - caption.append(document.createElement('br'), subtitle); + titleElement.append(document.createElement('br'), subtitle); } - caption.className = 'tableChart-caption'; - table.prepend(caption); + + // Set aria-labelledby on the table to reference the title + table.setAttribute('aria-labelledby', titleId); + + container.append(titleElement); } container.append(table); diff --git a/src/react/components/chart/__snapshots__/chart.test.tsx.snap b/src/react/components/chart/__snapshots__/chart.test.tsx.snap index d7b8436..6cc8f5c 100644 --- a/src/react/components/chart/__snapshots__/chart.test.tsx.snap +++ b/src/react/components/chart/__snapshots__/chart.test.tsx.snap @@ -56,14 +56,16 @@ exports[`Rendering test renders chart data correctly 1`] = ` class="tableChart" id="foobar" > +

+ Lukumäärä, Pääkaupunkiseutu (PKS), Yksiöt 2015Q1-2015Q2 muuttujana Rahoitusmuoto +

-
- Lukumäärä, Pääkaupunkiseutu (PKS), Yksiöt 2015Q1-2015Q2 muuttujana Rahoitusmuoto -
+

+ Lukumäärä, Pääkaupunkiseutu (PKS), Yksiöt 2015Q1-2015Q2 muuttujana Rahoitusmuoto +

-
- Lukumäärä, Pääkaupunkiseutu (PKS), Yksiöt 2015Q1-2015Q2 muuttujana Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
+

+ Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto +

-
- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -
Date: Thu, 6 Nov 2025 12:26:30 +0200 Subject: [PATCH 3/5] Adjust burgermenu and chart container padding based on visualization type and title --- .../chart/__snapshots__/chart.test.tsx.snap | 36 +++++++++---------- src/react/components/chart/chart.tsx | 26 +++++++++----- .../components/globalStyle/globalStyle.ts | 1 - 3 files changed, 35 insertions(+), 28 deletions(-) diff --git a/src/react/components/chart/__snapshots__/chart.test.tsx.snap b/src/react/components/chart/__snapshots__/chart.test.tsx.snap index 6cc8f5c..c6f9a57 100644 --- a/src/react/components/chart/__snapshots__/chart.test.tsx.snap +++ b/src/react/components/chart/__snapshots__/chart.test.tsx.snap @@ -3,10 +3,10 @@ exports[`Rendering test renders chart data correctly 1`] = `
+

+ Yksikkö: Lukumäärä: lukumäärä, Neliövuokra (eur/m2): eur / m2 +

+

+ Test footnote +

Lähde: PxVisualizer-fi

@@ -1071,50 +4106,11 @@ exports[`Rendering test renders table data correctly 1`] = ` `; -exports[`Rendering test renders table data correctly when given footnote 1`] = ` +exports[`Rendering test renders table data correctly with hidden context menu 1`] = `
-
-
- -
-
-
-

- Test footnote -

Lähde: PxVisualizer-fi

@@ -1759,13 +4752,13 @@ exports[`Rendering test renders table data correctly when given footnote 1`] = ` `; -exports[`Rendering test renders table data correctly when sources are on 1`] = ` +exports[`Rendering test renders table data correctly with hidden titles 1`] = `
-

- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -

@@ -2444,19 +5430,19 @@ exports[`Rendering test renders table data correctly when sources are on 1`] = ` `; -exports[`Rendering test renders table data correctly when titles are forced on 1`] = ` +exports[`Rendering test renders table data correctly with hidden titles and context menu enabled (should add padding) 1`] = `
@@ -3129,62 +6108,16 @@ exports[`Rendering test renders table data correctly when titles are forced on 1 `; -exports[`Rendering test renders table data correctly when units and footnote are on 1`] = ` +exports[`Rendering test renders table data correctly with hidden titles and hidden context menu 1`] = `
-
-
- -
-
-
-

- Tiedot 2022Q1-2022Q4 muuttujina Tiedot, Alue, Huoneluku, Rahoitusmuoto -

@@ -3806,12 +6739,6 @@ exports[`Rendering test renders table data correctly when units and footnote are
-

- Yksikkö: Lukumäärä: lukumäärä, Neliövuokra (eur/m2): eur / m2 -

-

- Test footnote -

Lähde: PxVisualizer-fi

@@ -3832,7 +6759,7 @@ exports[`Rendering test renders table data correctly with last updated date 1`] class="sc-jJLAfE cTICwY" >