Fix selection not deselected when TextField loses focus #103424
Conversation
…no selection is painted when no selectionColor is passed
| @@ -660,7 +660,8 @@ void main() { | |||
| expect(focusNode.hasFocus, isFalse); | |||
| }); | |||
|
|
|||
| testWidgets('use DefaultSelectionStyle for selection color', (WidgetTester tester) async { | |||
| testWidgets('selection color can be null with DefaultSelectionStyle as parent', (WidgetTester tester) async { | |||
There was a problem hiding this comment.
Maybe rename to EditableText does not derive selection color from DefaultSelectionStyle
| @@ -433,6 +433,62 @@ void main() { | |||
| }, | |||
| ); | |||
|
|
|||
| testWidgets('Text field drops selection color when losing focus', (WidgetTester tester) async { | |||
There was a problem hiding this comment.
Nit: Is it relevant to say "Regression test for #103341" here and elsewhere?
There was a problem hiding this comment.
LGTM 👍
Just as a thought, what if we passed a transparent Color as selectionColor instead of null?
That works too. Though if someone explicitly passes null as selectionColor to EditableText. I think they would expect no selection to be painted, where as now it defaults to the color defined by DefaultSelectionStyle.of(context).
There was a problem hiding this comment.
I don't have a strong opinion on this, but if I were to redesign this again, passing null as color means disabling the selection seems to be a weird API, In that case I would probably throw if the color is null
There was a problem hiding this comment.
I agree, though another small difference between null and Colors.transparent is that the TextHighlightPainter skips painting all together when the selectionColor is null vs when it is Colors.transparent it will still paint some values.
Description
The original issue was introduced by #101954 because the
selectionColoronEditableTextwas never null. Previously null was passed toEditableTextto paint no selection when it did not have focus.flutter/packages/flutter/lib/src/material/text_field.dart
Line 1244 in 5464c5b
EditableTextto be null again by only takingwidget.selectionColorand not inheritingDefaultSelectionStyle.of(context).Related Issues
Fixes #103341
Tests
Added tests to ensure selectionColor is dropped when focus is lost on TextField, and add regression test.
Pre-launch Checklist
///).