Skip to content

fix width calculations for tables#352

Merged
mpollmeier merged 3 commits intomasterfrom
maltek-patch-1
Mar 11, 2026
Merged

fix width calculations for tables#352
mpollmeier merged 3 commits intomasterfrom
maltek-patch-1

Conversation

@maltek
Copy link
Contributor

@maltek maltek commented Mar 11, 2026

we have a lot of tests that depend on the exact layout of the tables, e.g. see https://ci.shiftleftsecurity.com/blue/organizations/jenkins/CI-PR-CODESCIENCE-pr.Jenkinsfile/detail/CI-PR-CODESCIENCE-pr.Jenkinsfile/10852/tests

note: that's a blind fix. I haven't actually pulled this all the way up the dependency chain to verify that's the only inadvertent change we want to avoid.
@maltek maltek requested a review from mpollmeier March 11, 2026 11:53
@maltek
Copy link
Contributor Author

maltek commented Mar 11, 2026

Fixed some more bugs: The available width was taken as a max width per column instead of for the whole table.

@maltek maltek changed the title remove table min-width in order to keep output identical to before fix width calculations for tables Mar 11, 2026
@maltek
Copy link
Contributor Author

maltek commented Mar 11, 2026

I have tested with downstream now. We still have some failed tests, because the line breaking when the max width is exceeded works better now. I'd say that's worth it and we should adjust the affected tests.

@mpollmeier mpollmeier merged commit 391c11c into master Mar 11, 2026
1 check passed
@mpollmeier
Copy link
Contributor

started the release workflow: https://github.com/joernio/flatgraph/actions/runs/22957684345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants