[PR #7845] [MERGED] fix: some mobile search UI issues #8158

New issue

Closed

opened 2026-03-23 23:22:52 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:22:52 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7845
Author: @asjqkkkk
Created: 4/28/2025
Status: ✅ Merged
Merged: 4/29/2025
Merged by: @LucasXu0

Base: main ← Head: fix/mobile/lr

📝 Commits (10+)

ea46bf3 fix: some mobile search UI issues
ea3ea5c fix: some icon colors
2401c8f fix: show 'Add Link' while creating a link on mobile
1e1c749 fix: add tapping event for mobile embed link
83dd72d fix: bookmark can not open the non-http link
7d1446a fix: improve the behavior of links on mobile
284e9b3 fix: switch workspace will refresh the reminders
4ef8b56 Merge branch 'main' into fix/mobile/lr
8af33ba chore: remove unused import
5e20b46 Merge branch 'main' into fix/mobile/lr

📊 Changes

17 files changed (+224 additions, -175 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/cloud/uncategorized/appflowy_cloud_auth_test.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_edit_link_widget.dart (+3 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/search/mobile_search_cell.dart (+29 -20)
📝 frontend/appflowy_flutter/lib/mobile/presentation/search/mobile_search_textfield.dart (+11 -6)
📝 frontend/appflowy_flutter/lib/mobile/presentation/selection_menu/mobile_selection_menu_widget.dart (+6 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/link_embed/link_embed_block_component.dart (+60 -42)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/link_preview/custom_link_preview.dart (+4 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/link_preview/custom_link_preview_block_component.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/mention_link_block.dart (+43 -39)
📝 frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart (+7 -31)
📝 frontend/appflowy_flutter/lib/workspace/application/user/user_workspace_bloc.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/header/sidebar_top_menu.dart (+7 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart (+6 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_panel.dart (+32 -25)
➕ frontend/resources/flowy_icons/24x/double_back_arrow.svg (+4 -0)
➕ frontend/resources/flowy_icons/24x/three_dots.svg (+5 -0)

📄 Description

Feature Preview

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.

Summary by Sourcery

Refine the user interface for mobile search results and the selection menu.

Bug Fixes:

Display a default name for search results when the original name is empty.
Correct the alignment and increase the tappable area of the search cancel button.
Ensure the "No results" text in the selection menu uses the correct theme color instead of a hardcoded value.
Center the icon within its allocated space in the search result cell.
Add vertical spacing before the summary text in search result cells for better readability.
Adjust the height of the mobile search text field.

_{🔄 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/7845 **Author:** [@asjqkkkk](https://github.com/asjqkkkk) **Created:** 4/28/2025 **Status:** ✅ Merged **Merged:** 4/29/2025 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix/mobile/lr` --- ### 📝 Commits (10+) - [`ea46bf3`](https://github.com/AppFlowy-IO/AppFlowy/commit/ea46bf3aaca4cb811875496ac2b4a2803cabf6c0) fix: some mobile search UI issues - [`ea3ea5c`](https://github.com/AppFlowy-IO/AppFlowy/commit/ea3ea5ca2f2e5baebef88de6f92d15785196cb06) fix: some icon colors - [`2401c8f`](https://github.com/AppFlowy-IO/AppFlowy/commit/2401c8f91bdfffc367be85e03a529ac6e0b7eea9) fix: show 'Add Link' while creating a link on mobile - [`1e1c749`](https://github.com/AppFlowy-IO/AppFlowy/commit/1e1c749d9a5d1adde5f75f5b0327089c2994d961) fix: add tapping event for mobile embed link - [`83dd72d`](https://github.com/AppFlowy-IO/AppFlowy/commit/83dd72d8e08b8adabf50c8dd8a53e1a45fd40670) fix: bookmark can not open the non-http link - [`7d1446a`](https://github.com/AppFlowy-IO/AppFlowy/commit/7d1446ab32966095ee60c41e2b549472af364cbe) fix: improve the behavior of links on mobile - [`284e9b3`](https://github.com/AppFlowy-IO/AppFlowy/commit/284e9b333cc03fb50062005a024cfc85b23aa9e9) fix: switch workspace will refresh the reminders - [`4ef8b56`](https://github.com/AppFlowy-IO/AppFlowy/commit/4ef8b568e80e230a3838ea893e75866d29df937b) Merge branch 'main' into fix/mobile/lr - [`8af33ba`](https://github.com/AppFlowy-IO/AppFlowy/commit/8af33ba0e356be8cabca97c46502a9f33f285d4b) chore: remove unused import - [`5e20b46`](https://github.com/AppFlowy-IO/AppFlowy/commit/5e20b4603cab43cccdfb81f9130c3727f6473d85) Merge branch 'main' into fix/mobile/lr ### 📊 Changes **17 files changed** (+224 additions, -175 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/cloud/uncategorized/appflowy_cloud_auth_test.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_edit_link_widget.dart` (+3 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/search/mobile_search_cell.dart` (+29 -20) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/search/mobile_search_textfield.dart` (+11 -6) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/selection_menu/mobile_selection_menu_widget.dart` (+6 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/link_embed/link_embed_block_component.dart` (+60 -42) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/link_preview/custom_link_preview.dart` (+4 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/link_preview/custom_link_preview_block_component.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/mention_link_block.dart` (+43 -39) 📝 `frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart` (+7 -31) 📝 `frontend/appflowy_flutter/lib/workspace/application/user/user_workspace_bloc.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/header/sidebar_top_menu.dart` (+7 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart` (+6 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_panel.dart` (+32 -25) ➕ `frontend/resources/flowy_icons/24x/double_back_arrow.svg` (+4 -0) ➕ `frontend/resources/flowy_icons/24x/three_dots.svg` (+5 -0) </details> ### 📄 Description   ### Feature Preview  ---  #### 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. - [ ] All existing tests are passing. ## Summary by Sourcery Refine the user interface for mobile search results and the selection menu. Bug Fixes: - Display a default name for search results when the original name is empty. - Correct the alignment and increase the tappable area of the search cancel button. - Ensure the "No results" text in the selection menu uses the correct theme color instead of a hardcoded value. - Center the icon within its allocated space in the search result cell. - Add vertical spacing before the summary text in search result cells for better readability. - Adjust the height of the mobile search text field. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>