fix(flutter): 修复详情与播放器底部闪白

This commit is contained in:
Zhou Weidong committed 2026-10-20 19:04:25 +08:00
1 parent 9eec9ed77a
commit 0aa5a99a22
8 files changed
+118 -9

No files matched your search

+11 -5
View File
@@ -50,6 +50,16 @@ import 'core/theme/app_colors.dart';
import 'l10n/app_localizations.dart'; import 'l10n/app_localizations.dart';
import 'pages/home/home_view.dart'; import 'pages/home/home_view.dart';
/// 全局系统栏样式:浅暖导航栏、关闭 Android 对比遮罩。
final SystemUiOverlayStyle appSystemOverlayStyle = SystemUiOverlayStyle.dark
.copyWith(
statusBarColor: Colors.transparent,
systemNavigationBarColor: AppColors.ink,
systemNavigationBarDividerColor: Colors.transparent,
systemNavigationBarIconBrightness: Brightness.dark,
systemNavigationBarContrastEnforced: false,
);
class LumaDramaApp extends StatefulWidget { class LumaDramaApp extends StatefulWidget {
const LumaDramaApp({ const LumaDramaApp({
super.key, super.key,
@@ -263,11 +273,7 @@ class _LumaDramaAppState extends State<LumaDramaApp> {
theme: AppTheme.light(), theme: AppTheme.light(),
darkTheme: AppTheme.light(), darkTheme: AppTheme.light(),
builder: (context, child) => AnnotatedRegion<SystemUiOverlayStyle>( builder: (context, child) => AnnotatedRegion<SystemUiOverlayStyle>(
value: SystemUiOverlayStyle.dark.copyWith( value: appSystemOverlayStyle,
statusBarColor: Colors.transparent,
systemNavigationBarColor: AppColors.bar,
systemNavigationBarIconBrightness: Brightness.dark,
),
child: child ?? const SizedBox.shrink(), child: child ?? const SizedBox.shrink(),
), ),
home: const HomePage(), home: const HomePage(),
+5
View File
@@ -107,6 +107,11 @@ abstract final class AppTheme {
side: BorderSide(color: AppColors.border), side: BorderSide(color: AppColors.border),
), ),
), ),
bottomSheetTheme: const BottomSheetThemeData(
backgroundColor: Colors.transparent,
modalBackgroundColor: Colors.transparent,
surfaceTintColor: Colors.transparent,
),
switchTheme: SwitchThemeData( switchTheme: SwitchThemeData(
thumbColor: WidgetStateProperty.resolveWith( thumbColor: WidgetStateProperty.resolveWith(
(states) => states.contains(WidgetState.selected) (states) => states.contains(WidgetState.selected)
+12 -1
View File
@@ -3,6 +3,7 @@
import 'dart:async'; import 'dart:async';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import '../../core/data/models/drama.dart'; import '../../core/data/models/drama.dart';
@@ -83,12 +84,22 @@ class _DramaDetailPageState extends State<DramaDetailPage> {
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
return Theme( return AnnotatedRegion<SystemUiOverlayStyle>(
key: const Key('detail-system-overlay'),
value: SystemUiOverlayStyle.dark.copyWith(
statusBarColor: Colors.transparent,
systemNavigationBarColor: AppColors.ink,
systemNavigationBarDividerColor: Colors.transparent,
systemNavigationBarIconBrightness: Brightness.dark,
systemNavigationBarContrastEnforced: false,
),
child: Theme(
data: AppTheme.light(), data: AppTheme.light(),
child: ListenableBuilder( child: ListenableBuilder(
listenable: logic, listenable: logic,
builder: (context, _) => _page(context), builder: (context, _) => _page(context),
), ),
),
); );
} }
+8 -1
View File
@@ -238,7 +238,14 @@ class _PlayerPageState extends State<PlayerPage> with WidgetsBindingObserver {
], ],
); );
return AnnotatedRegion<SystemUiOverlayStyle>( return AnnotatedRegion<SystemUiOverlayStyle>(
value: SystemUiOverlayStyle.dark, key: const Key('player-system-overlay'),
value: SystemUiOverlayStyle.light.copyWith(
statusBarColor: Colors.transparent,
systemNavigationBarColor: Colors.black,
systemNavigationBarDividerColor: Colors.transparent,
systemNavigationBarIconBrightness: Brightness.light,
systemNavigationBarContrastEnforced: false,
),
child: Theme( child: Theme(
data: AppTheme.light(), data: AppTheme.light(),
child: widget.embedded child: widget.embedded
@@ -1,6 +1,7 @@
// Author: fruit.com.cn // Author: fruit.com.cn
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import '../../../core/data/models/drama.dart'; import '../../../core/data/models/drama.dart';
import '../../../core/formatting/localized_content.dart'; import '../../../core/formatting/localized_content.dart';
@@ -37,15 +38,27 @@ class EpisodePickerSheet extends StatelessWidget {
Widget build(BuildContext context) { Widget build(BuildContext context) {
final l10n = AppLocalizations.of(context)!; final l10n = AppLocalizations.of(context)!;
final episodes = drama.episodes; final episodes = drama.episodes;
return Container( return AnnotatedRegion<SystemUiOverlayStyle>(
key: const Key('player-episodes-system-overlay'),
value: SystemUiOverlayStyle.dark.copyWith(
statusBarColor: Colors.transparent,
systemNavigationBarColor: AppColors.card,
systemNavigationBarDividerColor: Colors.transparent,
systemNavigationBarIconBrightness: Brightness.dark,
systemNavigationBarContrastEnforced: false,
),
child: Container(
key: const Key('player-episodes-sheet'), key: const Key('player-episodes-sheet'),
height: MediaQuery.sizeOf(context).height * _sheetHeightFactor, height: MediaQuery.sizeOf(context).height * _sheetHeightFactor,
decoration: const BoxDecoration( decoration: const BoxDecoration(
color: AppColors.card, color: AppColors.card,
borderRadius: BorderRadius.vertical(top: Radius.circular(_sheetRadius)), borderRadius: BorderRadius.vertical(
top: Radius.circular(_sheetRadius),
),
), ),
child: SafeArea( child: SafeArea(
top: false, top: false,
bottom: true,
child: Column( child: Column(
crossAxisAlignment: CrossAxisAlignment.start, crossAxisAlignment: CrossAxisAlignment.start,
children: [ children: [
@@ -126,6 +139,7 @@ class EpisodePickerSheet extends StatelessWidget {
], ],
), ),
), ),
),
); );
} }
} }
@@ -1,6 +1,7 @@
// Author: fruit.com.cn // Author: fruit.com.cn
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart'; import 'package:flutter_test/flutter_test.dart';
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import 'package:shared_preferences/shared_preferences.dart'; import 'package:shared_preferences/shared_preferences.dart';
@@ -53,6 +54,11 @@ void main() {
) async { ) async {
await _pumpDetail(tester); await _pumpDetail(tester);
final overlay = tester.widget<AnnotatedRegion<SystemUiOverlayStyle>>(
find.byKey(const Key('detail-system-overlay')),
);
expect(overlay.value.systemNavigationBarColor, AppColors.ink);
expect(overlay.value.systemNavigationBarContrastEnforced, isFalse);
expect(find.byKey(const Key('detail-hero')), findsOneWidget); expect(find.byKey(const Key('detail-hero')), findsOneWidget);
expect( expect(
tester.getSize(find.byKey(const Key('detail-poster'))), tester.getSize(find.byKey(const Key('detail-poster'))),
@@ -92,6 +98,10 @@ void main() {
expect(find.byKey(const Key('detail-tags')), findsOneWidget); expect(find.byKey(const Key('detail-tags')), findsOneWidget);
expect(find.byKey(const Key('detail-episodes-section')), findsOneWidget); expect(find.byKey(const Key('detail-episodes-section')), findsOneWidget);
expect(tester.getSize(find.byKey(const Key('detail-cta'))).height, 46); expect(tester.getSize(find.byKey(const Key('detail-cta'))).height, 46);
final bottomBar = tester.widget<Container>(
find.byKey(const Key('detail-bottom-bar')),
);
expect((bottomBar.decoration! as BoxDecoration).color, AppColors.ink);
expect(find.byKey(const Key('detail-share')), findsOneWidget); expect(find.byKey(const Key('detail-share')), findsOneWidget);
expect(find.byKey(const Key('detail-more')), findsOneWidget); expect(find.byKey(const Key('detail-more')), findsOneWidget);
for (final key in const [Key('detail-share'), Key('detail-more')]) { for (final key in const [Key('detail-share'), Key('detail-more')]) {
+35
View File
@@ -1,6 +1,7 @@
// Author: fruit.com.cn // Author: fruit.com.cn
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart'; import 'package:flutter_test/flutter_test.dart';
import 'package:shared_preferences_platform_interface/in_memory_shared_preferences_async.dart'; import 'package:shared_preferences_platform_interface/in_memory_shared_preferences_async.dart';
import 'package:shared_preferences_platform_interface/shared_preferences_async_platform_interface.dart'; import 'package:shared_preferences_platform_interface/shared_preferences_async_platform_interface.dart';
@@ -50,6 +51,16 @@ void main() {
expect(tester.takeException(), isNull); expect(tester.takeException(), isNull);
}); });
testWidgets('should_use_black_system_navigation_bar_on_feed_and_player', (
tester,
) async {
await _pumpApp(tester);
_expectPlayerNavigationBar(tester);
await _openSeriesPlayer(tester);
_expectPlayerNavigationBar(tester);
});
testWidgets('should_fall_back_to_black_when_poster_is_empty', (tester) async { testWidgets('should_fall_back_to_black_when_poster_is_empty', (tester) async {
await _pumpApp( await _pumpApp(
tester, tester,
@@ -716,6 +727,20 @@ void main() {
final delegate = final delegate =
grid.gridDelegate as SliverGridDelegateWithFixedCrossAxisCount; grid.gridDelegate as SliverGridDelegateWithFixedCrossAxisCount;
expect(delegate.crossAxisCount, 6); expect(delegate.crossAxisCount, 6);
final overlay = tester.widget<AnnotatedRegion<SystemUiOverlayStyle>>(
find.byKey(const Key('player-episodes-system-overlay')),
);
expect(overlay.value.systemNavigationBarColor, AppColors.card);
expect(overlay.value.systemNavigationBarContrastEnforced, isFalse);
final safeArea = tester.widget<SafeArea>(
find
.descendant(
of: find.byKey(const Key('player-episodes-sheet')),
matching: find.byType(SafeArea),
)
.first,
);
expect(safeArea.bottom, isTrue);
expect(tester.takeException(), isNull); expect(tester.takeException(), isNull);
}); });
@@ -969,6 +994,16 @@ void main() {
}); });
} }
void _expectPlayerNavigationBar(WidgetTester tester) {
final overlay = tester.widget<AnnotatedRegion<SystemUiOverlayStyle>>(
find.byKey(const Key('player-system-overlay')).last,
);
expect(overlay.value.systemNavigationBarColor, Colors.black);
expect(overlay.value.systemNavigationBarDividerColor, Colors.transparent);
expect(overlay.value.systemNavigationBarContrastEnforced, isFalse);
expect(overlay.value.systemNavigationBarIconBrightness, Brightness.light);
}
void _expectFullscreenOverlay(WidgetTester tester) { void _expectFullscreenOverlay(WidgetTester tester) {
expect(find.byKey(const Key('player-back')), findsOneWidget); expect(find.byKey(const Key('player-back')), findsOneWidget);
expect(find.byKey(const Key('player-episode-heading')), findsOneWidget); expect(find.byKey(const Key('player-episode-heading')), findsOneWidget);
+21
View File
@@ -2,8 +2,10 @@
import 'dart:math'; import 'dart:math';
import 'package:flutter/material.dart';
import 'package:flutter/services.dart'; import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart'; import 'package:flutter_test/flutter_test.dart';
import 'package:fruitshorts/app.dart';
import 'package:fruitshorts/core/theme/app_colors.dart'; import 'package:fruitshorts/core/theme/app_colors.dart';
import 'package:fruitshorts/core/theme/app_theme.dart'; import 'package:fruitshorts/core/theme/app_theme.dart';
@@ -50,6 +52,25 @@ void main() {
expect(theme.appBarTheme.systemOverlayStyle, SystemUiOverlayStyle.dark); expect(theme.appBarTheme.systemOverlayStyle, SystemUiOverlayStyle.dark);
}); });
test('should_use_ink_navigation_bar_without_contrast_scrim', () {
expect(appSystemOverlayStyle.statusBarColor, Colors.transparent);
expect(appSystemOverlayStyle.systemNavigationBarColor, AppColors.ink);
expect(
appSystemOverlayStyle.systemNavigationBarDividerColor,
Colors.transparent,
);
expect(
appSystemOverlayStyle.systemNavigationBarIconBrightness,
Brightness.dark,
);
expect(appSystemOverlayStyle.systemNavigationBarContrastEnforced, isFalse);
final sheetTheme = AppTheme.light().bottomSheetTheme;
expect(sheetTheme.backgroundColor, Colors.transparent);
expect(sheetTheme.modalBackgroundColor, Colors.transparent);
expect(sheetTheme.surfaceTintColor, Colors.transparent);
});
test('should_expose_page_gradient_and_soft_card_shadow', () { test('should_expose_page_gradient_and_soft_card_shadow', () {
expect(AppTheme.pageGradient.colors, [AppColors.surfaceTop, AppColors.ink]); expect(AppTheme.pageGradient.colors, [AppColors.surfaceTop, AppColors.ink]);
expect(AppTheme.cardShadow.color, AppColors.cardShadow); expect(AppTheme.cardShadow.color, AppColors.cardShadow);