Unverified Commit d5b64806 authored by Hans Muller's avatar Hans Muller Committed by GitHub

Material.border type is now BorderRadiusGeometry (#27376)

parent cdddec74
...@@ -257,14 +257,16 @@ class Material extends StatefulWidget { ...@@ -257,14 +257,16 @@ class Material extends StatefulWidget {
/// The default value is [kThemeChangeDuration]. /// The default value is [kThemeChangeDuration].
final Duration animationDuration; final Duration animationDuration;
/// If non-null, the corners of this box are rounded by this [BorderRadius]. /// If non-null, the corners of this box are rounded by this
/// [BorderRadiusGeometry] value.
///
/// Otherwise, the corners specified for the current [type] of material are /// Otherwise, the corners specified for the current [type] of material are
/// used. /// used.
/// ///
/// If [shape] is non null then the border radius is ignored. /// If [shape] is non null then the border radius is ignored.
/// ///
/// Must be null if [type] is [MaterialType.circle]. /// Must be null if [type] is [MaterialType.circle].
final BorderRadius borderRadius; final BorderRadiusGeometry borderRadius;
/// The ink controller from the closest instance of this class that /// The ink controller from the closest instance of this class that
/// encloses the given context. /// encloses the given context.
...@@ -292,7 +294,7 @@ class Material extends StatefulWidget { ...@@ -292,7 +294,7 @@ class Material extends StatefulWidget {
textStyle?.debugFillProperties(properties, prefix: 'textStyle.'); textStyle?.debugFillProperties(properties, prefix: 'textStyle.');
properties.add(DiagnosticsProperty<ShapeBorder>('shape', shape, defaultValue: null)); properties.add(DiagnosticsProperty<ShapeBorder>('shape', shape, defaultValue: null));
properties.add(DiagnosticsProperty<bool>('borderOnForeground', borderOnForeground, defaultValue: true)); properties.add(DiagnosticsProperty<bool>('borderOnForeground', borderOnForeground, defaultValue: true));
properties.add(EnumProperty<BorderRadius>('borderRadius', borderRadius, defaultValue: null)); properties.add(DiagnosticsProperty<BorderRadiusGeometry>('borderRadius', borderRadius, defaultValue: null));
} }
/// The default radius of an ink splash in logical pixels. /// The default radius of an ink splash in logical pixels.
......
...@@ -56,6 +56,41 @@ class PaintRecorder extends CustomPainter { ...@@ -56,6 +56,41 @@ class PaintRecorder extends CustomPainter {
} }
void main() { void main() {
testWidgets('default Material debugFillProperties', (WidgetTester tester) async {
final DiagnosticPropertiesBuilder builder = DiagnosticPropertiesBuilder();
const Material().debugFillProperties(builder);
final List<String> description = builder.properties
.where((DiagnosticsNode node) => !node.isFiltered(DiagnosticLevel.info))
.map((DiagnosticsNode node) => node.toString())
.toList();
expect(description, <String>['type: canvas']);
});
testWidgets('Material implements debugFillProperties', (WidgetTester tester) async {
final DiagnosticPropertiesBuilder builder = DiagnosticPropertiesBuilder();
const Material(
type: MaterialType.canvas,
color: Color(0xFFFFFFFF),
textStyle: TextStyle(color: Color(0xff00ff00)),
borderRadius: BorderRadiusDirectional.all(Radius.circular(10)),
).debugFillProperties(builder);
final List<String> description = builder.properties
.where((DiagnosticsNode node) => !node.isFiltered(DiagnosticLevel.info))
.map((DiagnosticsNode node) => node.toString())
.toList();
expect(description, <String>[
'type: canvas',
'color: Color(0xffffffff)',
'textStyle.inherit: true',
'textStyle.color: Color(0xff00ff00)',
'borderRadius: BorderRadiusDirectional.circular(10.0)'
]);
});
testWidgets('LayoutChangedNotification test', (WidgetTester tester) async { testWidgets('LayoutChangedNotification test', (WidgetTester tester) async {
await tester.pumpWidget( await tester.pumpWidget(
Material( Material(
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment