Skip to content

fix: ThemedTable2 header overflow detection by including the sort icon width in the available text width, ensuring tooltips appear correctly #126

Merged
ManuelRomeroA merged 1 commit into
mainfrom
development
Mar 24, 2026
Merged

fix: ThemedTable2 header overflow detection by including the sort icon width in the available text width, ensuring tooltips appear correctly #126
ManuelRomeroA merged 1 commit into
mainfrom
development

Conversation

@ManuelRomeroA
Copy link
Copy Markdown
Contributor

This pull request addresses an issue with header overflow detection in the ThemedTable2 widget. The main improvement ensures that tooltips appear correctly by accounting for the width of the sort icon when calculating the available text width in table headers.

Table Header Overflow Fixes

  • Updated the calculation of available text width in ThemedTable2 headers to subtract the sort icon width when present, ensuring accurate overflow detection and proper tooltip display.

Documentation and Versioning

  • Updated the CHANGELOG.md to document the header overflow fix in version 7.5.20.
  • Bumped the package version to 7.5.20 in pubspec.yaml.

…n width in the available text width, ensuring tooltips appear correctly f
@github-actions
Copy link
Copy Markdown

📋 Changelog Summary

This PR includes the following changes:

🐛 Bug Fixes

  • ThemedTable2 header overflow detection by including the sort icon width in the available text width, ensuring tooltips appear correctly f (09d789e by Manuel romero)

📊 Diff Stats

Metric Count
📄 Files changed 3
🟢 Insertions $\color{green}\textsf{+8}$
🔴 Deletions $\color{red}\textsf{-2}$

🤖 Auto-generated by changelog workflow

@github-actions
Copy link
Copy Markdown

📊 Test Coverage Report

🎯 Dart/Flutter

Metric Value
Coverage 11.1%
Lines covered 10811 / 127
Lines missing -10684

🤖 Generated by coverage workflow

@ManuelRomeroA ManuelRomeroA merged commit 8c45431 into main Mar 24, 2026
4 checks passed
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.

1 participant