From a6d68ee6dd6ace73238042eb2e8111be863eecf2 Mon Sep 17 00:00:00 2001 From: Michael Goderbauer Date: Thu, 8 Aug 2019 19:31:28 -0700 Subject: [PATCH] remove Header flag from BottomNavigationBar items (#37870) --- .../src/material/bottom_navigation_bar.dart | 1 - .../material/bottom_navigation_bar_test.dart | 22 ------------------- 2 files changed, 23 deletions(-) diff --git a/packages/flutter/lib/src/material/bottom_navigation_bar.dart b/packages/flutter/lib/src/material/bottom_navigation_bar.dart index 1d6ec1a3dc..2151ccac46 100644 --- a/packages/flutter/lib/src/material/bottom_navigation_bar.dart +++ b/packages/flutter/lib/src/material/bottom_navigation_bar.dart @@ -473,7 +473,6 @@ class _BottomNavigationTile extends StatelessWidget { flex: size, child: Semantics( container: true, - header: true, selected: selected, child: Focus( child: Stack( diff --git a/packages/flutter/test/material/bottom_navigation_bar_test.dart b/packages/flutter/test/material/bottom_navigation_bar_test.dart index d8bc3a763f..963e4571ef 100644 --- a/packages/flutter/test/material/bottom_navigation_bar_test.dart +++ b/packages/flutter/test/material/bottom_navigation_bar_test.dart @@ -1192,24 +1192,17 @@ void main() { TestSemantics( flags: [ SemanticsFlag.isSelected, - SemanticsFlag.isHeader, ], actions: [SemanticsAction.tap], label: 'AC\nTab 1 of 3', textDirection: TextDirection.ltr, ), TestSemantics( - flags: [ - SemanticsFlag.isHeader, - ], actions: [SemanticsAction.tap], label: 'Alarm\nTab 2 of 3', textDirection: TextDirection.ltr, ), TestSemantics( - flags: [ - SemanticsFlag.isHeader, - ], actions: [SemanticsAction.tap], label: 'Hot Tub\nTab 3 of 3', textDirection: TextDirection.ltr, @@ -1260,24 +1253,17 @@ void main() { TestSemantics( flags: [ SemanticsFlag.isSelected, - SemanticsFlag.isHeader, ], actions: [SemanticsAction.tap], label: 'AC\nTab 1 of 3', textDirection: TextDirection.ltr, ), TestSemantics( - flags: [ - SemanticsFlag.isHeader, - ], actions: [SemanticsAction.tap], label: 'Alarm\nTab 2 of 3', textDirection: TextDirection.ltr, ), TestSemantics( - flags: [ - SemanticsFlag.isHeader, - ], actions: [SemanticsAction.tap], label: 'Hot Tub\nTab 3 of 3', textDirection: TextDirection.ltr, @@ -1565,16 +1551,12 @@ void main() { TestSemantics( flags: [ SemanticsFlag.isSelected, - SemanticsFlag.isHeader, ], actions: [SemanticsAction.tap], label: 'Red\nTab 1 of 2', textDirection: TextDirection.ltr, ), TestSemantics( - flags: [ - SemanticsFlag.isHeader, - ], actions: [SemanticsAction.tap], label: 'Green\nTab 2 of 2', textDirection: TextDirection.ltr, @@ -1623,16 +1605,12 @@ void main() { TestSemantics( flags: [ SemanticsFlag.isSelected, - SemanticsFlag.isHeader, ], actions: [SemanticsAction.tap], label: 'Red\nTab 1 of 2', textDirection: TextDirection.ltr, ), TestSemantics( - flags: [ - SemanticsFlag.isHeader, - ], actions: [SemanticsAction.tap], label: 'Green\nTab 2 of 2', textDirection: TextDirection.ltr,