[PR #7747] [MERGED] fix: mobile slash menu position error #8084

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: fix/mobile/slash_menu

📝 Commits (2)

6d253ea fix: mobile slash menu position error
7ce99f9 fix: mobile slash menu sometimes not showing up

📊 Changes

3 files changed (+49 additions, -34 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/mobile/presentation/selection_menu/mobile_selection_menu.dart (+46 -31)
📝 frontend/appflowy_flutter/pubspec.lock (+2 -2)
📝 frontend/appflowy_flutter/pubspec.yaml (+1 -1)

📄 Description

Fix:

Mobile slash menu position error
Sometimes mobile slash menu not showing up

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

Improve mobile selection menu positioning to handle different screen sizes and selection locations more robustly

Bug Fixes:

Fix the positioning of the mobile slash menu to correctly adapt to different screen sizes and selection locations

Enhancements:

Enhance the selection menu positioning logic to handle edge cases near screen boundaries
Improve menu alignment and offset calculations for better user experience

_{🔄 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/7747 **Author:** [@asjqkkkk](https://github.com/asjqkkkk) **Created:** 4/15/2025 **Status:** ✅ Merged **Merged:** 4/15/2025 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix/mobile/slash_menu` --- ### 📝 Commits (2) - [`6d253ea`](https://github.com/AppFlowy-IO/AppFlowy/commit/6d253ea40652aa7ff471d4cc618982360d054514) fix: mobile slash menu position error - [`7ce99f9`](https://github.com/AppFlowy-IO/AppFlowy/commit/7ce99f966a67481632e42d9ddcf10b129565b8ad) fix: mobile slash menu sometimes not showing up ### 📊 Changes **3 files changed** (+49 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/mobile/presentation/selection_menu/mobile_selection_menu.dart` (+46 -31) 📝 `frontend/appflowy_flutter/pubspec.lock` (+2 -2) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+1 -1) </details> ### 📄 Description Fix: - Mobile slash menu position error - Sometimes mobile slash menu not showing up ### Feature Preview  ---  #### 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. ## Summary by Sourcery Improve mobile selection menu positioning to handle different screen sizes and selection locations more robustly Bug Fixes: - Fix the positioning of the mobile slash menu to correctly adapt to different screen sizes and selection locations Enhancements: - Enhance the selection menu positioning logic to handle edge cases near screen boundaries - Improve menu alignment and offset calculations for better user experience --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>