Skip to content

[flutter_markdown] Adds onSelectionChanged in Markdown - #6169

Merged
auto-submit[bot] merged 8 commits into
flutter:mainfrom
leiatfly:main
Mar 18, 2024
Merged

auto-submit[bot] merged 8 commits into
flutter:mainfrom
leiatfly:main

Conversation

@leiatfly

@leiatfly leiatfly commented Feb 21, 2024 •

Copy link
Copy Markdown
Contributor

With this PR, a client can use 'onSelectionChanged' to handle selected text in Markdown.

This provides another way to fix issue #107073, along with the prior solution in PR/6062

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@leiatfly leiatfly changed the title Adds onSelectionChanged in Markdown [flutter_markdown] Adds onSelectionChanged in Markdown Feb 21, 2024

@domesticmouse domesticmouse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tarrinneal tarrinneal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two really minor comment changes and this is good to go!

@tarrinneal tarrinneal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to fix nits, and fix conflicts.

@tarrinneal tarrinneal added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 18, 2024
@auto-submit
auto-submit Bot merged commit 1b9bab3 into flutter:main Mar 18, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2024
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Mar 19, 2024
flutter/packages@a757073...a2f4ce0

2024-03-18 49699333+dependabot[bot]@users.noreply.github.com [sign_in]: Bump com.google.android.gms:play-services-auth from 20.7.0 to 21.0.0 in /packages/google_sign_in/google_sign_in_android/android (flutter/packages#6159)
2024-03-18 engine-flutter-autoroll@skia.org Roll Flutter from 7c860dd to f217fc1 (9 revisions) (flutter/packages#6350)
2024-03-18 16229121+leiatfly@users.noreply.github.com [flutter_markdown] Adds onSelectionChanged in Markdown (flutter/packages#6169)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
With this PR, a client can use 'onSelectionChanged' to handle selected text in Markdown.

This provides another way to fix issue [#107073](flutter/flutter#107073), along with the prior solution in [PR/6062](flutter#6062)
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
With this PR, a client can use 'onSelectionChanged' to handle selected text in Markdown.

This provides another way to fix issue [#107073](flutter/flutter#107073), along with the prior solution in [PR/6062](flutter#6062)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App p: flutter_markdown

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants