Rapport: curvekolom van het resourcediagram zo breed als de langste curvenaam (#113) - #142
Merged
Merged
Conversation
…urvenaam (#113, manuvarkey) De curvekolom stond vast op 98 px — de breedte waarop alle veertien talen hun langste curvenaam kwijt kunnen — ook als het rapport alleen "Uniform" toont. Nu meet het paneel op het geladen Inter-font de kop en de curvenamen die dít rapport écht toont (`measureCurveColumnWidth`, dezelfde route als de naamkolom: `measurePrintReport` heeft geen canvas) en geeft de breedte mee als `PrintOptions.curveColumnWidth`, geklemd tussen 40 en 98 px. Zonder meting (headless, oude aanroepen) blijft het maximum gelden — byte-identiek. Regressie in check-print-report: tabel 48 px smaller bij 50 px, vloer en maximum, en de meetfunctie zelf. Gids nl+en: "tot ruim 130 px breder". Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014zosxbSXvi3YSGmeN1MsQq
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What and why
Refs #113 — manuvarkey: "Width of Curve column can be smaller to optimise width of table."
De curvekolom stond vast op 98 px, de breedte waarop alle veertien talen hun langste curvenaam kwijt kunnen — ook als het rapport alleen "Uniform" toont. Nu meet
ReportPanelop het geladen Inter-font de kop en de curvenamen die dít rapport écht toont (measureCurveColumnWidth, dezelfde route als de automatische naamkolom:measurePrintReportheeft geen canvas en zou anders een andere tabelbreedte uitrekenen dan de render) en geeft de breedte mee alsPrintOptions.curveColumnWidth, geklemd tussenCURVE_COLUMN_WIDTH_MIN(40 px, kop en streepje passen altijd) enCOL.curve.w(98 px). Zonder meting (headless, oude aanroepen) blijft het maximum gelden — byte-identiek aan vóór.How it was verified
npm run verifygreen — lokaal, los: typecheck, lint, verify:docs, de volledige planningssuite (exit 0), browserspecsresource-gantt-reportenreport-options(3/3).tests/planning/check-print-report.ts— tabel 48 px smaller bijcurveColumnWidth: 50, vloer (10 ⇒ 40) en maximum (500 ⇒ 98), enmeasureCurveColumnWidthzelf (langste label + 2× celmarge + 1, geklemd).Does this touch
@tauri-apps/*— n.v.t.Documentation
public/docs/{nl,en}/gids-rapporten-printen.md: de curvekolom is zo breed als de langste curvenaam in het rapport.🤖 Generated with Claude Code
https://claude.ai/code/session_014zosxbSXvi3YSGmeN1MsQq
Generated by Claude Code