Skip to content

Cmckahan fix banner aspect ratio 16 9 - #1129

Closed
cmckahan wants to merge 6 commits into
Moonfin-Client:mainfrom
cmckahan:cmckahan-fix-banner-aspect-ratio-16-9
Closed

Cmckahan fix banner aspect ratio 16 9#1129
cmckahan wants to merge 6 commits into
Moonfin-Client:mainfrom
cmckahan:cmckahan-fix-banner-aspect-ratio-16-9

Conversation

@cmckahan

Copy link
Copy Markdown

Pull Request

Summary

Provide a brief description of what this PR changes and why.

Related Issues

Link related issues or tickets separated by commas.

  • Closes #
  • Fixes #
  • Related to #

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • UI/UX update
  • Documentation update
  • Build/CI change
  • Other (describe):

Changes Made

List the key changes included in this PR.

Platform

  • Android
  • iOS
  • tvOS
  • Web
  • macOS
  • Windows
  • Linux
  • All / Shared code

Testing

Describe how this change was tested.

  • Tested on emulator / simulator
  • Tested on physical device
  • Manual testing completed
  • Not tested (explain why):

Test Steps

Screenshots (if applicable)

Include screenshots or recordings for UI changes.

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced

cmckahan and others added 6 commits August 10, 2026 20:39
- Maintains platform-specific heights (TV: 320px, desktop: 240px, mobile: 200px)
- Adds AspectRatio widget to enforce consistent 16:9 aspect ratio
- Wraps content with Align widget to center the banner horizontally
- Banner width will now be calculated based on 16:9 ratio at fixed height

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@cmckahan cmckahan closed this Aug 11, 2026
@cmckahan
cmckahan deleted the cmckahan-fix-banner-aspect-ratio-16-9 branch August 11, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant