@@ -254,26 +254,31 @@ class _NavigationBarButton extends StatelessWidget {
254254    return  AnimatedScaleOnTap (
255255      scaleEnd:  0.875 ,
256256      duration:  const  Duration (milliseconds:  100 ),
257-       child:  GestureDetector (
258-         behavior:  HitTestBehavior .opaque,
259-         onTap:  onPressed,
260-         child:  Column (
261-           mainAxisSize:  MainAxisSize .min,
262-           children:  [
263-             SizedBox (height:  34 ,
264-               child:  Center (
265-                 child:  Icon (icon, size:  24 , color:  color,))),
266-             Flexible (
267-               child:  Text (
268-                 label,
269-                 style:  TextStyle (
270-                   fontSize:  12 ,
271-                   color:  color,
272-                   height:  1.0 ,),
273-                 textAlign:  TextAlign .center,
274-                 maxLines:  2 ,
275-                 overflow:  TextOverflow .ellipsis)),
276-           ])));
257+       child:  Material (
258+         type:  MaterialType .transparency,
259+         child:  InkWell (
260+           // TODO(#417): Disable splash effects for all buttons globally. 
261+           borderRadius:  BorderRadius .all (Radius .circular (4 )),
262+           splashFactory:  NoSplash .splashFactory,
263+           highlightColor:  designVariables.navigationButtonBg,
264+           onTap:  onPressed,
265+           child:  Column (
266+             mainAxisSize:  MainAxisSize .min,
267+             children:  [
268+               SizedBox (height:  34 ,
269+                 child:  Center (
270+                   child:  Icon (icon, size:  24 , color:  color,))),
271+               Flexible (
272+                 child:  Text (
273+                   label,
274+                   style:  TextStyle (
275+                     fontSize:  12 ,
276+                     color:  color,
277+                     height:  1.0 ,),
278+                   textAlign:  TextAlign .center,
279+                   maxLines:  2 ,
280+                   overflow:  TextOverflow .ellipsis)),
281+             ]))));
277282  }
278283}
279284
0 commit comments