[PR #417] [MERGED] Mobile performance improvements #8607

Closed
opened 2026-02-05 13:50:10 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/417
Author: @matthinc
Created: 8/1/2022
Status: Merged
Merged: 8/8/2022
Merged by: @alextran1502

Base: mainHead: dev/mobile-performance-tweaks


📝 Commits (10+)

  • fef67a4 First performance tweaks (caching and rendering improvemetns)
  • 27f00f0 Revert asset response caching
  • 9303746 3-step image loading in asset viewer
  • 9e1d1da Prevent panning and zooming until full-scale version is loaded
  • acb725e Loading indicator
  • 6aa4704 Adapt to gallery PR
  • 8270abd Cleanup
  • 3b87097 Dart format
  • 965fcc3 Fix exif sheet
  • e8e86ea Disable three stage loading until settings are available

📊 Changes

13 files changed (+159 additions, -81 deletions)

View changed files

📝 mobile/lib/main.dart (+6 -0)
📝 mobile/lib/modules/album/ui/album_viewer_thumbnail.dart (+2 -3)
📝 mobile/lib/modules/album/ui/shared_album_thumbnail_image.dart (+2 -3)
📝 mobile/lib/modules/asset_viewer/ui/remote_photo_view.dart (+58 -20)
📝 mobile/lib/modules/asset_viewer/ui/top_control_app_bar.dart (+10 -0)
📝 mobile/lib/modules/asset_viewer/views/gallery_viewer.dart (+7 -6)
📝 mobile/lib/modules/asset_viewer/views/image_viewer_page.dart (+19 -12)
📝 mobile/lib/modules/home/ui/thumbnail_image.dart (+2 -3)
📝 mobile/lib/modules/home/views/home_page.dart (+1 -0)
📝 mobile/lib/routing/router.gr.dart (+28 -34)
mobile/lib/utils/image_url_builder.dart (+16 -0)
📝 mobile/pubspec.lock (+7 -0)
📝 mobile/pubspec.yaml (+1 -0)

📄 Description

Different strategies (optimized rendering, caching) to improve the performance of the mobile app.

Progress:

  • Allow > 60 FPS rendering on some Android devices
  • Load assets in 3 steps (thumbnail -> SD -> HD) instead of 2 (thumbnail -> HD)
  • Prevent all gestures on remote_photo_view until HD version is loaded (prevents jumping)
  • Loading indicator in toolbar of image_viewer_page

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/417 **Author:** [@matthinc](https://github.com/matthinc) **Created:** 8/1/2022 **Status:** ✅ Merged **Merged:** 8/8/2022 **Merged by:** [@alextran1502](https://github.com/alextran1502) **Base:** `main` ← **Head:** `dev/mobile-performance-tweaks` --- ### 📝 Commits (10+) - [`fef67a4`](https://github.com/immich-app/immich/commit/fef67a4b2daaa87920569b579d34e2def232f922) First performance tweaks (caching and rendering improvemetns) - [`27f00f0`](https://github.com/immich-app/immich/commit/27f00f0e47678c19b05dd4ab353e6b5868c6f390) Revert asset response caching - [`9303746`](https://github.com/immich-app/immich/commit/9303746a3d89711e7595a525003771ff36fc4d24) 3-step image loading in asset viewer - [`9e1d1da`](https://github.com/immich-app/immich/commit/9e1d1da1c5120d9d4bb1aa036b68ffb33553d8a2) Prevent panning and zooming until full-scale version is loaded - [`acb725e`](https://github.com/immich-app/immich/commit/acb725e192715f07cfa8b3d8ab8eb840d0f621a1) Loading indicator - [`6aa4704`](https://github.com/immich-app/immich/commit/6aa47046f363e076b014b1737e5fae9c4804aa8f) Adapt to gallery PR - [`8270abd`](https://github.com/immich-app/immich/commit/8270abdb5cd4e863125c8bd58baebc329ae3e9e5) Cleanup - [`3b87097`](https://github.com/immich-app/immich/commit/3b870970f85a7fbfd13cdfc3ac89e79d060931fb) Dart format - [`965fcc3`](https://github.com/immich-app/immich/commit/965fcc34074ab969c0a1bacc86c42805834b4813) Fix exif sheet - [`e8e86ea`](https://github.com/immich-app/immich/commit/e8e86eae8aa29865f469b143fa5c12bc38917258) Disable three stage loading until settings are available ### 📊 Changes **13 files changed** (+159 additions, -81 deletions) <details> <summary>View changed files</summary> 📝 `mobile/lib/main.dart` (+6 -0) 📝 `mobile/lib/modules/album/ui/album_viewer_thumbnail.dart` (+2 -3) 📝 `mobile/lib/modules/album/ui/shared_album_thumbnail_image.dart` (+2 -3) 📝 `mobile/lib/modules/asset_viewer/ui/remote_photo_view.dart` (+58 -20) 📝 `mobile/lib/modules/asset_viewer/ui/top_control_app_bar.dart` (+10 -0) 📝 `mobile/lib/modules/asset_viewer/views/gallery_viewer.dart` (+7 -6) 📝 `mobile/lib/modules/asset_viewer/views/image_viewer_page.dart` (+19 -12) 📝 `mobile/lib/modules/home/ui/thumbnail_image.dart` (+2 -3) 📝 `mobile/lib/modules/home/views/home_page.dart` (+1 -0) 📝 `mobile/lib/routing/router.gr.dart` (+28 -34) ➕ `mobile/lib/utils/image_url_builder.dart` (+16 -0) 📝 `mobile/pubspec.lock` (+7 -0) 📝 `mobile/pubspec.yaml` (+1 -0) </details> ### 📄 Description Different strategies (optimized rendering, caching) to improve the performance of the mobile app. Progress: - Allow > 60 FPS rendering on some Android devices - Load assets in 3 steps (thumbnail -> SD -> HD) instead of 2 (thumbnail -> HD) - Prevent all gestures on `remote_photo_view` until HD version is loaded (prevents jumping) - Loading indicator in toolbar of `image_viewer_page` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 13:50:10 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#8607