Skip to content

fix: home panel max_width - #451

Merged
mverch67 merged 1 commit into
masterfrom
fix-panel-max-width
Oct 30, 2026
Merged

mverch67 merged 1 commit into
masterfrom
fix-panel-max-width

Conversation

@mverch67

@mverch67 mverch67 commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

For some reason the max_wdith 480 does not do it on large displays and crippled the home panel layout.

Summary by CodeRabbit

  • UI Updates
    • The home panel’s maximum width is now 440 pixels, down from 480 pixels. This makes the panel narrower when there is ample display space. The update affects the panel’s layout only; its content and controls remain unchanged.

@mverch67 mverch67 added the bugfix PR that resolves issues label Oct 30, 2026
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 6d94d18 and 669e8c6.

Files ignored due to path filters: 1
  • generated/ui_320x240/screens.c is excluded by !**/generated/**, !generated/**
Files selected for processing: 1
  • studio/320x240/TFT320x240.eez-project

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

Walkthrough

The HomeContainer panel’s maximum width changes from 480 to 440.

Changes: Home panel width

Layer / File(s) Summary
Update HomeContainer maximum width
studio/320x240/TFT320x240.eez-project
The HomeContainer panel’s maximum width changes from 480 to 440.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix

Suggested reviewers: alecperkins

Merge Risk: Minimal · up to 669e8

This focused layout adjustment presents no established merge-blocking risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the main change: fixing the home panel max_width behavior. It is concise and specific.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mverch67
mverch67 merged commit 89605c7 into master Oct 30, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR that resolves issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant