Skip to content

Fix Fast Transfer badge overlapping Selected chip#5567

Open
omi-discord-vector[bot] wants to merge 1 commit intomainfrom
atlas/fix-fast-transfer-badge-overlap
Open

Fix Fast Transfer badge overlapping Selected chip#5567
omi-discord-vector[bot] wants to merge 1 commit intomainfrom
atlas/fix-fast-transfer-badge-overlap

Conversation

@omi-discord-vector
Copy link

Problem

On the Fast Transfer settings page, the "5X FASTER" badge and "Selected" chip overlap on narrower screens because the title text in the inner Row has no flex constraint.

Fix

  • Wrap title Text in Flexible with TextOverflow.ellipsis so it shrinks instead of overflowing
  • Add SizedBox(width: 10) gap between the Expanded column and the Selected/Select chip

One-file change in fast_transfer_settings_page.dart.

Wrap title text in Flexible so it shrinks on narrow screens instead
of overflowing into the Selected/Select chip. Add SizedBox(width:10)
gap between the Expanded column and the chip for consistent spacing.
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 11, 2026

Greptile Summary

This PR makes a targeted two-line UI fix in fast_transfer_settings_page.dart to prevent the "5X FASTER" badge from visually overlapping the "Selected/Select" chip on narrower screen widths.

Changes:

  • The title Text widget inside the inner Row (within the Expanded column) is now wrapped in a Flexible widget with overflow: TextOverflow.ellipsis. This allows the title to shrink and truncate gracefully instead of consuming the full row width and pushing the badge against the chip.
  • A const SizedBox(width: 10) spacer has been added between the Expanded column and the chip container in the outer Row, ensuring a visible gap at all screen sizes.

Both changes are correct and work together well — Flexible handles the inner-row layout budget, while the SizedBox guarantees explicit breathing room between the content area and the interactive chip in the outer Row.

Confidence Score: 5/5

  • This PR is safe to merge — it is a minimal, well-scoped Flutter layout fix with no logic or state changes.
  • The change touches only widget layout (no business logic, no API calls, no state mutations). Both additions (Flexible with TextOverflow.ellipsis and SizedBox(width: 10)) are idiomatic Flutter and correctly fix the reported overlap without introducing side effects. No regressions are expected.
  • No files require special attention.

Important Files Changed

Filename Overview
app/lib/pages/conversations/fast_transfer_settings_page.dart Wraps the title Text in Flexible with TextOverflow.ellipsis and adds a SizedBox(width: 10) spacer between the Expanded column and the Selected/Select chip to prevent overlap on narrow screens.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Outer Row] --> B[Icon Container 44x44]
    A --> C[SizedBox width=14]
    A --> D[Expanded Column]
    A --> E["SizedBox width=10 ✨ NEW"]
    A --> F[Selected / Select Chip]

    D --> G[Inner Row]
    D --> H[SizedBox height=4]
    D --> I[Speed Text]

    G --> J["Flexible ✨ NEW\n(TextOverflow.ellipsis)"]
    G --> K[SizedBox width=8]
    G --> L[Badge Container\ne.g. 5X FASTER]

    J --> M[Title Text]
Loading

Last reviewed commit: cc2aa9a

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.

0 participants