AppFlowy-IO/AppFlowy

Fork

You've already forked AppFlowy

mirror of https://github.com/AppFlowy-IO/AppFlowy.git synced 2026-03-24 04:46:56 +00:00

Code Issues 952 Projects Packages Wiki Activity

[PR #5830] [MERGED] fix: photo gallery improvements + launch review fixes #7032

New issue

Closed

opened 2026-03-23 23:17:47 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:17:47 +00:00

Owner

Copy link

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5830
Author: @Xazin
Created: 7/29/2024
Status: ✅ Merged
Merged: 7/30/2024
Merged by: @LucasXu0

Base: main ← Head: fix/photo-gallery-improvements

📝 Commits (4)

93011ee fix: photo gallery improvements
d02cf4b fix: improve when to show billing/plan pages
38d6814 feat: add grid photo gallery layout
8f30559 fix: close inline actions menu

📊 Changes

25 files changed (+872 additions, -266 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_multi_image_block_test.dart (+2 -1)
➕ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/image_render.dart (+42 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/layouts/image_browser_layout.dart (+105 -148)
➕ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/layouts/image_grid_layout.dart (+323 -0)
➕ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/layouts/multi_image_layouts.dart (+79 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/multi_image_block_component.dart (+48 -8)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/multi_image_menu.dart (+121 -15)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/multi_image_placeholder.dart (+6 -4)
📝 frontend/appflowy_flutter/lib/plugins/inline_actions/widgets/inline_actions_handler.dart (+6 -7)
📝 frontend/appflowy_flutter/lib/startup/deps_resolver.dart (+0 -5)
📝 frontend/appflowy_flutter/lib/workspace/application/settings/ai/settings_ai_bloc.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/workspace/application/settings/settings_dialog_bloc.dart (+55 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/pages/setting_ai_view/local_ai_setting.dart (+14 -37)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/settings_dialog.dart (+4 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/setting_appflowy_cloud.dart (+22 -21)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_menu.dart (+3 -6)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/image_viewer/interactive_image_toolbar.dart (+7 -5)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/image_viewer/interactive_image_viewer.dart (+8 -4)
📝 frontend/appflowy_flutter/pubspec.lock (+8 -0)
📝 frontend/appflowy_flutter/pubspec.yaml (+2 -0)

...and 5 more files

📄 Description

Closes: #5814
Closes: #5557
Closes: #2947
Closes: #5424

Close icon too small in interactive image viewer
Double tap to exit image in interactive image viewer
Upload image order
Center align image options in broser
Remove odd selection behavior when double tapping gallery block
Add Grid layout

Feature Preview

https://github.com/user-attachments/assets/51717931-138d-4da7-afc6-55a43445e072

PR Checklist

My code adheres to AppFlowy's Conventions
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

_{🔄 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/AppFlowy-IO/AppFlowy/pull/5830 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 7/29/2024 **Status:** ✅ Merged **Merged:** 7/30/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix/photo-gallery-improvements` --- ### 📝 Commits (4) - [`93011ee`](https://github.com/AppFlowy-IO/AppFlowy/commit/93011eeb96d816af02db20bdccaa53beac347c9d) fix: photo gallery improvements - [`d02cf4b`](https://github.com/AppFlowy-IO/AppFlowy/commit/d02cf4b4b09f5aa9c12a71ab2a55791b49b16dfa) fix: improve when to show billing/plan pages - [`38d6814`](https://github.com/AppFlowy-IO/AppFlowy/commit/38d6814796a11abff5c1ced304b05746a9e7964e) feat: add grid photo gallery layout - [`8f30559`](https://github.com/AppFlowy-IO/AppFlowy/commit/8f305598a9a76d2900c0234355bb43d271617aef) fix: close inline actions menu ### 📊 Changes **25 files changed** (+872 additions, -266 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_multi_image_block_test.dart` (+2 -1) ➕ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/image_render.dart` (+42 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/layouts/image_browser_layout.dart` (+105 -148) ➕ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/layouts/image_grid_layout.dart` (+323 -0) ➕ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/layouts/multi_image_layouts.dart` (+79 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/multi_image_block_component.dart` (+48 -8) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/multi_image_menu.dart` (+121 -15) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/multi_image_block_component/multi_image_placeholder.dart` (+6 -4) 📝 `frontend/appflowy_flutter/lib/plugins/inline_actions/widgets/inline_actions_handler.dart` (+6 -7) 📝 `frontend/appflowy_flutter/lib/startup/deps_resolver.dart` (+0 -5) 📝 `frontend/appflowy_flutter/lib/workspace/application/settings/ai/settings_ai_bloc.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/workspace/application/settings/settings_dialog_bloc.dart` (+55 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/pages/setting_ai_view/local_ai_setting.dart` (+14 -37) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/settings_dialog.dart` (+4 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/setting_appflowy_cloud.dart` (+22 -21) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_menu.dart` (+3 -6) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/image_viewer/interactive_image_toolbar.dart` (+7 -5) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/image_viewer/interactive_image_viewer.dart` (+8 -4) 📝 `frontend/appflowy_flutter/pubspec.lock` (+8 -0) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+2 -0) _...and 5 more files_ </details> ### 📄 Description Closes: #5814 Closes: #5557 Closes: #2947 Closes: #5424 - [x] Close icon too small in interactive image viewer - [x] Double tap to exit image in interactive image viewer - [x] Upload image order - [x] Center align image options in broser - [x] Remove odd selection behavior when double tapping gallery block - [x] Add Grid layout ### Feature Preview https://github.com/user-attachments/assets/51717931-138d-4da7-afc6-55a43445e072 #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] I've listed at least one issue that this PR fixes in the description above. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [x] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

mirror

2026-03-23 23:17:47 +00:00

closed this issue
added the
pull-request
label

No Branch/Tag specified

main

copilot/fix-date-format-issue

feat/mention_a_person

merge_upstream_v6

mobile/release/0.9.3

release/0.9.3

release/0.9.2

select_source

release/0.9.1

refactor_import_data

release/0.8.9.1

release/0.8.9

anon_user_mode

release/0.8.8

fix_windows_selection

release/0.8.7

annieappflowy-patch-1

windows_local_ai

release/0.8.6

release/0.8.5

release/0.8.4

chore/optional-response-format

release/0.8.3

release/0.8.2

release/0.8.1

release/0.8.0

release/0.7.9

feat/support-rename-workspace

chat_rag_only

calculate_ui_test

release/0.7.8

release/0.7.7

stringfi_cells

hotfix/0.7.5

release/0.7.4

client-api-update

feat/collab-editing

fix_calculation_bug

feat/upload-file-in-document

update-client-api

feat/support-global-comment-on-web

update-client-api-2

feat/support-duplicate-publish-on-web

feat/ai-plan-billing

chore/test-web-deploy

chore/enable-billing-for-testing

build/test

ai_tag_feature

chore/enable-billing-plan

feat/billing-client-windows

chore/support-openai-chat

fix/try-no-media-kit

sign-in-error-format

workspace-invite-ci-test

last-workspace

workspace-invite-wip

feat/date-text-input

appflowy_v1

0.11.1

0.11.0

0.10.9

0.11.2

0.10.7

0.10.6

0.10.5

0.11.3

0.11.4

0.10.8

0.10.4

0.10.3

0.10.2

0.10.0

0.10.1

0.9.9

0.9.8

0.9.7

0.9.5

0.9.6

0.9.4

0.9.3

0.9.2

0.9.1

0.9.0

0.8.9

0.8.8

0.8.7

0.8.6

0.8.5

0.8.4

0.8.3

0.8.2

0.8.1

0.8.0

0.7.9

0.7.8

0.7.7

0.7.6

0.7.5

0.7.4

0.7.3

0.7.2

0.7.1

0.7.0

0.6.9

0.6.8

0.6.7.2

0.6.7.1

0.6.7

0.6.6

0.6.5

0.6.4

0.6.3

0.6.2

0.6.1

v0.6.1

0.6.0

0.5.9

0.5.8

0.5.7

0.5.6

0.5.5

0.5.4

0.5.3

0.5.2

0.5.1

0.5.0

0.4.9

0.4.8

0.4.7

0.4.6

0.4.5

0.4.4

0.4.3

0.4.2

0.4.1

0.4.0

0.3.9.1

0.3.9

0.3.8

0.3.7

0.3.6

0.3.5

0.3.4

0.3.3

0.3.2

0.3.1

0.3.0

0.2.9

0.2.8

0.2.7

0.2.6

0.2.5

0.2.4

0.2.3

0.2.2

0.2.1

0.2.0

0.1.6

0.1.5

0.1.4

0.1.3

0.1.2

0.1.1

0.1.0

0.0.9.1

0.0.9

0.0.8.1

0.0.8

0.0.7.1

0.0.7

0.0.6.2

0.0.6.1

0.0.6

0.0.6_beta.1

0.0.5.3

0.0.5.2

0.0.5.1

0.0.5

0.0.5_beta.2

0.0.5_beta.1

0.0.4

0.0.4_beta.3

0.0.4_beta.2

0.0.4_beta.1

0.0.3

0.0.3_beta.2

0.0.3_beta.1

0.0.2

0.0.1

Labels

Clear labels

good first issue for devs

good first issue for experienced devs

grid

hacktoberfest

HACKTOBERFEST-ACCEPTED

Mirrored from GitHub Pull Request

No labels

good first issue for devs

good first issue for experienced devs

grid

hacktoberfest

HACKTOBERFEST-ACCEPTED

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

AppFlowy-IO/AppFlowy#7032

Reference in a new issue

Repository

AppFlowy-IO/AppFlowy

Title

Body

No description provided.

Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?

Rows
Columns