Skip to content

fix(widgets): real thumbnails for cloud-only files; bars agree with their number - #88

Merged
jiunbae merged 3 commits into
mainfrom
fix/r14b-widget-thumbnails
Oct 4, 2026
Merged

jiunbae merged 3 commits into
mainfrom
fix/r14b-widget-thumbnails

Conversation

@jiunbae

@jiunbae jiunbae commented Oct 4, 2026

Copy link
Copy Markdown
Member

Follow-up to #87, from the user's check that the screenshot widget showed no previews and looked unlike the approved design.

  • Cloud-only files: Quick Look has no preview for a OneDrive/iCloud file that isn't downloaded (QLThumbnailError 3). So an image of 25 MB or less is now read through ImageIO, which asks the provider to download it. If that fails, the file icon stands in as -icon.png and the picture is retried every 10 minutes. Before this, the stand-in icon was saved under the picture's name and kept for good. The name scheme is bumped (v2) so the old ones are pruned.
  • Bars agree with their number: aas draws the used share while saying "97% left". A bar that is the exact complement of the percentage beside it is turned around (SharedShelf.matchingBar, tested).
  • Matches the mockup: list initials sit on the widget's accent colour (red in trouble), and screenshot names shorten to their time ("Screenshot 20:05").
  • The publish continuation runs on the main actor (removes a Swift 6 warning).

Checked: swift test 567 tests, 0 failures. Both extension schemes build. Previews rendered from real shared data. Signed build installed on june-mba: 5 real thumbnails plus 7 icons for OneDrive files that can't be downloaded right now (the OneDrive app isn't running, so reads time out).

🤖 Generated with Claude Code

jiunbae and others added 3 commits October 4, 2026 17:56
…heir number

Quick Look has no preview for a OneDrive or iCloud file that is not
downloaded, so the newest screenshots showed a generic icon — and the
icon was kept for good. Read such an image (≤ 25 MB) to make its
thumbnail, and retry a stand-in icon every ten minutes. A bar that
fills the other way from its "97% left" is turned around, list
initials take the widget's colour, and screenshot names shorten to
their time, as in the approved design.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@jiunbae
jiunbae merged commit 7d27a10 into main Oct 4, 2026
1 check passed
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