-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Text selection located wrong position when selecting multiple lines over max lines #96471
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
Conversation
|
Gold has detected about 2 new digest(s) on patchset 2. |
|
Gold has detected about 2 new digest(s) on patchset 3. |
chunhtai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
cc @justinmc for one more approval |
justinmc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem to work in every case. I tried it when the field is vertically centered in the app and the toolbar shows up in the wrong place:
Code
import 'package:flutter/material.dart';
void main() => runApp(const MyApp());
class MyApp extends StatelessWidget {
const MyApp({
Key? key,
}) : super(key: key);
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Flutter Demo',
theme: ThemeData(
primarySwatch: Colors.blue,
),
home: Scaffold(
appBar: AppBar(
title: const Text('Flutter Demo'),
),
body: const Center(
child: TextField(
maxLines: 2,
),
),
),
);
}
}Unfortunately our existing tests didn't catch that.
|
@SuhwanCha I'm going to close this since it's been stale for awhile and has merge conflicts. If you still want to fix this bug, feel free to let me know and I'm happy to help with reviews. |

Fixes #96453
This PR fixes bugs that text selection located wrong position when selecting multiple lines over max lines.
Pre-launch Checklist
///).