Skip to content

fix developer settings API key containers not stretching full width#7964

Merged
mdmohsin7 merged 1 commit into
mainfrom
fix/developer-settings-api-key-container-width
Jun 15, 2026
Merged

fix developer settings API key containers not stretching full width#7964
mdmohsin7 merged 1 commit into
mainfrom
fix/developer-settings-api-key-container-width

Conversation

@krushnarout

Copy link
Copy Markdown
Member

Added width: double.infinity to the loading, error, and empty-state containers in the Developer Settings API key section so they stretch to fill the available width.

Before:

IMG_E69C1BA3D3F0-1

After:

524F1576-0B92-4751-B525-1F186470A4BB_1_201_a

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds width: double.infinity to the loading, error, and empty-state Container widgets inside _buildApiKeysList in the Developer Settings page, making them fill the available horizontal space consistently with the populated-keys list container.

  • The fix targets exactly three state containers (loading, error, empty) that lacked a width constraint, while the populated state (_buildSectionContainer) already expanded correctly.
  • No logic, state management, or localization changes are included.

Confidence Score: 5/5

Safe to merge — a minimal, targeted layout-only fix with no logic changes.

The change adds width: double.infinity to three Container widgets that previously had no width constraint, correcting a visual inconsistency. The diff is three one-line additions with no side effects on state, navigation, or data handling.

No files require special attention.

Important Files Changed

Filename Overview
app/lib/pages/settings/developer.dart Three Container widgets in _buildApiKeysList now include width: double.infinity to stretch to full width in loading, error, and empty states.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[_buildApiKeysList] --> B{provider.isLoading\n&& keys.isEmpty?}
    B -- Yes --> C["Container\n(width: double.infinity ✅)\nLoading Spinner"]
    B -- No --> D{provider.error\n!= null?}
    D -- Yes --> E["Container\n(width: double.infinity ✅)\nError Text"]
    D -- No --> F{provider.keys\n.isEmpty?}
    F -- Yes --> G["Container\n(width: double.infinity ✅)\nEmpty State"]
    F -- No --> H["_buildSectionContainer\n(already full width)"]
Loading

Reviews (1): Last reviewed commit: "fix developer settings API key container..." | Re-trigger Greptile

@mdmohsin7 mdmohsin7 merged commit bb7f917 into main Jun 15, 2026
3 checks passed
@mdmohsin7 mdmohsin7 deleted the fix/developer-settings-api-key-container-width branch June 15, 2026 11:16
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