[PR #6998] [MERGED] feat: combine icons and emojis into one menu(#6806) #7678

New issue

Closed

opened 2026-03-23 23:20:43 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:20:43 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6998
Author: @asjqkkkk
Created: 12/17/2024
Status: ✅ Merged
Merged: 12/19/2024
Merged by: @LucasXu0

Base: main ← Head: br_6806

📝 Commits (1)

1fe31f0 feat: combine icons and emojis into one menu(#6806)

📊 Changes

55 files changed (+825 additions, -418 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/document/document_find_menu_test.dart (+0 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_cover_image_test.dart (+2 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/settings/sign_in_page_settings_test.dart (+1 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_icon_test.dart (+154 -68)
📝 frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_view_item_test.dart (+6 -1)
📝 frontend/appflowy_flutter/integration_test/shared/common_operations.dart (+13 -4)
📝 frontend/appflowy_flutter/integration_test/shared/emoji.dart (+51 -0)
📝 frontend/appflowy_flutter/integration_test/shared/expectation.dart (+46 -13)
📝 frontend/appflowy_flutter/lib/mobile/application/recent/recent_view_bloc.dart (+13 -8)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/mobile_view_page.dart (+7 -6)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page_header.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/recent_folder/mobile_recent_view.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/shared/mobile_page_card.dart (+10 -12)
📝 frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart (+7 -5)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_input/chat_mention_page_menu.dart (+5 -4)
📝 frontend/appflowy_flutter/lib/plugins/base/emoji/emoji_picker_screen.dart (+24 -7)
➖ frontend/appflowy_flutter/lib/plugins/base/icon/icon_picker_page.dart (+0 -28)
➕ frontend/appflowy_flutter/lib/plugins/base/icon/icon_widget.dart (+29 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/card/container/card_container.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/field/field_editor.dart (+18 -9)

...and 35 more files

📄 Description

Feature Preview

For handling issue #6806

Desktop

Sidebar Display and Modification
Article - Table of Contents - Display and Modification
Article - Title - Display and Modification
Setting - Move To - Display and Modification
Linked Page - Display and Modification
Search Articles - Display

Mobile

Space - Display
Linked Page -Display
Emoji Icon - Display
Recent articles - Display
Icon Selector

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/6998 **Author:** [@asjqkkkk](https://github.com/asjqkkkk) **Created:** 12/17/2024 **Status:** ✅ Merged **Merged:** 12/19/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `br_6806` --- ### 📝 Commits (1) - [`1fe31f0`](https://github.com/AppFlowy-IO/AppFlowy/commit/1fe31f0b84625f75c38c822bfcd824ec763448d8) feat: combine icons and emojis into one menu(#6806) ### 📊 Changes **55 files changed** (+825 additions, -418 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_find_menu_test.dart` (+0 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_cover_image_test.dart` (+2 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/settings/sign_in_page_settings_test.dart` (+1 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_icon_test.dart` (+154 -68) 📝 `frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_view_item_test.dart` (+6 -1) 📝 `frontend/appflowy_flutter/integration_test/shared/common_operations.dart` (+13 -4) 📝 `frontend/appflowy_flutter/integration_test/shared/emoji.dart` (+51 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/expectation.dart` (+46 -13) 📝 `frontend/appflowy_flutter/lib/mobile/application/recent/recent_view_bloc.dart` (+13 -8) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/mobile_view_page.dart` (+7 -6) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page_header.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/recent_folder/mobile_recent_view.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/shared/mobile_page_card.dart` (+10 -12) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart` (+7 -5) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_input/chat_mention_page_menu.dart` (+5 -4) 📝 `frontend/appflowy_flutter/lib/plugins/base/emoji/emoji_picker_screen.dart` (+24 -7) ➖ `frontend/appflowy_flutter/lib/plugins/base/icon/icon_picker_page.dart` (+0 -28) ➕ `frontend/appflowy_flutter/lib/plugins/base/icon/icon_widget.dart` (+29 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/card/container/card_container.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/field/field_editor.dart` (+18 -9) _...and 35 more files_ </details> ### 📄 Description ### Feature Preview For handling issue #6806 #### Desktop 1. Sidebar Display and Modification ![image](https://github.com/user-attachments/assets/42961e61-1968-46ee-854b-3eaa81c4bf3a) 2. Article - Table of Contents - Display and Modification ![image](https://github.com/user-attachments/assets/3e76678d-0500-4166-a969-2387d7526db2) 3. Article - Title - Display and Modification ![image](https://github.com/user-attachments/assets/8cb3c02c-bcb1-4009-897d-b515a4c6172d) 4. Setting - Move To - Display and Modification ![image](https://github.com/user-attachments/assets/d83356c0-1386-4b30-b080-a17674a64121) 5. Linked Page - Display and Modification ![image](https://github.com/user-attachments/assets/ee2fd6f8-7b85-402a-a67a-ca9a2fd448e2) 6. Search Articles - Display ![image](https://github.com/user-attachments/assets/cd10008d-e867-4560-b8d0-d29781933747) #### Mobile 1. Space - Display ![image](https://github.com/user-attachments/assets/10140e25-d965-4395-8671-a92e8ab5a2c2) 2. Linked Page -Display ![image](https://github.com/user-attachments/assets/97e8ea34-4ff8-4090-babe-d65e2a87c954) 3. Emoji Icon - Display ![image](https://github.com/user-attachments/assets/6434c5d7-7a40-4625-abb1-51d44ca8be91) 4. Recent articles - Display ![image](https://github.com/user-attachments/assets/c5bcb9db-b838-4e5a-8e8d-eb6e36ef046a) 5. Icon Selector ![image](https://github.com/user-attachments/assets/b7a3726b-3718-424c-b95c-a2f07468c837) #### 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>