Skip to content

tweak(ui): 'new update' image layout optimization to fit more content#5306

Open
ux-git wants to merge 1 commit into
sws2apps:mainfrom
ux-git:tweak_ui_image_viewer
Open

tweak(ui): 'new update' image layout optimization to fit more content#5306
ux-git wants to merge 1 commit into
sws2apps:mainfrom
ux-git:tweak_ui_image_viewer

Conversation

@ux-git

@ux-git ux-git commented Jul 20, 2026

Copy link
Copy Markdown
Member

Description

Tweaked the update window image layout. The image container had a fixed height which caused excessive empty vertical space on mobile displays. It now uses a dynamic height with maxHeight: '200px' and objectFit: 'contain'. The parent container padding was also updated to use 16px 0 for all breakpoints.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

@vercel

vercel Bot commented Jul 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
staging-organized-app Ready Ready Preview Jul 20, 2026 9:12pm
test-organized-app Ready Ready Preview Jul 20, 2026 9:12pm

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The image viewer now has reduced vertical padding, and slide images use automatic height, a 200px maximum height, and contained object fitting.

Changes

Image viewer layout

Layer / File(s) Summary
Responsive image layout
src/features/whats_new/image_viewer/index.tsx
The viewer padding changed from 24px 0 to 16px 0, and slide images now use responsive sizing with height: auto, maxHeight: 200px, and objectFit: contain.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title matches the main change: optimizing the update window image layout to fit more content.
Description check ✅ Passed The description directly describes the image layout and padding changes in the update window.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@sonarqubecloud

Copy link
Copy Markdown

@mergify

mergify Bot commented Jul 20, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@ux-git ux-git changed the title tweak(ui): new update image layout optimization to fit more content tweak(ui): 'new update' image layout optimization to fit more content Jul 21, 2026
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.

1 participant