Skip to content

fix: chart review HIGH + MEDIUM bugs (bar, line, table) - #870

Merged
alfredo1996 merged 6 commits into
release/1.0from
fix/v1.0-chart-review-high-severity
May 20, 2026
Merged

fix: chart review HIGH + MEDIUM bugs (bar, line, table)#870
alfredo1996 merged 6 commits into
release/1.0from
fix/v1.0-chart-review-high-severity

fix: chart review medium — auto-derived ARIA descriptions

93661e7
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed May 19, 2026 in 42s

Quality Gate failed

Failed conditions
8.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 108 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZs&open=AZ4_u9rAmVS1d7_QUIZs&pullRequest=870

Check warning on line 116 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZx&open=AZ4_u9rAmVS1d7_QUIZx&pullRequest=870

Check warning on line 114 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZv&open=AZ4_u9rAmVS1d7_QUIZv&pullRequest=870

Check warning on line 109 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZt&open=AZ4_u9rAmVS1d7_QUIZt&pullRequest=870

Check warning on line 115 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZw&open=AZ4_u9rAmVS1d7_QUIZw&pullRequest=870

Check warning on line 311 in app/src/components/table-renderer.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use <section aria-label=...>, or <section aria-labelledby=...> instead of the "region" role to ensure accessibility across all devices. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_3Ig5Tfc-xpUCQ--V&open=AZ4_3Ig5Tfc-xpUCQ--V&pullRequest=870

Check failure on line 99 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZr&open=AZ4_u9rAmVS1d7_QUIZr&pullRequest=870

Check failure on line 95 in component/src/charts/line-chart.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9l0mVS1d7_QUIZq&open=AZ4_u9l0mVS1d7_QUIZq&pullRequest=870

Check warning on line 110 in component/src/charts/chart-utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=alfredo1996_neoboard&issues=AZ4_u9rAmVS1d7_QUIZu&open=AZ4_u9rAmVS1d7_QUIZu&pullRequest=870
View more details on SonarQubeCloud