fix: attachment preview header layout issues - WPB-23931

[WilhelmOks]

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

Issue

In the file attachment header, the space between the first line (file icon and file type) and the second line (file name) was too large.
The issue was that in many places the size was set to take up maximum space, either through maxHeight: .infinity or through Spacer(). Then to reduce the height, an explicit height was set (height: 74).
This not only caused the layout issues but also broke the UI when the font size was set to very large.
I fixed it by removing the indefinite growing and the explicit heights.

Additionally I added a gray background and some padding in SwiftUI Previews to better see if there are issues with the layout.

Testing

Send messages with attachments of different files and check if the header layout is ok.
Especially the space between the file type and the file name should not be too large.
Most important to check are video files and documents.

[github-actions]

Test Results

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

Results for commit c1574ff.

♻️ This comment has been updated with latest results.

Summary: workflow run #23584918494
Allure report (download zip): html-report-28824-fix_WPB-23931-large-attachment-preview-header-padding

[datadog-wireapp]

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: c1574ff | Docs | Was this helpful? Give us feedback!}

[jullianm]

It does fix the issue in conversation previews however it also changes the layout (height) of draft previews which might be acceptable but I think it's better to keep the draft previews as it was before

[WilhelmOks]

It does fix the issue in conversation previews however it also changes the layout (height) of draft previews which might be acceptable but I think it's better to keep the draft previews as it was before

In the latest commit I changed it for the draft previews to keep the same bubble height as the other elements in the list

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[jullianm]

Looks good!