[PR #7837] [MERGED] fix: mobile link LR issues #8151

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: fix/mobile/link

📝 Commits (2)

5aa5fd4 fix: mobile link LR issues
c03f1ba fix: change mobile toolbar link icon

📊 Changes

5 files changed (+14 additions, -5 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/core/helpers/url_launcher.dart (+3 -2)
📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_edit_link_widget.dart (+7 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/desktop_toolbar/link/link_search_text_field.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mobile_toolbar_v3/basic_toolbar_item.dart (+1 -1)
📝 frontend/resources/flowy_icons/24x/m_toolbar_link.svg (+1 -1)

📄 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

Fix link handling and interaction issues on mobile.

Bug Fixes:

Ensure URLs without a scheme are correctly opened on Android by prepending "https://".
Improve tap target areas for link search results and remove link buttons in the mobile bottom sheet.
Apply consistent text styling to the link input field in the mobile bottom sheet.

_{🔄 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/7837 **Author:** [@asjqkkkk](https://github.com/asjqkkkk) **Created:** 4/26/2025 **Status:** ✅ Merged **Merged:** 4/28/2025 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix/mobile/link` --- ### 📝 Commits (2) - [`5aa5fd4`](https://github.com/AppFlowy-IO/AppFlowy/commit/5aa5fd4afdc6005430e4f14a5d584163fd9ba44c) fix: mobile link LR issues - [`c03f1ba`](https://github.com/AppFlowy-IO/AppFlowy/commit/c03f1ba0b917cc394508af1889635aecb57d940a) fix: change mobile toolbar link icon ### 📊 Changes **5 files changed** (+14 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/core/helpers/url_launcher.dart` (+3 -2) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_edit_link_widget.dart` (+7 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/desktop_toolbar/link/link_search_text_field.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mobile_toolbar_v3/basic_toolbar_item.dart` (+1 -1) 📝 `frontend/resources/flowy_icons/24x/m_toolbar_link.svg` (+1 -1) </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. - [x] All existing tests are passing. ## Summary by Sourcery Fix link handling and interaction issues on mobile. Bug Fixes: - Ensure URLs without a scheme are correctly opened on Android by prepending "https://". - Improve tap target areas for link search results and remove link buttons in the mobile bottom sheet. - Apply consistent text styling to the link input field in the mobile bottom sheet. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>