[PR #762] [MERGED] feat(mobile) integrate new grid system to upstream #8752

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

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/762
Author: @matthinc
Created: 9/29/2022
Status: Merged
Merged: 10/14/2022
Merged by: @alextran1502

Base: mainHead: feature/albums-new-grid


📝 Commits (10+)

  • 8440d98 Improve scrolling performance in albums and search
  • dd71a53 Hide scroll handle for lists < 100 assets
  • 1970a64 Use new asset grid for search result page
  • 50842ef Add tests
  • 347ac70 Make new asset grid the default
  • a117e89 Move selection logic to asset grid class
  • 6b84534 Get rid of home page state provider
  • 3c807ae Exernalize multiselect state
  • 6ab6507 Revert changes to albums
  • 2c12f53 Fix storage indicator settings

📊 Changes

25 files changed (+933 additions, -1128 deletions)

View changed files

📝 mobile/assets/i18n/en-US.json (+1 -4)
mobile/lib/modules/home/models/home_page_state.model.dart (+0 -47)
📝 mobile/lib/modules/home/providers/home_page_render_list_provider.dart (+3 -84)
mobile/lib/modules/home/providers/home_page_state.provider.dart (+0 -92)
mobile/lib/modules/home/providers/multiselect.provider.dart (+5 -0)
mobile/lib/modules/home/ui/asset_grid/asset_grid_data_structure.dart (+103 -0)
mobile/lib/modules/home/ui/asset_grid/daily_title_text.dart (+72 -0)
📝 mobile/lib/modules/home/ui/asset_grid/disable_multi_select_button.dart (+36 -40)
📝 mobile/lib/modules/home/ui/asset_grid/draggable_scrollbar_custom.dart (+0 -0)
mobile/lib/modules/home/ui/asset_grid/immich_asset_grid.dart (+274 -0)
📝 mobile/lib/modules/home/ui/asset_grid/monthly_title_text.dart (+0 -0)
📝 mobile/lib/modules/home/ui/asset_grid/thumbnail_image.dart (+172 -176)
mobile/lib/modules/home/ui/asset_list_v2/daily_title_text.dart (+0 -107)
mobile/lib/modules/home/ui/asset_list_v2/immich_asset_grid.dart (+0 -167)
📝 mobile/lib/modules/home/ui/control_bottom_app_bar.dart (+10 -11)
mobile/lib/modules/home/ui/daily_title_text.dart (+0 -109)
📝 mobile/lib/modules/home/ui/delete_diaglog.dart (+4 -9)
mobile/lib/modules/home/ui/image_grid.dart (+0 -47)
📝 mobile/lib/modules/home/views/home_page.dart (+45 -117)
📝 mobile/lib/modules/search/providers/search_result_page.provider.dart (+12 -0)

...and 5 more files

📄 Description

Won't merge until we received community feedback about the experimental home asset grid.


🔄 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/762 **Author:** [@matthinc](https://github.com/matthinc) **Created:** 9/29/2022 **Status:** ✅ Merged **Merged:** 10/14/2022 **Merged by:** [@alextran1502](https://github.com/alextran1502) **Base:** `main` ← **Head:** `feature/albums-new-grid` --- ### 📝 Commits (10+) - [`8440d98`](https://github.com/immich-app/immich/commit/8440d9890c28fa022db124847069f45b66d777a0) Improve scrolling performance in albums and search - [`dd71a53`](https://github.com/immich-app/immich/commit/dd71a53f5ed9b079ea94eb31156f484bd94aa0e4) Hide scroll handle for lists < 100 assets - [`1970a64`](https://github.com/immich-app/immich/commit/1970a64f6fd7ed46d5f12155e40890614e958356) Use new asset grid for search result page - [`50842ef`](https://github.com/immich-app/immich/commit/50842ef815c2e6961b54b86ad06f4dd48a80e11f) Add tests - [`347ac70`](https://github.com/immich-app/immich/commit/347ac70063bb7bc8205296178cd7b9ab8e20c33e) Make new asset grid the default - [`a117e89`](https://github.com/immich-app/immich/commit/a117e897caca2ce19c1912d4f51b105ada2a8e2a) Move selection logic to asset grid class - [`6b84534`](https://github.com/immich-app/immich/commit/6b8453463243be8163f30fbbbe767d54c2fd0200) Get rid of home page state provider - [`3c807ae`](https://github.com/immich-app/immich/commit/3c807ae86e9eb854fce115709c178ef267e306e2) Exernalize multiselect state - [`6ab6507`](https://github.com/immich-app/immich/commit/6ab6507db9c21ed6d36551aae22736f65d8a1f3f) Revert changes to albums - [`2c12f53`](https://github.com/immich-app/immich/commit/2c12f539379bd53a0dc6392575de7bf4ae86ee4f) Fix storage indicator settings ### 📊 Changes **25 files changed** (+933 additions, -1128 deletions) <details> <summary>View changed files</summary> 📝 `mobile/assets/i18n/en-US.json` (+1 -4) ➖ `mobile/lib/modules/home/models/home_page_state.model.dart` (+0 -47) 📝 `mobile/lib/modules/home/providers/home_page_render_list_provider.dart` (+3 -84) ➖ `mobile/lib/modules/home/providers/home_page_state.provider.dart` (+0 -92) ➕ `mobile/lib/modules/home/providers/multiselect.provider.dart` (+5 -0) ➕ `mobile/lib/modules/home/ui/asset_grid/asset_grid_data_structure.dart` (+103 -0) ➕ `mobile/lib/modules/home/ui/asset_grid/daily_title_text.dart` (+72 -0) 📝 `mobile/lib/modules/home/ui/asset_grid/disable_multi_select_button.dart` (+36 -40) 📝 `mobile/lib/modules/home/ui/asset_grid/draggable_scrollbar_custom.dart` (+0 -0) ➕ `mobile/lib/modules/home/ui/asset_grid/immich_asset_grid.dart` (+274 -0) 📝 `mobile/lib/modules/home/ui/asset_grid/monthly_title_text.dart` (+0 -0) 📝 `mobile/lib/modules/home/ui/asset_grid/thumbnail_image.dart` (+172 -176) ➖ `mobile/lib/modules/home/ui/asset_list_v2/daily_title_text.dart` (+0 -107) ➖ `mobile/lib/modules/home/ui/asset_list_v2/immich_asset_grid.dart` (+0 -167) 📝 `mobile/lib/modules/home/ui/control_bottom_app_bar.dart` (+10 -11) ➖ `mobile/lib/modules/home/ui/daily_title_text.dart` (+0 -109) 📝 `mobile/lib/modules/home/ui/delete_diaglog.dart` (+4 -9) ➖ `mobile/lib/modules/home/ui/image_grid.dart` (+0 -47) 📝 `mobile/lib/modules/home/views/home_page.dart` (+45 -117) 📝 `mobile/lib/modules/search/providers/search_result_page.provider.dart` (+12 -0) _...and 5 more files_ </details> ### 📄 Description Won't merge until we received community feedback about the experimental home asset grid. --- <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:52:39 +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#8752