fix(flutter): 首页 feed 静默刷新与进度条细节

This commit is contained in:
Zhou Weidong committed 2026-10-20 14:11:42 +08:00
1 parent a3126723d2
commit 82fe237c24
8 files changed
+190 -17

No files matched your search

@@ -44,6 +44,19 @@ class CatalogController extends ChangeNotifier {
}
}
/// 静默刷新首页聚合:成功替换数据,失败返回 false。
///
/// 不触碰 [loading]/[error],因此页面不会切到全屏 loading 或清空旧数据。
Future<bool> refresh() async {
try {
feed = await repository.fetchHomeFeed();
notifyListeners();
return true;
} catch (_) {
return false;
}
}
/// 需要完整剧集时补一次详情;已加载过则直接返回,避免重复请求。
Future<Drama> requireDetail(Drama drama) async {
if (drama.detailLoaded) return drama;
+2 -2
View File
@@ -71,7 +71,7 @@ class HomePage extends StatelessWidget {
return _HomeTabs(
key: ValueKey(catalog.dramas.first.id),
dramas: catalog.dramas,
onRefreshFeed: catalog.load,
onRefreshFeed: catalog.refresh,
);
}
}
@@ -103,7 +103,7 @@ class _HomeTabs extends StatefulWidget {
});
final List<Drama> dramas;
final VoidCallback onRefreshFeed;
final Future<bool> Function() onRefreshFeed;
@override
State<_HomeTabs> createState() => _HomeTabsState();
+8 -2
View File
@@ -76,7 +76,7 @@ class PlayerPage extends StatefulWidget {
final ValueChanged<Drama>? onDramaChanged;
final ValueChanged<bool>? onImmersiveChanged;
final Future<void> Function(Drama drama, Episode episode)? onOpenFullPlayer;
final VoidCallback? onRefreshFeed;
final Future<bool> Function()? onRefreshFeed;
final WalletController? wallet;
final DownloadController? downloads;
@@ -420,7 +420,13 @@ class _PlayerPageState extends State<PlayerPage> with WidgetsBindingObserver {
if (!mounted || !widget.active) return;
logic.session.setUserWantsPlay(true);
},
onRefreshFeed: () => widget.onRefreshFeed?.call(),
onRefreshFeed: () async {
final ok = await widget.onRefreshFeed?.call() ?? true;
if (!context.mounted || ok) return;
ScaffoldMessenger.of(
context,
).showSnackBar(SnackBar(content: Text(l10n.catalogError)));
},
onDownload: () => requestEpisodeDownload(context, episode.id),
onClear: () => setState(() => _chromeVisible = false),
onSearch:
@@ -14,6 +14,7 @@ import 'interaction_rail.dart';
part 'home_feed_overlay_parts.dart';
const Color _overlayPill = Color(0x66000000);
const Color _fullBarBackground = Color(0x4D000000);
const double _topIconSize = 36;
const double _topIconGlyph = 22;
const double _tabGap = 22;
@@ -25,7 +26,7 @@ const double _metaFontSize = 12;
const double _tagFontSize = 11;
const double _fullBarHeight = 44;
const double _progressHeight = 2;
const double _progressHitHeight = 20;
const double _progressHitHeight = 24;
const int _maxTags = 4;
final RegExp _tagSplitter = RegExp(r'[,/、·]');
@@ -100,7 +101,7 @@ class _HomeFeedOverlayState extends State<HomeFeedOverlay> {
_header(context),
Expanded(
child: Align(
alignment: const Alignment(1.0, 0.45),
alignment: const Alignment(1.0, 0.68),
child: Padding(
padding: const EdgeInsets.fromLTRB(0, 0, 10, 16),
child: _rail(),
@@ -358,6 +359,12 @@ class _HomeFeedOverlayState extends State<HomeFeedOverlay> {
height: _fullBarHeight,
child: Stack(
children: [
Positioned.fill(
child: Container(
key: const Key('home-feed-full-series-bg'),
color: _fullBarBackground,
),
),
Positioned.fill(
child: Padding(
padding: const EdgeInsets.symmetric(horizontal: 14),
@@ -4,6 +4,8 @@ part of 'home_feed_overlay.dart';
const double _thumbSize = 6;
const double _thumbActiveSize = 10;
const double _trackActiveHeight = 4;
const double _lineCenterFromBottom = 6;
/// 顶部频道 tab:漫剧点击提示即将上线。
enum _FeedTab {
@@ -199,10 +201,6 @@ class _FeedProgressBarState extends State<_FeedProgressBar> {
return GestureDetector(
key: const Key('home-feed-progress'),
behavior: HitTestBehavior.opaque,
onTapDown: (details) {
widget.onScrubStart?.call();
_commit(details.localPosition.dx / width);
},
onHorizontalDragStart: (details) {
widget.onScrubStart?.call();
_update(details.localPosition.dx / width);
@@ -233,11 +231,14 @@ class _FeedProgressBarState extends State<_FeedProgressBar> {
}
Widget _track(double fraction, double width) {
final active = _preview != null;
final size = active ? _thumbActiveSize : _thumbSize;
final dragging = _preview != null;
final trackHeight = dragging ? _trackActiveHeight : widget.height;
final size = dragging ? _thumbActiveSize : _thumbSize;
final left = (width * fraction - size / 2)
.clamp(0.0, width - size)
.toDouble();
final lineBottom = _lineCenterFromBottom - trackHeight / 2;
final thumbBottom = _lineCenterFromBottom - size / 2;
return SizedBox(
height: widget.hitHeight,
child: Stack(
@@ -246,21 +247,25 @@ class _FeedProgressBarState extends State<_FeedProgressBar> {
Positioned(
left: 0,
right: 0,
bottom: 0,
child: Container(height: widget.height, color: Colors.white24),
bottom: lineBottom,
child: Container(
key: const Key('home-feed-progress-track'),
height: trackHeight,
color: Colors.white24,
),
),
Positioned(
left: 0,
bottom: 0,
bottom: lineBottom,
child: Container(
width: width * fraction,
height: widget.height,
height: trackHeight,
color: AppColors.blue,
),
),
Positioned(
left: left,
bottom: 0,
bottom: thumbBottom,
child: Container(
key: const Key('home-feed-progress-thumb'),
width: size,
@@ -0,0 +1,72 @@
// Author: fruit.com.cn
import 'package:flutter_test/flutter_test.dart';
import 'package:fruitshorts/core/data/models/drama.dart';
import 'package:fruitshorts/core/data/models/home_feed.dart';
import 'package:fruitshorts/core/data/repositories/catalog_repository.dart';
import 'package:fruitshorts/core/data/services/catalog_controller.dart';
void main() {
test('should_keep_state_when_silent_refresh_fails', () async {
final repository = _FakeCatalog(fail: true);
final controller = CatalogController(repository);
const oldFeed = HomeFeed();
controller.feed = oldFeed;
final ok = await controller.refresh();
expect(ok, isFalse);
expect(controller.feed, same(oldFeed));
expect(controller.loading, isFalse);
expect(controller.error, isNull);
});
test(
'should_replace_feed_without_touching_loading_when_refresh_succeeds',
() async {
final repository = _FakeCatalog();
final controller = CatalogController(repository);
controller.feed = const HomeFeed();
final next = HomeFeed(
sections: [
FeedSection(code: 'new', title: 'New', dramas: const [_drama]),
],
);
repository.next = next;
final ok = await controller.refresh();
expect(ok, isTrue);
expect(controller.feed, same(next));
expect(controller.loading, isFalse);
expect(controller.error, isNull);
},
);
}
class _FakeCatalog implements CatalogRepository {
_FakeCatalog({this.fail = false});
bool fail;
HomeFeed next = const HomeFeed();
@override
Future<HomeFeed> fetchHomeFeed() async {
if (fail) throw StateError('offline');
return next;
}
@override
Future<HomeFeed> fetchDiscoverFeed() async => const HomeFeed();
@override
Future<Drama> fetchDrama(String dramaId) async => _drama;
}
const _drama = Drama(
id: 'refresh-drama',
title: 'Refresh Drama',
synopsis: 'synopsis',
posterUrl: '',
detailLoaded: true,
);
+44
View File
@@ -211,6 +211,12 @@ void main() {
final bar = tester.getRect(find.byKey(const Key('home-feed-full-series')));
expect(bar.height, 44);
expect(
tester
.widget<Container>(find.byKey(const Key('home-feed-full-series-bg')))
.color,
const Color(0x4D000000),
);
expect(
tester.getRect(find.textContaining('Watch full series')).center.dy,
closeTo(bar.center.dy, 2),
@@ -219,6 +225,23 @@ void main() {
tester.getSize(find.byKey(const Key('home-feed-progress-thumb'))),
const Size(6, 6),
);
expect(
tester.getSize(find.byKey(const Key('home-feed-progress'))).height,
greaterThanOrEqualTo(24),
);
expect(
tester
.getRect(find.byKey(const Key('home-feed-progress-thumb')))
.center
.dy,
closeTo(
tester
.getRect(find.byKey(const Key('home-feed-progress-track')))
.center
.dy,
0.5,
),
);
await tester.tap(find.byKey(const Key('home-feed-refresh')));
await tester.pump();
@@ -233,6 +256,23 @@ void main() {
tester.getSize(find.byKey(const Key('home-feed-progress-thumb'))),
const Size(10, 10),
);
expect(
tester.getSize(find.byKey(const Key('home-feed-progress-track'))).height,
4,
);
expect(
tester
.getRect(find.byKey(const Key('home-feed-progress-thumb')))
.center
.dy,
closeTo(
tester
.getRect(find.byKey(const Key('home-feed-progress-track')))
.center
.dy,
0.5,
),
);
await gesture.up();
await tester.pumpAndSettle();
expect(seeked, isNotNull);
@@ -241,6 +281,10 @@ void main() {
tester.getSize(find.byKey(const Key('home-feed-progress-thumb'))),
const Size(6, 6),
);
expect(
tester.getSize(find.byKey(const Key('home-feed-progress-track'))).height,
2,
);
await tester.tap(find.byKey(const Key('home-feed-full-series')));
await tester.pump();
+26
View File
@@ -96,6 +96,32 @@ void main() {
expect(find.text('Coming soon'), findsOneWidget);
});
testWidgets('should_refresh_feed_without_blank_loading', (tester) async {
await _pumpApp(tester);
await tester.tap(find.byKey(const Key('home-feed-refresh')));
await tester.pump();
expect(find.byType(CircularProgressIndicator), findsNothing);
expect(find.byKey(const Key('home-feed-title')), findsOneWidget);
await tester.pumpAndSettle();
expect(find.byKey(const Key('home-feed-title')), findsOneWidget);
});
testWidgets('should_keep_feed_rail_above_bottom_panels', (tester) async {
await _pumpApp(tester);
final railTop = tester.getCenter(find.byKey(const Key('rail-save'))).dy;
final railBottom = tester
.getRect(find.byKey(const Key('rail-share')))
.bottom;
final infoTop = tester
.getRect(find.byKey(const Key('home-feed-series-meta')))
.top;
expect(railTop, greaterThan(844 * 0.35));
expect(railBottom, lessThanOrEqualTo(infoTop));
});
testWidgets('should_restore_light_theme_after_leaving_feed_tab', (
tester,
) async {