Tomasz Gucio
ca6f23e8af
Remove Editable.onCaretChanged callback ( #109114 )
2023-06-27 22:20:15 +02:00
Luccas Clezar
a90c33fd61
iOS TextSelectionToolbar fidelity ( #127757 )
...
CupertinoTextSelectionToolbar is different from the native one, with some UI and UX issues. More details on the linked issue.
https://github.com/flutter/flutter/issues/127756
Currently the only problem that I listed on the linked issue that I couldn't fix was the horizontal scrolling, but to workaround this I added a GestureDetector to change pages when swiping the toolbar. It's not exactly the same as native as there is no scroll animation, but it works.
I'm creating this PR a little early to have some feedback as these changes were more complex than the ones in my last PR. Probably best if @justinmc is involved ð
|Version|Video|
|-|-|
|Flutter Old|<video src="https://github.com/flutter/flutter/assets/12024080/7cf81075-46ec-4970-b118-cc27b60ddac0 "></video>|
|Flutter New|<video src="https://github.com/flutter/flutter/assets/12024080/c9e27a53-f94c-4cb0-9b76-e47b73841dcb "></video>|
|Native|<video src="https://github.com/flutter/flutter/assets/12024080/468c7d5b-ba93-4bd4-8f6e-8ec2644b9866 "></video>|
2023-06-27 17:50:07 +00:00
hangyu
9459f1c517
Fix chinese text is not selected by long press ( #129320 )
...
issue:#126652
in Chinese text, word length is 1 and (position.offset == word.end) happens a lot.
Update if (position.offset >= word.end) to if (position.offset > word.end) to resolve the issue that Chinese characters are not selected.
2023-06-27 01:15:38 +00:00
Taha Tesser
c8e42b47f5
Fix AnimatedList & AnimatedGrid doesn't apply MediaQuery padding ( #129556 )
...
fixes [AnimatedList does not take SafeArea into account when building the list ](https://github.com/flutter/flutter/issues/129539 )
### Description
This PR fixes an issue for `AnimatedList` & `AnimatedGrid` where `MediaQuery` padding isn't applied. See the [source](a20db068dd/packages/flutter/lib/src/widgets/scroll_view.dart (L803-L833)https://github.com/flutter/flutter/assets/48603081/01917900-cd26-4ca1-8e51-b7dcd1241471 )
The fix applies `MediaQuery` padding to the inner `SliverPadding` in `AnimatedList` & `AnimatedGrid`.
### Code sample
<details>
<summary>expand to view the code sample</summary>
```dart
import 'package:flutter/material.dart';
void main() => runApp(const MyApp());
class MyApp extends StatelessWidget {
const MyApp({super.key});
@override
Widget build(BuildContext context) {
return MaterialApp(
debugShowCheckedModeBanner: false,
theme: ThemeData(useMaterial3: true),
home: const Example(),
);
}
}
class Example extends StatelessWidget {
const Example({super.key});
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: const Text('Sample'),
),
body: Row(
children: <Widget>[
Expanded(
child: Column(
children: <Widget>[
const Text('ListView'),
Expanded(
child: ListView.builder(
itemCount: 50,
itemBuilder: (_, int index) {
return ColoredBox(
color: Theme.of(context).colorScheme.primaryContainer,
child: Center(
child: Text('$index', textAlign: TextAlign.center),
),
);
},
),
),
],
),
),
const VerticalDivider(width: 4),
Expanded(
child: Column(
children: <Widget>[
const Text('AnimatedList'),
Expanded(
child: AnimatedList(
initialItemCount: 50,
itemBuilder: (_, int index, __) {
return ColoredBox(
color: Theme.of(context).colorScheme.primaryContainer,
child: Center(
child: Text('$index', textAlign: TextAlign.center),
),
);
},
),
),
],
),
),
const VerticalDivider(width: 4),
Expanded(
child: Column(
children: <Widget>[
const Text('AnimatedGrid'),
Expanded(
child: AnimatedGrid(
initialItemCount: 50,
gridDelegate: const SliverGridDelegateWithFixedCrossAxisCount(
crossAxisCount: 2,
),
itemBuilder: (_, int index, __) {
return ColoredBox(
color: Theme.of(context).colorScheme.primaryContainer,
child: Center(
child: Text('$index', textAlign: TextAlign.center),
),
);
},
),
),
],
))
],
),
);
}
}
```
</details>
### Before
### After
2023-06-26 16:19:02 +00:00
Loïc Sharma
e1e2ebd176
Add CallbackShortcuts widget of the week video ( #129296 )
2023-06-23 20:16:28 +00:00
Paurakh Sharma Humagain
a9164a4d76
fix: Inconsistency of SelectionArea when scrolling ( #128765 )
...
This PR fixes inconsistency of SelectionArea when scrolling by removing the `_clearSelection` on `longPressUpdate` gesture. This in turn makes the selection UX much better.
*List which issues are fixed by this PR. You must list at least one issue.*
Fixes: https://github.com/flutter/flutter/issues/120892
*If you had to change anything in the [flutter/tests] repo, include a link to the migration guide as per the [breaking change policy].*
2023-06-23 16:47:47 +00:00
Xilai Zhang
363e2f7e0b
Revert "[web] Migrate framework to fully use package:web" ( #129400 )
...
Reverts flutter/flutter#128901
context: ([cl/542712088](https://critique.corp.google.com/542712088 ) , [b/288495136](https://b.corp.google.com/288495136 ))
2023-06-23 04:11:55 +00:00
Renzo Olivares
3c366b7011
_TapStatusTrackerMixin should wait until the next PointerDownEvent before resetting its state when the tap timer has elapsed (#129312 )
...
`_TapStatusTrackerMixin` used by `BaseTapAndDragGestureRecognizer` should wait until the next tap down before resetting its state when the `_consecutiveTapTimer` times out. This is because `BaseTapAndDragGestureRecognizer` may not have fired its tap down/tap up event before the state has been reset preventing it from firing the tap down/tap up callbacks at all because `currentDown` and `currentUp` are reset to `null`.
Fixes #129161
2023-06-23 00:10:32 +00:00
Kate Lovett
087377ea2f
Revert "Fix Material 3 Scrollable TabBar" ( #129383 )
...
Reverts flutter/flutter#125974
2023-06-22 22:34:06 +00:00
Victor Ohashi
39bf968168
fix: SearchAnchor View not resizing when in nested navigator ( #128357 )
...
Similar to what was done on https://github.com/flutter/flutter/pull/127198 , look for the closest navigator instead of screen size.
Fixes: https://github.com/flutter/flutter/issues/128344
2023-06-22 21:20:41 +00:00
Taha Tesser
d99e5fdd2c
Fix Slider overlay remains when unfocused ( #129115 )
...
fixes https://github.com/flutter/flutter/issues/129016
2023-06-22 21:20:40 +00:00
Mouad Debbar
bec24367b0
[web] Migrate framework to fully use package:web ( #128901 )
...
Remove `dom.dart`'s JS interop definitions in favor of `package:web`.
Part of https://github.com/flutter/flutter/issues/113402
Part of https://github.com/flutter/flutter/issues/127030
2023-06-22 21:20:38 +00:00
Qun Cheng
5140b0f0c6
Add onSubmitted property to SearchBar ( #129365 )
...
Fixes #126551
This PR is to add `onSubmitted` property to `SearchBar`.
2023-06-22 21:19:01 +00:00
Ahmed Elsayed
042c0366c9
Remove unnecessary variable _hasPrimaryFocus ( #129066 )
...
`_hasPrimaryFocus` variable and its related code is no longer needed after using `InkWell` for `DropdownButton` at https://github.com/flutter/flutter/pull/95906
2023-06-22 17:58:49 +00:00
Taha Tesser
32fde139bc
Fix Material 3 Scrollable TabBar ( #125974 )
...
fix https://github.com/flutter/flutter/issues/117722
### Description
1. Fix the divider doesn't stretch to take all the available width in the scrollable tab bar in M3
2. Add `dividerHeight` property.
3. Update the default tab alignment for the scrollable tab bar to match the specs (this is backward compatible for M2 with the new `tabAlignment` property).
### Bug (default tab alignment)
### Fix (default tab alignment)
### Code sample
<details>
<summary>code sample</summary>
```dart
import 'package:flutter/material.dart';
/// Flutter code sample for [TabBar].
void main() => runApp(const TabBarApp());
class TabBarApp extends StatelessWidget {
const TabBarApp({super.key});
@override
Widget build(BuildContext context) {
return MaterialApp(
theme: ThemeData(
// tabBarTheme: const TabBarTheme(tabAlignment: TabAlignment.start),
useMaterial3: true,
),
home: const TabBarExample(),
);
}
}
class TabBarExample extends StatefulWidget {
const TabBarExample({super.key});
@override
State<TabBarExample> createState() => _TabBarExampleState();
}
class _TabBarExampleState extends State<TabBarExample> {
bool rtl = false;
@override
Widget build(BuildContext context) {
return DefaultTabController(
initialIndex: 1,
length: 3,
child: Directionality(
textDirection: rtl ? TextDirection.rtl : TextDirection.ltr,
child: Scaffold(
appBar: AppBar(
title: const Text('TabBar Sample'),
),
body: const Column(
children: <Widget>[
Text('Scrollable-TabAlignment.start'),
TabBar(
isScrollable: true,
tabAlignment: TabAlignment.start,
tabs: <Widget>[
Tab(
icon: Icon(Icons.cloud_outlined),
),
Tab(
icon: Icon(Icons.beach_access_sharp),
),
Tab(
icon: Icon(Icons.brightness_5_sharp),
),
],
),
Text('Scrollable-TabAlignment.startOffset'),
TabBar(
isScrollable: true,
tabAlignment: TabAlignment.startOffset,
tabs: <Widget>[
Tab(
icon: Icon(Icons.cloud_outlined),
),
Tab(
icon: Icon(Icons.beach_access_sharp),
),
Tab(
icon: Icon(Icons.brightness_5_sharp),
),
],
),
Text('Scrollable-TabAlignment.center'),
TabBar(
isScrollable: true,
tabAlignment: TabAlignment.center,
tabs: <Widget>[
Tab(
icon: Icon(Icons.cloud_outlined),
),
Tab(
icon: Icon(Icons.beach_access_sharp),
),
Tab(
icon: Icon(Icons.brightness_5_sharp),
),
],
),
Spacer(),
Text('Non-scrollable-TabAlignment.fill'),
TabBar(
tabAlignment: TabAlignment.fill,
tabs: <Widget>[
Tab(
icon: Icon(Icons.cloud_outlined),
),
Tab(
icon: Icon(Icons.beach_access_sharp),
),
Tab(
icon: Icon(Icons.brightness_5_sharp),
),
],
),
Text('Non-scrollable-TabAlignment.center'),
TabBar(
tabAlignment: TabAlignment.center,
tabs: <Widget>[
Tab(
icon: Icon(Icons.cloud_outlined),
),
Tab(
icon: Icon(Icons.beach_access_sharp),
),
Tab(
icon: Icon(Icons.brightness_5_sharp),
),
],
),
Spacer(),
],
),
floatingActionButton: FloatingActionButton.extended(
onPressed: () {
setState(() {
rtl = !rtl;
});
},
label: const Text('Switch Direction'),
icon: const Icon(Icons.swap_horiz),
),
),
),
);
}
}
```
</details>
2023-06-22 17:30:46 +00:00
Masatoshi Tsushima
0da8012cc7
Fix: Closing bottom sheet and removing FAB cause assertion failure ( #128566 )
...
Fixes #128562
2023-06-22 17:00:58 +00:00
Taha Tesser
2e05371c88
Add InputDecorationTheme.merge ( #129011 )
...
fixes [[Proposal] `InputDecorationTheme.merge()`](https://github.com/flutter/flutter/issues/125471 )
2023-06-22 16:02:08 +00:00
Yegor
07772a3d23
[framework,web] add FlutterTimeline and semantics benchmarks that use it ( #128366 )
...
## FlutterTimeline
Add a new class `FlutterTimeline` that's a drop-in replacement for `Timeline` from `dart:developer`. In addition to forwarding invocations of `startSync`, `finishSync`, `timeSync`, and `instantSync` to `dart:developer`, provides the following extra methods that make is easy to collect timings for code blocks on a frame-by-frame basis:
* `debugCollect()` - aggregates timings since the last reset, or since the app launched.
* `debugReset()` - forgets all data collected since the previous reset, or since the app launched. This allows clearing data from previous frames so timings can be attributed to the current frame.
* `now` - this was enhanced so that it works on the web by calling `window.performance.now` (in `Timeline` this is a noop in Dart web compilers).
* `collectionEnabled` - a field that controls whether `FlutterTimeline` stores timings in memory. By default this is disabled to avoid unexpected overhead (although the class is designed for minimal and predictable overhead). Specific benchmarks can enable collection to report to Skia Perf.
## Semantics benchmarks
Add `BenchMaterial3Semantics` that benchmarks the cost of semantics when constructing a screen full of Material 3 widgets from nothing. It is expected that semantics will have non-trivial cost in this case, but we should strive to keep it much lower than the rendering cost. This is the case already. This benchmark shows that the cost of semantics is <10%.
Add `BenchMaterial3ScrollSemantics` that benchmarks the cost of scrolling a previously constructed screen full of Material 3 widgets. The expectation should be that semantics will have trivial cost, since we're just shifting some widgets around. As of today, the numbers are not great, with semantics taking >50% of frame time, which is what prompted this PR in the first place. As we optimize this, we want to see this number improve.
2023-06-21 21:37:02 +00:00
Pierre-Louis
28ca523ac2
Remove incorrect non-nullable assumption from ShapeDecoration.lerp ( #129298 )
...
Fixes https://github.com/flutter/flutter/issues/129299
2023-06-21 21:34:58 +00:00
Justin McCandless
16eb4f2c08
Gracefully handle negative position in getWordAtOffset ( #128464 )
...
Should fix an unreproducible crash in text editing and track it with an assertion.
2023-06-21 14:22:04 -07:00
Renzo Olivares
b36ef583fb
Selection area right click behavior should match native ( #128224 )
...
This change updates `SelectableRegion`s right-click gesture to match native platform behavior.
Before: Right-click gesture selects word at position and opens context menu (All Platforms)
After:
- Linux, toggles context menu on/off, and collapses selection when click was not on an active selection (uncollapsed).
- Windows, Android, Fuchsia, shows context menu at right-clicked position (unless the click is at an active selection).
- macOS, toggles the context menu if right click was at the same position as the previous / or selects word at position and opens context menu.
- iOS, selects word at position and opens context menu.
This change also prevents the `copy` menu button from being shown when there is a collapsed selection (nothing to copy).
Fixes #117561
2023-06-21 19:32:04 +00:00
LongCatIsLooong
04ff86f020
Relax OverlayPortal asserts ( #129053 )
...
Fixes https://github.com/flutter/flutter/issues/129025
Also
- simplifies OverlayPortal code a bit and adds an assert.
- `Tooltip` shouldn't rebuild when hiding/showing the tooltip
2023-06-21 02:40:48 +00:00
Andrew Kolos
e1e45ca37a
update resolution-aware asset docs links ( #128769 )
...
Updates some doc comment links.
Fixes https://github.com/flutter/flutter/issues/128139
2023-06-20 23:45:52 +00:00
Justin McCandless
40b4bc996c
Fix: Magnifier appears and won't dismiss ( #128545 )
...
Fixes a bug when tapping near certain TextFields.
2023-06-20 16:36:45 -07:00
Tae Hyung Kim
541fdd60d3
DecoratedSliver ( #127823 )
...
This is a second attempt to merge #107269 . Currently I've fixed two of the issues:
1. Fixed horizontal scrollview by using a switch statement to consider vertical/horizontal case.
2. Fixed issue of `paintExtent` not being the right extent for painting. Rather using a `scrollExtent` for the main axis length of the decoration box and painting it offsetted by the `scrollOffset`.
3. If the sliver child has inifinite scrollExtent, then we only draw the decoration down to the bottom of the `cacheExtent`. The developer is expected to ensure that the border does not creep up above the cache area.
This PR includes a test that checks that the correct rectangle is drawn at a certain scrollOffset for both the horizontal and vertical case which should be sufficient for checking that `SliverDecoration` works properly now.
Fixes https://github.com/flutter/flutter/issues/107498 .
2023-06-20 23:35:42 +00:00
Md. Yeasin Sheikh
b733901de1
fixed PreferredSize constuctor invocations ( #128181 )
...
This PR changes the `PreferredSize` constructor parameter positioning to cover the constructor invocations. It helps on auto-complete and suggestion.
*List which issues are fixed by this PR. You must list at least one issue.*
Fixes #128178
2023-06-20 22:21:12 +00:00
Swaroop S
80935ad984
Adding ScrollController support for Stepper widget ( #128814 )
...
This PR is to add **controller** property to the **Stepper** widget, so
that user has the flexibility to control the scroll offset for various
purposes(especially in case of scroll animations)!
Fixes #61207
Co-authored-by: Taha Tesser <tessertaha@gmail.com>
2023-06-20 09:04:41 -07:00
Leigha Jarett
b2c993cad3
Add to API docs to explain what Assist and Suggestion chips are ( #129034 )
...
Fixes https://github.com/flutter/flutter/issues/128028 .
It seems pretty straightforward so don't think it warrants the effort to add more samples.
2023-06-20 09:03:21 +00:00
Taha Tesser
467c970bfb
Introduce MaterialState color property for chips ( #128584 )
...
fixes https://github.com/flutter/flutter/issues/115827
fixes https://github.com/flutter/flutter/issues/101325
### Description
1. This PR adds a new MaterialState `color` property to all the chips (this makes it possible to customize chips in all states from the M3 specs).
2. Updated defaults to use the new MaterialState `color` property.
3. Updated and added new tests to all the chip test classes.
<details>
<summary>code sample</summary>
```dart
import 'package:flutter/material.dart';
const Color disabledColor = Colors.black26;
const Color backgroundColor = Colors.cyan;
final Color disabledSelectedColor = Colors.red.shade100;
const Color selectedColor = Colors.amber;
final MaterialStateProperty<Color> color =
MaterialStateProperty.resolveWith((Set<MaterialState> states) {
if (states.contains(MaterialState.disabled) &&
states.contains(MaterialState.selected)) {
return disabledSelectedColor;
}
if (states.contains(MaterialState.disabled)) {
return disabledColor;
}
if (states.contains(MaterialState.selected)) {
return selectedColor;
}
return backgroundColor;
});
void main() => runApp(const MyApp());
class MyApp extends StatelessWidget {
const MyApp({super.key});
@override
Widget build(BuildContext context) {
return MaterialApp(
debugShowCheckedModeBanner: false,
theme: ThemeData(
useMaterial3: true,
// chipTheme: ChipThemeData(color: color),
),
home: const Example(),
);
}
}
class Example extends StatefulWidget {
const Example({super.key});
@override
State<Example> createState() => _ExampleState();
}
class _ExampleState extends State<Example> {
bool enabled = false;
bool selected = true;
@override
Widget build(BuildContext context) {
const Widget verticalSpace = SizedBox(height: 20);
return Scaffold(
body: Center(
child: Column(
children: <Widget>[
const SizedBox(height: 25),
Row(
mainAxisAlignment: MainAxisAlignment.spaceEvenly,
children: <Widget>[
const Card(
elevation: 0.0,
color: disabledColor,
child: Padding(
padding: EdgeInsets.all(8.0),
child: Text('disabledColor'),
),
),
const Card(
elevation: 0.0,
color: backgroundColor,
child: Padding(
padding: EdgeInsets.all(8.0),
child: Text('backgroundColor'),
),
),
Card(
elevation: 0.0,
color: disabledSelectedColor,
child: const Padding(
padding: EdgeInsets.all(8.0),
child: Text('disabledSelectedColor'),
),
),
const Card(
elevation: 0.0,
color: selectedColor,
child: Padding(
padding: EdgeInsets.all(8.0),
child: Text('selectedColor'),
),
),
],
),
const Spacer(),
Row(
mainAxisAlignment: MainAxisAlignment.spaceEvenly,
children: <Widget>[
Column(
mainAxisSize: MainAxisSize.min,
mainAxisAlignment: MainAxisAlignment.center,
children: <Widget>[
RawChip(
selected: selected,
selectedColor: selectedColor,
color: color,
label: const Text('RawChip'),
isEnabled: enabled,
onSelected: enabled ? (bool value) {} : null,
),
verticalSpace,
InputChip(
isEnabled: enabled,
selected: selected,
selectedColor: selectedColor,
color: color,
label: const Text('InputChip'),
onSelected: enabled ? (bool value) {} : null,
),
],
),
Column(
mainAxisSize: MainAxisSize.min,
children: <Widget>[
FilterChip(
selected: selected,
selectedColor: selectedColor,
color: color,
label: const Text('FilterChip'),
onSelected: enabled ? (bool value) {} : null,
),
verticalSpace,
FilterChip.elevated(
selected: selected,
selectedColor: selectedColor,
color: color,
label: const Text('FilterChip.elevated'),
onSelected: enabled ? (bool value) {} : null,
),
],
),
Column(
mainAxisSize: MainAxisSize.min,
children: <Widget>[
ChoiceChip(
selected: selected,
selectedColor: selectedColor,
color: color,
label: const Text('ChoiceChip'),
onSelected: enabled ? (bool value) {} : null,
),
verticalSpace,
ChoiceChip.elevated(
selected: selected,
selectedColor: selectedColor,
color: color,
label: const Text('ChoiceChip.elevated'),
onSelected: enabled ? (bool value) {} : null,
),
],
),
],
),
const Spacer(),
Row(
children: <Widget>[
Flexible(
child: SwitchListTile(
title: const Text('Enabled'),
value: enabled,
onChanged: (bool value) {
setState(() => enabled = value);
},
),
),
Flexible(
child: SwitchListTile(
title: const Text('Selected'),
value: selected,
onChanged: (bool value) {
setState(() => selected = value);
},
),
),
],
)
],
),
),
);
}
}
```
</details>
### Before (not possible to customize disabled and selected chips)
### After (using disabled and selected chips using the new MaterialState `color` property)
2023-06-19 22:03:26 +00:00
Taha Tesser
165d237792
Fix InputDecoration.applyDefaults ignoring some properties ( #129010 )
...
fixes [`InputDecoration.applyDefaults` ignores some properties](https://github.com/flutter/flutter/issues/127330 )
<details>
<summary>code sample</summary>
```dart
import 'package:flutter/material.dart';
void main() => runApp(const MyApp());
class MyApp extends StatelessWidget {
const MyApp({super.key});
@override
Widget build(BuildContext context) {
return MaterialApp(
debugShowCheckedModeBanner: false,
theme: ThemeData(useMaterial3: true),
home: const Example(),
);
}
}
class Example extends StatelessWidget {
const Example({super.key});
@override
Widget build(BuildContext context) {
const TextStyle textStyle = TextStyle(color: Color(0xFF00FFFF));
const Color color = Color(0xFF00FF00);
const InputBorder inputBorder = OutlineInputBorder(
borderSide: BorderSide(
color: Color(0xFF0000FF),
),
);
final InputDecoration appliedDefaults =
const InputDecoration().applyDefaults(
const InputDecorationTheme(
labelStyle: textStyle,
floatingLabelStyle: textStyle,
helperStyle: textStyle,
helperMaxLines: 2,
hintStyle: textStyle,
errorStyle: textStyle,
errorMaxLines: 2,
floatingLabelBehavior: FloatingLabelBehavior.never,
floatingLabelAlignment: FloatingLabelAlignment.center,
isDense: true,
contentPadding: EdgeInsets.all(1.0),
iconColor: color,
prefixStyle: textStyle,
prefixIconColor: color,
suffixStyle: textStyle,
suffixIconColor: color,
counterStyle: textStyle,
filled: true,
fillColor: color,
focusColor: color,
hoverColor: color,
errorBorder: inputBorder,
focusedBorder: inputBorder,
focusedErrorBorder: inputBorder,
disabledBorder: inputBorder,
enabledBorder: inputBorder,
border: InputBorder.none,
alignLabelWithHint: true,
constraints: BoxConstraints(
minWidth: 10, maxWidth: 20, minHeight: 30, maxHeight: 40),
),
);
return Scaffold(
appBar: AppBar(
title: const Text('InputDecoration().applyDefaults'),
),
// Centered FilledButton.
body: Center(
child: SingleChildScrollView(
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: <Widget>[
ColoredBox(
color: appliedDefaults.labelStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.labelStyle.toString()),
),
ColoredBox(
color: appliedDefaults.floatingLabelStyle
.toString()
.contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.floatingLabelStyle.toString()),
),
ColoredBox(
color: appliedDefaults.helperStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.helperStyle.toString()),
),
ColoredBox(
color: appliedDefaults.hintStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.hintStyle.toString()),
),
ColoredBox(
color: appliedDefaults.errorStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.errorStyle.toString()),
),
ColoredBox(
color: appliedDefaults.iconColor.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.iconColor.toString()),
),
ColoredBox(
color: appliedDefaults.prefixStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.prefixStyle.toString()),
),
ColoredBox(
color:
appliedDefaults.prefixIconColor.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.prefixIconColor.toString()),
),
ColoredBox(
color: appliedDefaults.suffixStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.suffixStyle.toString()),
),
ColoredBox(
color:
appliedDefaults.suffixIconColor.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.suffixIconColor.toString()),
),
ColoredBox(
color: appliedDefaults.counterStyle.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.counterStyle.toString()),
),
ColoredBox(
color: appliedDefaults.fillColor.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.fillColor.toString()),
),
ColoredBox(
color: appliedDefaults.focusColor.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.focusColor.toString()),
),
ColoredBox(
color: appliedDefaults.hoverColor.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.hoverColor.toString()),
),
ColoredBox(
color: appliedDefaults.errorBorder.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.errorBorder.toString()),
),
ColoredBox(
color: appliedDefaults.focusedBorder.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.focusedBorder.toString()),
),
ColoredBox(
color: appliedDefaults.focusedErrorBorder
.toString()
.contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.focusedErrorBorder.toString()),
),
ColoredBox(
color:
appliedDefaults.disabledBorder.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.disabledBorder.toString()),
),
ColoredBox(
color: appliedDefaults.enabledBorder.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.enabledBorder.toString()),
),
ColoredBox(
color: appliedDefaults.border.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.border.toString()),
),
ColoredBox(
color: appliedDefaults.constraints.toString().contains('null')
? Colors.red
: Colors.green,
child: Text(appliedDefaults.constraints.toString()),
),
],
),
),
),
);
}
}
```
</details>
### Before
### After
2023-06-19 07:06:19 +00:00
Polina Cherkasova
9690394772
Update objectToDiagnosticsNode to stop failing. ( #129027 )
...
Prerequisite for https://github.com/flutter/devtools/pull/5918
2023-06-16 19:39:54 +00:00
Polina Cherkasova
7214cb28be
Accept Diagnosticable as input in inspector API. ( #128962 )
2023-06-16 10:45:34 -07:00
Michael Goderbauer
5ab5d82a39
Remove AbstractNode from RenderObject and deprecate it ( #128973 )
...
It's time to say good bye to an old friend.
It has outlived its usefulness.
Farewell, AbstractNode! ð«¡
2023-06-16 16:16:36 +00:00
Mouad Debbar
fc8856eb80
[web] Don't crash on const HtmlElementView() ( #128965 )
...
Previously, when the code contained `const HtmlElementView()` it would break even if it's guarded by `if (kIsWeb)`.
This PR makes it such that `const HtmlElementView()` is allowed but it still throws if it gets inserted into the widget tree by mistake on non-web platforms.
One improvement we can make in the future is to have a conditional import:
- `_html_element_view_web.dart` that contains the real `HtmlElementView` that can only be instantiated on web.
- `_html_element_view_io.dart` that contains a stub with an unimplemented `build()` method.
Fixes https://github.com/flutter/flutter/issues/43532
2023-06-16 13:53:05 +00:00
Polina Cherkasova
dd4a8150a8
Update getProperties to handle Diagnosticable as input. ( #128897 )
2023-06-15 17:04:46 -07:00
Qun Cheng
f8e8d89b85
Set Semantics.button to true for date widget ( #128824 )
...
Fixes #127863
This PR is to set `Semantics.button` to true for date widget in `DatePicker`.
https://github.com/flutter/flutter/assets/36861262/b366f25a-6bf3-4e3f-b944-ab71197b73e5
2023-06-15 20:13:07 +00:00
Mouad Debbar
b0188cd182
[web] Pass creation params to the platform view factory ( #128146 )
...
This concludes step 1 of the `HtmlElementView` improvements. It's now possible to pass creation params to platform view factories directly from `HtmlElementView`.
Here's a sample app using a single factory to render platform views in different colors:
<details>
<summary>Code sample</summary>
```dart
import 'dart:js_interop';
import 'dart:ui_web' as ui_web;
import 'package:flutter/material.dart';
import 'package:web/web.dart' as web;
void main() {
runApp(MyApp());
}
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Platform View Demo',
home: Scaffold(
appBar: AppBar(
title: Text('Platform View Demo'),
),
body: Center(
child: Column(
mainAxisAlignment: MainAxisAlignment.spaceEvenly,
children: [
BoxWrapper('red'),
BoxWrapper(null),
BoxWrapper('blue'),
],
),
),
),
);
}
}
bool isRegistered = false;
class BoxWrapper extends StatelessWidget {
const BoxWrapper(this.cssColor);
final String? cssColor;
void register() {
if (isRegistered) return;
isRegistered = true;
ui_web.platformViewRegistry.registerViewFactory('my-platform-view', (
id, {
Object? params,
}) {
params as String?;
final element = web.document.createElement('div'.toJS) as web.HTMLElement;
element.textContent = 'Platform View'.toJS;
element.style
..lineHeight = '100px'.toJS
..fontSize = '24px'.toJS
..backgroundColor = (params ?? 'pink').toJS
..textAlign = 'center'.toJS;
return element;
});
}
@override
Widget build(BuildContext context) {
register();
return SizedBox(
width: 200,
height: 100,
child: Card(
child: HtmlElementView(
viewType: 'my-platform-view',
creationParams: cssColor,
),
),
);
}
}
```
</details>
Depends on https://github.com/flutter/engine/pull/42255
Part of https://github.com/flutter/flutter/issues/127030
2023-06-15 18:00:25 +00:00
Qun Cheng
7d950864a1
Update golden tests for material ( #128839 )
2023-06-14 16:09:27 -07:00
Polina Cherkasova
ed3d46305c
Update getChildrenSummaryTree to handle Diagnosticable as input. ( #128833 )
2023-06-14 23:01:22 +00:00
Ian Hickson
d64332a0aa
Improve the error message for non-normalized constraints ( #127906 )
...
We probably added RenderBox.layout after this error was created and of course we weren't writing tests back then...
2023-06-14 22:50:33 +00:00
Ian Hickson
34f39a208e
ContextAction.isEnabled needs a context ( #127721 )
...
...and lots of things that fall out from that
2023-06-14 22:47:06 +00:00
LouiseHsu
4effd9c46f
Remove temporary default case for PointerSignalKind ( #128900 )
...
This PR cleans up some prep that was added as part of
https://github.com/flutter/flutter/pull/120731 which was done to unblock
https://github.com/flutter/engine/pull/39637
Since the work done in that PR was reverted, this should be removed.
## Pre-launch Checklist
- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [ ] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] All existing and new tests are passing.
2023-06-14 14:50:37 -07:00
Mouad Debbar
95be76ab7e
[web] Migrate framework away from dart:html and package:js ( #128580 )
...
Use `package:web` and `dart:js_interop` instead.
Part of https://github.com/flutter/flutter/issues/127030
2023-06-14 17:43:39 +00:00
cruiser-baxter
52c4db8d33
Fixed slider value indicator not disappearing after a bit on desktop platform when slider is clicked not dragged ( #128137 )
...
In slider.dart within the _startInteraction method and within the below conditional.
"if (!_active && !hasFocus &&
_state.valueIndicatorController.status == AnimationStatus.completed)"
**Changed to:**
"f (!_active &&
_state.valueIndicatorController.status == AnimationStatus.completed)"
This allows the value indicator to disappear after a bit when clicked instead of dragged on Desktop platform.
I also added a test in slider_test.dart to detect the bug if it ever returns.
Fixes https://github.com/flutter/flutter/issues/123313
2023-06-14 17:11:08 +00:00
Michael Goderbauer
16e6be8b39
Inline AbstractNode into SemanticsNode and Layer ( #128467 )
2023-06-14 09:36:41 -07:00
Kate Lovett
a0bfde9bfb
Fix ensureVisible and default focus traversal for reversed scrollables ( #128756 )
...
Fixes https://github.com/flutter/flutter/issues/128749
The ScrollPositionAlignmentPolicy does not account for AxisDirection, which meant default focus traversal of reversed scrollables did not work. The policy doesn't know about the axis direction, so this is corrected in the ScrollPosition where all the information is available before calling on the viewport for the new target scroll offset.
This fixes that by flipping the policy (unless explicit) so that focus traversal works.
2023-06-13 16:51:22 +00:00
Kate Lovett
9e0ad4c63f
Fix paint offset in reverse for 2D ( #128724 )
...
Fixes https://github.com/flutter/flutter/issues/128723
The paint offset was incorrectly being computed when one or both axis was in the reverse direction. Instead of using the paint extent, the child's size should be used.
2023-06-12 23:28:15 +00:00
nt4f04uNd
ad8c997817
Fix RangeSlider notifies start and end twice when participates in gesture arena ( #128674 )
...
Fixes https://github.com/flutter/flutter/issues/128433
This PR applies the same fix as we did for Slider https://github.com/flutter/flutter/pull/82152
2023-06-12 15:20:11 +00:00
Taha Tesser
ca5aa2329a
Update ListTile text defaults to use ColorScheme ( #128581 )
...
fixes https://github.com/flutter/flutter/issues/128569
<details>
<summary>code sample</summary>
```dart
import 'package:flutter/material.dart';
void main() {
runApp(const ListTileApp());
}
class ListTileApp extends StatelessWidget {
const ListTileApp({super.key});
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Flutter Demo',
theme: ThemeData(
useMaterial3: true,
colorScheme: ColorScheme.fromSeed(seedColor: Colors.red).copyWith(
onSurface: Colors.yellow,
onSurfaceVariant: Colors.green,
),
),
home: const Scaffold(
body: Center(
child: ListTile(
title: Text('title'),
subtitle: Text('subtitle'),
),
),
),
);
}
}
```
</details>
# Description
M3 ListTile couldn't be customized using `ColorScheme` colors.
- This PR updates the list tile text defaults to `ColorScheme` text color tokens.
- Improved the `ListTile` template to use the token group.
- Update docs and tests.
```dart
colorScheme: ColorScheme.fromSeed(seedColor: Colors.red).copyWith(
onSurface: Colors.yellow,
onSurfaceVariant: Colors.green,
),
```
### Before
### After
2023-06-12 14:52:06 +00:00
LongCatIsLooong
e1fdb1aa74
migrate Tooltip to use OverlayPortal ( #127728 )
...
https://github.com/flutter/flutter/issues/7151 isn't a problem with OverlayPortal so the test is removed.
Also removed some `mounted` checks since they're no longer needed.
2023-06-10 20:54:37 +00:00
