docs: show units on comparison charts

[aqilaziz]

Summary

• add explicit unit labels to the shared comparison bar chart header and SVG axis
• include the unit in the chart accessibility description
• set clearer unit labels for count-based charts such as dependencies and ops/sec

Testing

• pnpm exec prettier --check packages/docs/src/components/ComparisonBarChart.astro packages/docs/src/components/DepsChart.astro packages/docs/src/components/ProdDepsChart.astro packages/docs/src/components/DuplicateDependencyChart.astro packages/docs/src/components/SSROpsChart.astro packages/docs/src/lib/types.ts
• pnpm --filter @framework-tracker/docs type-check
• pnpm --filter @framework-tracker/docs build
• git diff --check

Fixes #214

[AlexanderKaran]

@aqilaziz For UX/UI changes can you please add a screen shot or gif of the changes

[aqilaziz]

Added screenshots for the chart unit labels.

Dev Time charts:

Run Time charts:

Validation run:

• pnpm --filter @framework-tracker/docs type-check
• pnpm --filter @framework-tracker/docs build

[AlexanderKaran]

Thanks @aqilaziz, look at those screenshots, we now have three units in some places. My fault task was not super clear,

I think it would be best to just remove units from the Graph titles and just have it in the graphs themselves

[AlexanderKaran]

Clean up other units so its just the unit in the graph, remove from the title for example

[aqilaziz]

Addressed the review feedback by removing units from the chart titles. Units now stay in the chart UI/axis only.

Updated runtime screenshot:

Validation:

• pnpm exec prettier --check packages/docs/src/components/MPAFPChart.astro packages/docs/src/components/MPAFCPChart.astro packages/docs/src/components/MPAINPChart.astro packages/docs/src/components/SPAFPChart.astro packages/docs/src/components/SPAFCPChart.astro packages/docs/src/components/SPAINPChart.astro packages/docs/src/components/SSROpsChart.astro
• pnpm --filter @framework-tracker/docs type-check
• pnpm --filter @framework-tracker/docs build
• git diff --check

[AlexanderKaran]

@aqilaziz Great work thank you so much

[43081j]

is there any way we can put the label on the axis?

really it should look like this:

                 |
                 |
time (ms)        |
                 |
                 |

i think 🤔 or sideways. @AlexanderKaran ?

its an axis label, thats where it'd normally be i think