Skip to content

Replace fixed height containers with flexible constraints for game titles#9

Draft
Copilot wants to merge 2 commits intomasterfrom
copilot/adjust-title-spacing
Draft

Replace fixed height containers with flexible constraints for game titles#9
Copilot wants to merge 2 commits intomasterfrom
copilot/adjust-title-spacing

Conversation

Copy link
Copy Markdown

Copilot AI commented Jan 27, 2026

Long game titles (e.g., "Identity Crisis (Body Snatchers)", "Replay (Emotions, Genres, Mix)") overflow fixed-height containers and get truncated.

Changes

Replaced height: 90 with constraints: BoxConstraints(minHeight: 90) in title display containers:

Container(
-  height: 90,
   padding: EdgeInsets.all(constants.BORDER_PADDING),
+  constraints: BoxConstraints(minHeight: 90),
   child: Center(
     child: Text(
       "$randomGame",

Modified files:

  • lib/game-picker.dart - Random game title display
  • lib/suggestion.dart - Random suggestion display

The containers now expand vertically as needed while maintaining minimum height for short titles. The Expanded GridView below adapts to remaining space.

Original prompt

This section details on the original issue you should resolve

<issue_title>when games have long titles it gets cut off</issue_title>
<issue_description></issue_description>

<agent_instructions>adjust spacing so that long titles don't get cut off</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: sarahwithanhwilson <39014384+sarahwithanhwilson@users.noreply.github.com>
Copilot AI changed the title [WIP] Adjust spacing for long game titles Replace fixed height containers with flexible constraints for game titles Jan 27, 2026
@sarahwithanhwilson
Copy link
Copy Markdown
Owner

Why use BoxConstraints?

@sarahwithanhwilson
Copy link
Copy Markdown
Owner

@copilot wh use BoxConstraints?

Copy link
Copy Markdown
Author

Copilot AI commented Jan 27, 2026

@copilot wh use BoxConstraints?

Using BoxConstraints(minHeight: 90) instead of height: 90 allows the container to grow taller when needed for long game titles while maintaining a 90px minimum for short titles. With a fixed height, long titles get cut off. With minHeight, the text can wrap to multiple lines and the container expands to show all content.

@sarahwithanhwilson
Copy link
Copy Markdown
Owner

@cubic-dev-ai

@cubic-dev-ai
Copy link
Copy Markdown

cubic-dev-ai Bot commented Jan 29, 2026

@cubic-dev-ai

@sarahwithanhwilson I have started the AI code review. It will take a few minutes to complete.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

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.

when games have long titles it gets cut off

2 participants