firka/flutter
firka/flutter
4
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Gate the PhysicalModel shadows behind a flag (#9198)

Browse Source 
Fixes https://github.com/flutter/flutter/issues/9186
This commit is contained in:
Jason Simmons 2017-04-04 16:42:48 -07:00 committed by GitHub
parent 3000c8bb59
commit d302213270
2 changed files with 37 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

53
packages/flutter/lib/src/material/material.dart
View File

@ -7,6 +7,7 @@ import 'package:flutter/rendering.dart';
import 'package:flutter/widgets.dart';
import 'constants.dart';
import 'shadows.dart';
import 'theme.dart';
/// Signature for the callback used by ink effects to obtain the rectangle for the effect.
@ -187,6 +188,9 @@ class Material extends StatefulWidget {
/// The default radius of an ink splash in logical pixels.
static const double defaultSplashRadius = 35.0;
// Temporary flag used to enable the PhysicalModel shadow implementation.
static bool debugEnablePhysicalModel = false;
}
class _MaterialState extends State<Material> with TickerProviderStateMixin {
@ -232,30 +236,41 @@ class _MaterialState extends State<Material> with TickerProviderStateMixin {
)
);
if (config.type == MaterialType.circle) {
contents = new PhysicalModel(
shape: BoxShape.circle,
elevation: config.elevation,
color: backgroundColor,
child: contents,
);
} else if (config.type == MaterialType.transparency) {
if (radius == null) {
contents = new ClipRect(child: contents);
if (Material.debugEnablePhysicalModel) {
if (config.type == MaterialType.circle) {
contents = new PhysicalModel(
shape: BoxShape.circle,
elevation: config.elevation,
color: backgroundColor,
child: contents,
);
} else if (config.type == MaterialType.transparency) {
if (radius == null) {
contents = new ClipRect(child: contents);
} else {
contents = new ClipRRect(
borderRadius: radius,
child: contents
);
}
} else {
contents = new PhysicalModel(
shape: BoxShape.rectangle,
borderRadius: radius ?? BorderRadius.zero,
elevation: config.elevation,
color: backgroundColor,
child: contents,
);
}
} else {
if (config.type == MaterialType.circle) {
contents = new ClipOval(child: contents);
} else if (kMaterialEdges[config.type] != null) {
contents = new ClipRRect(
borderRadius: radius,
child: contents
);
}
} else {
contents = new PhysicalModel(
shape: BoxShape.rectangle,
borderRadius: radius ?? BorderRadius.zero,
elevation: config.elevation,
color: backgroundColor,
child: contents,
);
}
if (config.type != MaterialType.transparency) {
@ -264,6 +279,8 @@ class _MaterialState extends State<Material> with TickerProviderStateMixin {
duration: kThemeChangeDuration,
decoration: new BoxDecoration(
borderRadius: radius,
boxShadow: config.elevation == 0 || Material.debugEnablePhysicalModel ?
null : kElevationToShadow[config.elevation],
shape: config.type == MaterialType.circle ? BoxShape.circle : BoxShape.rectangle
),
child: new Container(

2
packages/flutter/test/material/ink_paint_test.dart
View File

@ -9,6 +9,8 @@ import 'package:flutter_test/flutter_test.dart';
import '../rendering/mock_canvas.dart';
void main() {
Material.debugEnablePhysicalModel = true;
testWidgets('Does the ink widget render a border radius', (WidgetTester tester) async {
final Color highlightColor = new Color(0xAAFF0000);
final Color splashColor = new Color(0xAA0000FF);