removed obsolete RTL animated icon workaround
This commit is contained in:
parent
67b7842ec0
commit
ed148712d2
6 changed files with 10 additions and 1335 deletions
|
@ -18,7 +18,6 @@ import 'package:aves/widgets/collection/collection_page.dart';
|
||||||
import 'package:aves/widgets/collection/entry_set_action_delegate.dart';
|
import 'package:aves/widgets/collection/entry_set_action_delegate.dart';
|
||||||
import 'package:aves/widgets/collection/filter_bar.dart';
|
import 'package:aves/widgets/collection/filter_bar.dart';
|
||||||
import 'package:aves/widgets/collection/query_bar.dart';
|
import 'package:aves/widgets/collection/query_bar.dart';
|
||||||
import 'package:aves/widgets/common/animated_icons_fix.dart';
|
|
||||||
import 'package:aves/widgets/common/app_bar_subtitle.dart';
|
import 'package:aves/widgets/common/app_bar_subtitle.dart';
|
||||||
import 'package:aves/widgets/common/app_bar_title.dart';
|
import 'package:aves/widgets/common/app_bar_title.dart';
|
||||||
import 'package:aves/widgets/common/basic/menu.dart';
|
import 'package:aves/widgets/common/basic/menu.dart';
|
||||||
|
@ -174,9 +173,8 @@ class _CollectionAppBarState extends State<CollectionAppBar> with SingleTickerPr
|
||||||
return IconButton(
|
return IconButton(
|
||||||
// key is expected by test driver
|
// key is expected by test driver
|
||||||
key: const Key('appbar-leading-button'),
|
key: const Key('appbar-leading-button'),
|
||||||
// TODO TLAD [rtl] replace to regular `AnimatedIcon` when this is fixed: https://github.com/flutter/flutter/issues/60521
|
icon: AnimatedIcon(
|
||||||
icon: AnimatedIconFixIssue60521(
|
icon: AnimatedIcons.menu_arrow,
|
||||||
icon: AnimatedIconsFixIssue60521.menu_arrow,
|
|
||||||
progress: _browseToSelectAnimation,
|
progress: _browseToSelectAnimation,
|
||||||
),
|
),
|
||||||
onPressed: onPressed,
|
onPressed: onPressed,
|
||||||
|
|
File diff suppressed because it is too large
Load diff
|
@ -4,7 +4,6 @@ import 'package:aves/model/filters/filters.dart';
|
||||||
import 'package:aves/model/selection.dart';
|
import 'package:aves/model/selection.dart';
|
||||||
import 'package:aves/model/source/collection_source.dart';
|
import 'package:aves/model/source/collection_source.dart';
|
||||||
import 'package:aves/theme/durations.dart';
|
import 'package:aves/theme/durations.dart';
|
||||||
import 'package:aves/widgets/common/animated_icons_fix.dart';
|
|
||||||
import 'package:aves/widgets/common/app_bar_subtitle.dart';
|
import 'package:aves/widgets/common/app_bar_subtitle.dart';
|
||||||
import 'package:aves/widgets/common/app_bar_title.dart';
|
import 'package:aves/widgets/common/app_bar_title.dart';
|
||||||
import 'package:aves/widgets/common/basic/menu.dart';
|
import 'package:aves/widgets/common/basic/menu.dart';
|
||||||
|
@ -98,9 +97,8 @@ class _FilterGridAppBarState<T extends CollectionFilter> extends State<FilterGri
|
||||||
return IconButton(
|
return IconButton(
|
||||||
// key is expected by test driver
|
// key is expected by test driver
|
||||||
key: const Key('appbar-leading-button'),
|
key: const Key('appbar-leading-button'),
|
||||||
// TODO TLAD [rtl] replace to regular `AnimatedIcon` when this is fixed: https://github.com/flutter/flutter/issues/60521
|
icon: AnimatedIcon(
|
||||||
icon: AnimatedIconFixIssue60521(
|
icon: AnimatedIcons.menu_arrow,
|
||||||
icon: AnimatedIconsFixIssue60521.menu_arrow,
|
|
||||||
progress: _browseToSelectAnimation,
|
progress: _browseToSelectAnimation,
|
||||||
),
|
),
|
||||||
onPressed: onPressed,
|
onPressed: onPressed,
|
||||||
|
|
|
@ -16,7 +16,6 @@ import 'package:aves/model/source/tag.dart';
|
||||||
import 'package:aves/ref/mime_types.dart';
|
import 'package:aves/ref/mime_types.dart';
|
||||||
import 'package:aves/theme/icons.dart';
|
import 'package:aves/theme/icons.dart';
|
||||||
import 'package:aves/widgets/collection/collection_page.dart';
|
import 'package:aves/widgets/collection/collection_page.dart';
|
||||||
import 'package:aves/widgets/common/animated_icons_fix.dart';
|
|
||||||
import 'package:aves/widgets/common/expandable_filter_row.dart';
|
import 'package:aves/widgets/common/expandable_filter_row.dart';
|
||||||
import 'package:aves/widgets/common/extensions/build_context.dart';
|
import 'package:aves/widgets/common/extensions/build_context.dart';
|
||||||
import 'package:aves/widgets/common/identity/aves_filter_chip.dart';
|
import 'package:aves/widgets/common/identity/aves_filter_chip.dart';
|
||||||
|
@ -61,9 +60,8 @@ class CollectionSearchDelegate {
|
||||||
// so the leading may mistakenly switch to the close button
|
// so the leading may mistakenly switch to the close button
|
||||||
return canPop
|
return canPop
|
||||||
? IconButton(
|
? IconButton(
|
||||||
// TODO TLAD [rtl] replace to regular `AnimatedIcon` when this is fixed: https://github.com/flutter/flutter/issues/60521
|
icon: AnimatedIcon(
|
||||||
icon: AnimatedIconFixIssue60521(
|
icon: AnimatedIcons.menu_arrow,
|
||||||
icon: AnimatedIconsFixIssue60521.menu_arrow,
|
|
||||||
progress: transitionAnimation,
|
progress: transitionAnimation,
|
||||||
),
|
),
|
||||||
onPressed: () => _goBack(context),
|
onPressed: () => _goBack(context),
|
||||||
|
|
|
@ -1,5 +1,4 @@
|
||||||
import 'package:aves/theme/icons.dart';
|
import 'package:aves/theme/icons.dart';
|
||||||
import 'package:aves/widgets/common/animated_icons_fix.dart';
|
|
||||||
import 'package:aves/widgets/common/extensions/build_context.dart';
|
import 'package:aves/widgets/common/extensions/build_context.dart';
|
||||||
import 'package:aves/widgets/common/identity/empty.dart';
|
import 'package:aves/widgets/common/identity/empty.dart';
|
||||||
import 'package:aves/widgets/common/identity/highlight_title.dart';
|
import 'package:aves/widgets/common/identity/highlight_title.dart';
|
||||||
|
@ -21,9 +20,8 @@ class SettingsSearchDelegate extends SearchDelegate {
|
||||||
@override
|
@override
|
||||||
Widget buildLeading(BuildContext context) {
|
Widget buildLeading(BuildContext context) {
|
||||||
return IconButton(
|
return IconButton(
|
||||||
// TODO TLAD [rtl] replace to regular `AnimatedIcon` when this is fixed: https://github.com/flutter/flutter/issues/60521
|
icon: AnimatedIcon(
|
||||||
icon: AnimatedIconFixIssue60521(
|
icon: AnimatedIcons.menu_arrow,
|
||||||
icon: AnimatedIconsFixIssue60521.menu_arrow,
|
|
||||||
progress: transitionAnimation,
|
progress: transitionAnimation,
|
||||||
),
|
),
|
||||||
onPressed: () => Navigator.pop(context),
|
onPressed: () => Navigator.pop(context),
|
||||||
|
|
|
@ -1,6 +1,5 @@
|
||||||
import 'package:aves/model/entry.dart';
|
import 'package:aves/model/entry.dart';
|
||||||
import 'package:aves/theme/icons.dart';
|
import 'package:aves/theme/icons.dart';
|
||||||
import 'package:aves/widgets/common/animated_icons_fix.dart';
|
|
||||||
import 'package:aves/widgets/common/extensions/build_context.dart';
|
import 'package:aves/widgets/common/extensions/build_context.dart';
|
||||||
import 'package:aves/widgets/common/identity/empty.dart';
|
import 'package:aves/widgets/common/identity/empty.dart';
|
||||||
import 'package:aves/widgets/common/providers/media_query_data_provider.dart';
|
import 'package:aves/widgets/common/providers/media_query_data_provider.dart';
|
||||||
|
@ -27,9 +26,8 @@ class InfoSearchDelegate extends SearchDelegate {
|
||||||
@override
|
@override
|
||||||
Widget buildLeading(BuildContext context) {
|
Widget buildLeading(BuildContext context) {
|
||||||
return IconButton(
|
return IconButton(
|
||||||
// TODO TLAD [rtl] replace to regular `AnimatedIcon` when this is fixed: https://github.com/flutter/flutter/issues/60521
|
icon: AnimatedIcon(
|
||||||
icon: AnimatedIconFixIssue60521(
|
icon: AnimatedIcons.menu_arrow,
|
||||||
icon: AnimatedIconsFixIssue60521.menu_arrow,
|
|
||||||
progress: transitionAnimation,
|
progress: transitionAnimation,
|
||||||
),
|
),
|
||||||
onPressed: () => Navigator.pop(context),
|
onPressed: () => Navigator.pop(context),
|
||||||
|
|
Loading…
Reference in a new issue