Skip to content

Improve photo loading and full-screen gallery swiping - #253

Open
Xare123 wants to merge 2 commits into
OpenBubbles:rustpushfrom
Xare123:agent/photo-viewer-loading-and-swipe
Open

Improve photo loading and full-screen gallery swiping#253
Xare123 wants to merge 2 commits into
OpenBubbles:rustpushfrom
Xare123:agent/photo-viewer-loading-and-swipe

Conversation

@Xare123

@Xare123 Xare123 commented Aug 16, 2026

Copy link
Copy Markdown

Summary

  • Guard asynchronous image loads from updating disposed widgets
  • Bound decoded image dimensions to the displayed size and a 1024-pixel ceiling
  • Let unzoomed full-screen photos yield horizontal gestures to the surrounding gallery
  • Add deterministic coverage for normal, invalid-metadata, and oversized decode dimensions

Why

Large photos should not be decoded at unnecessarily high resolution for a small message bubble, and users should be able to swipe directly between full-screen images. Zoomed photos retain their own pan behavior.

Validation

Manual interaction still recommended

Open a thread with multiple photos, swipe between them, zoom and pan without accidental paging, then dismiss and reopen while an image is loading.

Scope

This intentionally excludes the broad media cache, audio/video controller, sticker, reaction, lifecycle, workflow, and documentation changes from #231.

@Xare123
Xare123 marked this pull request as ready for review August 16, 2026 02:15
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