[PR #6529] [MERGED] feat: right click to open view more actions #7440

New issue

Closed

opened 2026-03-23 23:19:38 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:19:38 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6529
Author: @Xazin
Created: 10/10/2024
Status: ✅ Merged
Merged: 10/17/2024
Merged by: @Xazin

Base: main ← Head: feat/view-more-right-click

📝 Commits (10+)

85a528f feat: right click to open view more actions
b03d381 test: fix tests
5a8ac17 test: fix failing test
c9a045e fix: hide in move to page menu
c74c859 test: add simple test
4bae25b test: add negative case in cloud test
c2efeef chore: check code related to sidebar spaces
9730101 chore: merge branch 'upstream/main' into feat/view-more-right-click
1c9aef9 fix: view item
b4feb48 chore: merge branch 'upstream/main' into feat/view-more-right-click

📊 Changes

21 files changed (+538 additions, -486 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/cloud/sidebar/sidebar_move_page_test.dart (+9 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_deletion_test.dart (+4 -2)
📝 frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_test.dart (+1 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_test_runner.dart (+2 -0)
➕ frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_view_item_test.dart (+39 -0)
📝 frontend/appflowy_flutter/integration_test/shared/base.dart (+1 -6)
📝 frontend/appflowy_flutter/integration_test/shared/common_operations.dart (+6 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/favorites/favorite_folder.dart (+46 -63)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/favorites/favorite_more_actions.dart (+10 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/_section_folder.dart (+20 -31)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/move_to/move_page_menu.dart (+26 -44)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart (+19 -37)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/shared_widget.dart (+4 -7)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space.dart (+39 -46)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/draggable_view_item.dart (+3 -10)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart (+172 -176)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_more_action_button.dart (+10 -15)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/pop_up_action.dart (+8 -5)
📝 frontend/appflowy_flutter/packages/appflowy_popover/lib/src/layout.dart (+57 -5)
📝 frontend/appflowy_flutter/packages/appflowy_popover/lib/src/popover.dart (+51 -26)

...and 1 more files

📄 Description

Closes: #3135

I want to add a few tests for this before merging - thus still WIP!

Feature Preview

https://github.com/user-attachments/assets/6fafd0e8-9c60-41a1-8040-c020ce9d359e

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/6529 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 10/10/2024 **Status:** ✅ Merged **Merged:** 10/17/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `feat/view-more-right-click` --- ### 📝 Commits (10+) - [`85a528f`](https://github.com/AppFlowy-IO/AppFlowy/commit/85a528ff435963bc11a2495128a1eb48eebd7f5d) feat: right click to open view more actions - [`b03d381`](https://github.com/AppFlowy-IO/AppFlowy/commit/b03d381be5377a9efbe27947c0588b2f171efd0c) test: fix tests - [`5a8ac17`](https://github.com/AppFlowy-IO/AppFlowy/commit/5a8ac17e6126b3e3b9b4108a5f4c0164dce4656f) test: fix failing test - [`c9a045e`](https://github.com/AppFlowy-IO/AppFlowy/commit/c9a045e1645e59ea478133d1c0713fd967869413) fix: hide in move to page menu - [`c74c859`](https://github.com/AppFlowy-IO/AppFlowy/commit/c74c8590c227d3644bcd1d4fc7f6288221007374) test: add simple test - [`4bae25b`](https://github.com/AppFlowy-IO/AppFlowy/commit/4bae25b4d2ecc51fee2f5636909c204058d3086f) test: add negative case in cloud test - [`c2efeef`](https://github.com/AppFlowy-IO/AppFlowy/commit/c2efeefbf5453f29db9bc6c5409b3ebcb70ff58f) chore: check code related to sidebar spaces - [`9730101`](https://github.com/AppFlowy-IO/AppFlowy/commit/9730101dd6b6ba33a3aa0bc19e69f76f76487fbd) chore: merge branch 'upstream/main' into feat/view-more-right-click - [`1c9aef9`](https://github.com/AppFlowy-IO/AppFlowy/commit/1c9aef9c86f7c130aea1bde0fc15a506a695f34b) fix: view item - [`b4feb48`](https://github.com/AppFlowy-IO/AppFlowy/commit/b4feb481dc07d0efb6f8226040a8c463e823eb96) chore: merge branch 'upstream/main' into feat/view-more-right-click ### 📊 Changes **21 files changed** (+538 additions, -486 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/cloud/sidebar/sidebar_move_page_test.dart` (+9 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_deletion_test.dart` (+4 -2) 📝 `frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_test.dart` (+1 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_test_runner.dart` (+2 -0) ➕ `frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_view_item_test.dart` (+39 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/base.dart` (+1 -6) 📝 `frontend/appflowy_flutter/integration_test/shared/common_operations.dart` (+6 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/favorites/favorite_folder.dart` (+46 -63) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/favorites/favorite_more_actions.dart` (+10 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/_section_folder.dart` (+20 -31) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/move_to/move_page_menu.dart` (+26 -44) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart` (+19 -37) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/shared_widget.dart` (+4 -7) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space.dart` (+39 -46) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/draggable_view_item.dart` (+3 -10) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart` (+172 -176) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_more_action_button.dart` (+10 -15) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/pop_up_action.dart` (+8 -5) 📝 `frontend/appflowy_flutter/packages/appflowy_popover/lib/src/layout.dart` (+57 -5) 📝 `frontend/appflowy_flutter/packages/appflowy_popover/lib/src/popover.dart` (+51 -26) _...and 1 more files_ </details> ### 📄 Description Closes: #3135 I want to add a few tests for this before merging - thus still WIP! ### Feature Preview https://github.com/user-attachments/assets/6fafd0e8-9c60-41a1-8040-c020ce9d359e #### 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. - [x] 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>