Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mobile] Context Menu Remains Visible After Text Input or Paste Action #2401

Open
1 task done
AlexeyKatsuro opened this issue Nov 27, 2024 · 6 comments
Open
1 task done
Labels
bug Something isn't working major Significant impact on the user experience or performance. This issue should be addressed in the next regression For issues where a previously working feature is now broken due to recent changes.

Comments

@AlexeyKatsuro
Copy link

AlexeyKatsuro commented Nov 27, 2024

Is there an existing issue for this?

Flutter Quill version

10.8.5

Steps to reproduce

Scenarios:

Scenario 1: Entering New Text

  1. Type text in the input field.
  2. Select a portion of the text or double-tap it to display the context menu.
  3. Start typing new text.

Scenario 2: Using the "Paste" Action

  1. Type text in the input field.
  2. Select a portion of the text or double-tap it to display the context menu.
  3. Tap the "Paste" option in the context menu.

Expected results

The context menu should close automatically after new text is entered or the "Paste" action is performed.

Actual results

The context menu remains visible, obstructing the user experience.

Additional Context

Similar as #2111

Screenshots / Video demonstration Actual result
Screen.Recording.2024-11-27.at.10.mp4

Expected results

Screen.Recording.2024-11-27.at.10.1.mp4
@AlexeyKatsuro AlexeyKatsuro added the bug Something isn't working label Nov 27, 2024
@EchoEllet
Copy link
Collaborator

Is this a regression or related to #2026?

@AlexeyKatsuro
Copy link
Author

@EchoEllet I'm not certain. The original issue (#2111) was marked as resolved after PR #2116, which was merged after the PR you referenced (#2026).

@EchoEllet
Copy link
Collaborator

The magnifier feature has some other regressions that have not been solved yet. We will need to try older versions.

@AlexeyKatsuro
Copy link
Author

If you can specify the version to test on, I will do it.

@EchoEllet
Copy link
Collaborator

Try with 9.4.0, ensure to remove the caret (^) to force using this version:

flutter_quill: 9.4.0

Or:

$ git clone --depth 1 --branch v9.4.0 https://github.com/singerdmx/flutter-quill.git
$ (cd flutter-quill/example && flutter run)

@AlexeyKatsuro
Copy link
Author

The issue does not reproduce on this version

@EchoEllet EchoEllet added regression For issues where a previously working feature is now broken due to recent changes. major Significant impact on the user experience or performance. This issue should be addressed in the next labels Nov 27, 2024
@EchoEllet EchoEllet mentioned this issue Dec 11, 2024
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working major Significant impact on the user experience or performance. This issue should be addressed in the next regression For issues where a previously working feature is now broken due to recent changes.
Projects
None yet
Development

No branches or pull requests

2 participants