[PR #7015] [MERGED] feat: support recent section in icon picker #7687

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: feat/icons

📝 Commits (5)

694abea fix: scroll event will be intercepted by tooltips
1909e49 fix: improve the logic to filter emojis or icons by keyword
570b7fd feat: add the recent icons and emojis to the selector
0cf25f1 refactor: optimize the code
f66f5bc fix: ensure the focus of emoji_search_bar not be lost within a second

📊 Changes

15 files changed (+462 additions, -265 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/database/database_row_page_test.dart (+9 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_cover_image_test.dart (+9 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_icon_test.dart (+62 -2)
📝 frontend/appflowy_flutter/lib/core/config/kv_keys.dart (+5 -0)
📝 frontend/appflowy_flutter/lib/plugins/base/emoji/emoji_picker.dart (+42 -15)
➖ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/icon/icon_selector.dart (+0 -160)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/emoji_search_bar.dart (+16 -0)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/flowy_icon_emoji_picker.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon.dart (+4 -1)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_picker.dart (+85 -56)
➕ frontend/appflowy_flutter/lib/shared/icon_emoji_picker/recent_icons.dart (+95 -0)
📝 frontend/appflowy_flutter/lib/startup/tasks/app_widget.dart (+38 -26)
📝 frontend/appflowy_flutter/pubspec.lock (+2 -2)
📝 frontend/appflowy_flutter/pubspec.yaml (+1 -1)
➕ frontend/appflowy_flutter/test/unit_test/util/recent_icons_test.dart (+93 -0)

📄 Description

Feature Preview

Add the recent icons and emojis to the selector

Recent Emoji	Recent Icon

Bug fixes

Make the Tooltip corresponding to Emojis appear above in the selector.
Dismiss Tooltip while scrolling
Improve the logic to filter emojis or icons by keyword (#6248)
The focus disappears after the Emoji Picker is opened

[Filter Emoji] Before:

https://github.com/user-attachments/assets/449dcce0-b5d4-41e7-8cb4-7a6240effcc2

[Filter Emoji] After:

https://github.com/user-attachments/assets/637d344c-776d-43e9-a69f-c98757eb9397

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/7015 **Author:** [@asjqkkkk](https://github.com/asjqkkkk) **Created:** 12/19/2024 **Status:** ✅ Merged **Merged:** 12/26/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `feat/icons` --- ### 📝 Commits (5) - [`694abea`](https://github.com/AppFlowy-IO/AppFlowy/commit/694abea114fe98606f8f4ddf2b8a499f6bf0f7ee) fix: scroll event will be intercepted by tooltips - [`1909e49`](https://github.com/AppFlowy-IO/AppFlowy/commit/1909e497de9c3ee1310d714b80e0274d52e2e8ee) fix: improve the logic to filter emojis or icons by keyword - [`570b7fd`](https://github.com/AppFlowy-IO/AppFlowy/commit/570b7fd39c757e0a66f8602ad809a6fef0bdfb55) feat: add the recent icons and emojis to the selector - [`0cf25f1`](https://github.com/AppFlowy-IO/AppFlowy/commit/0cf25f1ab9ec1432469cd829a14e68315a0f1f48) refactor: optimize the code - [`f66f5bc`](https://github.com/AppFlowy-IO/AppFlowy/commit/f66f5bc25944fefcefbd123af0f3fb4d56482ffe) fix: ensure the focus of emoji_search_bar not be lost within a second ### 📊 Changes **15 files changed** (+462 additions, -265 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_row_page_test.dart` (+9 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_cover_image_test.dart` (+9 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/sidebar/sidebar_icon_test.dart` (+62 -2) 📝 `frontend/appflowy_flutter/lib/core/config/kv_keys.dart` (+5 -0) 📝 `frontend/appflowy_flutter/lib/plugins/base/emoji/emoji_picker.dart` (+42 -15) ➖ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/icon/icon_selector.dart` (+0 -160) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/emoji_search_bar.dart` (+16 -0) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/flowy_icon_emoji_picker.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon.dart` (+4 -1) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_picker.dart` (+85 -56) ➕ `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/recent_icons.dart` (+95 -0) 📝 `frontend/appflowy_flutter/lib/startup/tasks/app_widget.dart` (+38 -26) 📝 `frontend/appflowy_flutter/pubspec.lock` (+2 -2) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+1 -1) ➕ `frontend/appflowy_flutter/test/unit_test/util/recent_icons_test.dart` (+93 -0) </details> ### 📄 Description ### Feature Preview Add the recent icons and emojis to the selector Recent Emoji | Recent Icon ---|--- ![image](https://github.com/user-attachments/assets/c46b2650-1990-40b5-92df-918fc7d9767b)| ![image](https://github.com/user-attachments/assets/d93250c2-0c72-467b-9df2-78b76f8f92e1) ### Bug fixes - Make the Tooltip corresponding to Emojis appear above in the selector. - Dismiss Tooltip while scrolling - Improve the logic to filter emojis or icons by keyword (#6248) - The focus disappears after the Emoji Picker is opened [Filter Emoji] Before: https://github.com/user-attachments/assets/449dcce0-b5d4-41e7-8cb4-7a6240effcc2 [Filter Emoji] After: https://github.com/user-attachments/assets/637d344c-776d-43e9-a69f-c98757eb9397 ---  #### 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>