Skip to content

fix: truncate long contact name in favorites tile (WT-1051)#1373

Merged
SERDUN merged 1 commit into
developfrom
fix/contact-name-truncation
Jun 9, 2026
Merged

fix: truncate long contact name in favorites tile (WT-1051)#1373
SERDUN merged 1 commit into
developfrom
fix/contact-name-truncation

Conversation

@SERDUN

@SERDUN SERDUN commented Jun 9, 2026

Copy link
Copy Markdown
Member

Overview

WT-1051 reported that a long contact name in the Contacts list was not truncated and overflowed, breaking the list layout. The Contacts tabs were already fixed by WT-1537 (ContactTile now wraps the name in an Expanded with maxLines + TextOverflow.ellipsis), but the Favorites tile had the same defect left over.

FavoriteTile rendered the name and the label/number subtitle through a plain ListTile with no line or overflow constraint, so a long favorite name still wrapped over multiple lines.

Changes

  • favorite_tile.dart: add maxLines: 1 + overflow: TextOverflow.ellipsis to the title (both presence and non-presence branches) and to the subtitle, matching the truncation behavior of ContactTile.

Testing

  • dart format --set-exit-if-changed clean.
  • dart analyze on the file: no issues.
  • Pre-push hooks green (flutter analyze + full test suite, 747 tests).

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes WT-1051 layout break in the Favorites list by ensuring long favorite contact names (and the subtitle) are truncated instead of wrapping/overflowing, aligning behavior with ContactTile.

Changes:

  • Add maxLines: 1 and TextOverflow.ellipsis to FavoriteTile title in both presence and non-presence branches.
  • Add maxLines: 1 and TextOverflow.ellipsis to FavoriteTile subtitle.

@SERDUN SERDUN merged commit 5f19f0f into develop Jun 9, 2026
3 checks passed
@SERDUN SERDUN deleted the fix/contact-name-truncation branch June 9, 2026 12:44
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.

2 participants