Skip to content

Conversation

@WilhelmOks
Copy link
Collaborator

@WilhelmOks WilhelmOks commented Jan 28, 2026

TaskWPB-23084 [iOS] Display link created indication in the files lists (large font size improvement)

Issue

Improving the spacing of the file icon so that it doesn't waste too much space when the Dynamic Type is set to a large font.

Testing

  • Set the iPhone global text size to a large value
  • Open the Wire Drive files list
  • make sure one of the files has a Shared Link icon
  • Check the spacing of the file icon and the Link indicator
  • The horizontal spacing shouldn't be too large and the Link icon should not overlap with the text on the right

Before (too much wasted space around file icon):
Bildschirmfoto 2026-01-28 um 14 20 04

After (no wasted space):
Bildschirmfoto 2026-01-28 um 14 39 22


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
Contributor

github-actions bot commented Jan 28, 2026

Test Results

2 120 tests   2 093 ✅  2m 44s ⏱️
  354 suites     27 💤
    2 files        0 ❌

Results for commit 6e8b0e7.

♻️ This comment has been updated with latest results.

@datadog-wireapp
Copy link

datadog-wireapp bot commented Jan 28, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

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

Copy link
Contributor

@jullianm jullianm left a comment

Choose a reason for hiding this comment

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

Nice work

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.

3 participants