[PR #5889] [MERGED] feat: optimize editor toolbar tooltip #7068

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5889
Author: @LucasXu0
Created: 8/6/2024
Status: ✅ Merged
Merged: 8/7/2024
Merged by: @LucasXu0

Base: main ← Head: optimize_editor_tooltip

📝 Commits (10+)

b937909 chore: improve icon picker dark mode color
da23928 feat: optimize editor toolbar tooltip style
a20d436 feat: customize markdown item tooltip message
4df54f8 chore: find the tooltip by rich message
8403203 feat: add hover effect in toolbar item
dde9d1a feat: add hover effect in toolbar item
6df1dbc chore: optimize hover color in light mode
08d6e26 chore: fix integration test
56e8b19 chore: optimize align & font toolbar item
9344791 chore: fix integration test

📊 Changes

19 files changed (+293 additions, -233 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_inline_math_equation_test.dart (+7 -7)
➖ frontend/appflowy_flutter/integration_test/desktop/uncategorized/open_ai_smart_menu_test.dart (+0 -113)
📝 frontend/appflowy_flutter/integration_test/shared/base.dart (+12 -3)
📝 frontend/appflowy_flutter/integration_test/shared/editor_test_operations.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/base/emoji/emoji_picker.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+2 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/align_toolbar_item/align_toolbar_item.dart (+48 -39)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/font/customize_font_toolbar_item.dart (+48 -40)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/inline_math_equation/inline_math_equation_toolbar_item.dart (+16 -4)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/openai/widgets/smart_edit_toolbar_item.dart (+21 -6)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_style.dart (+85 -0)
➕ frontend/appflowy_flutter/lib/shared/icon_emoji_picker/colors.dart (+26 -0)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/emoji_search_bar.dart (+5 -3)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/emoji_skin_tone.dart (+3 -1)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_picker.dart (+3 -2)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_search_bar.dart (+4 -2)
📝 frontend/appflowy_flutter/packages/flowy_infra_ui/lib/widget/flowy_tooltip.dart (+8 -8)
📝 frontend/appflowy_flutter/pubspec.lock (+2 -2)
📝 frontend/appflowy_flutter/pubspec.yaml (+1 -1)

📄 Description

Feature Preview

Use AppFlowy style tooltip in toolbar item
Add hover effect to toolbar item

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/5889 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 8/6/2024 **Status:** ✅ Merged **Merged:** 8/7/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `optimize_editor_tooltip` --- ### 📝 Commits (10+) - [`b937909`](https://github.com/AppFlowy-IO/AppFlowy/commit/b9379094c4314d82ec71771cfd518c388feea4ac) chore: improve icon picker dark mode color - [`da23928`](https://github.com/AppFlowy-IO/AppFlowy/commit/da23928342126cd3b57be90e14f7a6945c31eda5) feat: optimize editor toolbar tooltip style - [`a20d436`](https://github.com/AppFlowy-IO/AppFlowy/commit/a20d436886f1f418187efecb7e50c1da7d98923d) feat: customize markdown item tooltip message - [`4df54f8`](https://github.com/AppFlowy-IO/AppFlowy/commit/4df54f8aae33b47a81fd00e689cebb61c3ef9ea9) chore: find the tooltip by rich message - [`8403203`](https://github.com/AppFlowy-IO/AppFlowy/commit/84032038b4a63a24b0f4b5415069d16341f7a932) feat: add hover effect in toolbar item - [`dde9d1a`](https://github.com/AppFlowy-IO/AppFlowy/commit/dde9d1a64fc8044b6306c215d95b50f8619e3c4b) feat: add hover effect in toolbar item - [`6df1dbc`](https://github.com/AppFlowy-IO/AppFlowy/commit/6df1dbc863b69f7fe3329fd43d884c7a362fb924) chore: optimize hover color in light mode - [`08d6e26`](https://github.com/AppFlowy-IO/AppFlowy/commit/08d6e261f755781d254d53d3b238f06d4541cc2a) chore: fix integration test - [`56e8b19`](https://github.com/AppFlowy-IO/AppFlowy/commit/56e8b19d2c5115f5743ceb2b7c211033e87caf06) chore: optimize align & font toolbar item - [`9344791`](https://github.com/AppFlowy-IO/AppFlowy/commit/934479165f441f784efcc4ac1af6595a31844e6e) chore: fix integration test ### 📊 Changes **19 files changed** (+293 additions, -233 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_inline_math_equation_test.dart` (+7 -7) ➖ `frontend/appflowy_flutter/integration_test/desktop/uncategorized/open_ai_smart_menu_test.dart` (+0 -113) 📝 `frontend/appflowy_flutter/integration_test/shared/base.dart` (+12 -3) 📝 `frontend/appflowy_flutter/integration_test/shared/editor_test_operations.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/base/emoji/emoji_picker.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+2 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/align_toolbar_item/align_toolbar_item.dart` (+48 -39) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/font/customize_font_toolbar_item.dart` (+48 -40) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/inline_math_equation/inline_math_equation_toolbar_item.dart` (+16 -4) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/openai/widgets/smart_edit_toolbar_item.dart` (+21 -6) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_style.dart` (+85 -0) ➕ `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/colors.dart` (+26 -0) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/emoji_search_bar.dart` (+5 -3) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/emoji_skin_tone.dart` (+3 -1) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_picker.dart` (+3 -2) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_search_bar.dart` (+4 -2) 📝 `frontend/appflowy_flutter/packages/flowy_infra_ui/lib/widget/flowy_tooltip.dart` (+8 -8) 📝 `frontend/appflowy_flutter/pubspec.lock` (+2 -2) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+1 -1) </details> ### 📄 Description   ### Feature Preview - Use AppFlowy style tooltip in toolbar item - Add hover effect to toolbar item <img width="683" alt="Screenshot 2024-08-07 at 10 32 06" src="https://github.com/user-attachments/assets/cc2dc98c-8b0b-4dd6-ae90-6abdc03f67cc">  ---  #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>