Skip to content

fix: remove top padding from video and files thumbnails - WPB-23931#4466

Merged
jullianm merged 2 commits intorelease/cycle-4.17from
fix/remove-top-padding-files-previews
Mar 23, 2026
Merged

fix: remove top padding from video and files thumbnails - WPB-23931#4466
jullianm merged 2 commits intorelease/cycle-4.17from
fix/remove-top-padding-files-previews

Conversation

@jullianm
Copy link
Copy Markdown
Contributor

@jullianm jullianm commented Mar 20, 2026

BugWPB-23931 [iOS][Files in conversation] Remove top padding from video and files thumbnails

Issue

This PR removes top padding from video and files thumbnails. Also added some corner radius on the large document preview so it is aligned with the large video preview.

Testing

Describe how to test.

Optional: attachments like images, videos, etc.


Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 20, 2026

Test Results

2 140 tests   2 113 ✅  2m 43s ⏱️
  353 suites     27 💤
    2 files        0 ❌

Results for commit 518a825.

♻️ This comment has been updated with latest results.

Summary: workflow run #23427570392
Allure report (download zip): html-report-28707-fix_remove-top-padding-files-previews

Copy link
Copy Markdown
Collaborator

@WilhelmOks WilhelmOks left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link
Copy Markdown

@jullianm jullianm enabled auto-merge (squash) March 23, 2026 08:19
@jullianm jullianm merged commit a5e6f38 into release/cycle-4.17 Mar 23, 2026
12 checks passed
@jullianm jullianm deleted the fix/remove-top-padding-files-previews branch March 23, 2026 08:40
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.

4 participants