Phase 1 audit fixes: - Use FadeSlidePageRoute for DebugLogScreen navigation (consistent transitions) - Add volumePrecisionMode and volumePrecisionModeDescription to localizations - Replace hardcoded empty state strings in all row widgets with localized versions: - album_row, artist_row, track_row, playlist_row - podcast_row, radio_station_row, audiobook_row, series_row Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
195 lines
6.0 KiB
Dart
195 lines
6.0 KiB
Dart
import 'package:flutter/material.dart';
|
|
import 'package:provider/provider.dart';
|
|
import 'package:cached_network_image/cached_network_image.dart';
|
|
import '../l10n/app_localizations.dart';
|
|
import '../models/media_item.dart';
|
|
import '../providers/music_assistant_provider.dart';
|
|
import '../services/debug_logger.dart';
|
|
import 'radio_station_card.dart';
|
|
|
|
class RadioStationRow extends StatefulWidget {
|
|
final String title;
|
|
final Future<List<MediaItem>> Function() loadRadioStations;
|
|
final String? heroTagSuffix;
|
|
final double? rowHeight;
|
|
final List<MediaItem>? Function()? getCachedRadioStations;
|
|
|
|
const RadioStationRow({
|
|
super.key,
|
|
required this.title,
|
|
required this.loadRadioStations,
|
|
this.heroTagSuffix,
|
|
this.rowHeight,
|
|
this.getCachedRadioStations,
|
|
});
|
|
|
|
@override
|
|
State<RadioStationRow> createState() => _RadioStationRowState();
|
|
}
|
|
|
|
class _RadioStationRowState extends State<RadioStationRow> with AutomaticKeepAliveClientMixin {
|
|
List<MediaItem> _radioStations = [];
|
|
bool _isLoading = true;
|
|
bool _hasLoaded = false;
|
|
|
|
static final _logger = DebugLogger();
|
|
|
|
@override
|
|
bool get wantKeepAlive => true;
|
|
|
|
@override
|
|
void initState() {
|
|
super.initState();
|
|
// Get cached data synchronously BEFORE first build (no spinner flash)
|
|
final cached = widget.getCachedRadioStations?.call();
|
|
if (cached != null && cached.isNotEmpty) {
|
|
_radioStations = cached;
|
|
_isLoading = false;
|
|
}
|
|
_loadRadioStations();
|
|
}
|
|
|
|
Future<void> _loadRadioStations() async {
|
|
if (_hasLoaded) return;
|
|
_hasLoaded = true;
|
|
|
|
// Load fresh data
|
|
try {
|
|
final freshStations = await widget.loadRadioStations();
|
|
if (mounted && freshStations.isNotEmpty) {
|
|
setState(() {
|
|
_radioStations = freshStations;
|
|
_isLoading = false;
|
|
});
|
|
// Pre-cache images for smooth hero animations
|
|
_precacheRadioImages(freshStations);
|
|
}
|
|
} catch (e) {
|
|
// Silent failure - keep showing cached data
|
|
}
|
|
|
|
if (mounted && _isLoading) {
|
|
setState(() => _isLoading = false);
|
|
}
|
|
}
|
|
|
|
void _precacheRadioImages(List<MediaItem> stations) {
|
|
if (!mounted) return;
|
|
final maProvider = context.read<MusicAssistantProvider>();
|
|
|
|
final stationsToCache = stations.take(10);
|
|
|
|
for (final station in stationsToCache) {
|
|
final imageUrl = maProvider.api?.getImageUrl(station, size: 256);
|
|
if (imageUrl != null) {
|
|
precacheImage(
|
|
CachedNetworkImageProvider(imageUrl),
|
|
context,
|
|
).catchError((_) => false);
|
|
}
|
|
}
|
|
}
|
|
|
|
Widget _buildContent(double contentHeight, ColorScheme colorScheme) {
|
|
// Only show loading if we have no data at all
|
|
if (_radioStations.isEmpty && _isLoading) {
|
|
return const Center(child: CircularProgressIndicator());
|
|
}
|
|
|
|
if (_radioStations.isEmpty) {
|
|
return Center(
|
|
child: Text(
|
|
S.of(context)!.noRadioStations,
|
|
style: TextStyle(color: colorScheme.onSurface.withOpacity(0.5)),
|
|
),
|
|
);
|
|
}
|
|
|
|
// Card layout: circle image + name below (same as ArtistRow)
|
|
// Text area: 8px gap + ~36px for 2-line name = ~44px
|
|
const textAreaHeight = 44.0;
|
|
final imageSize = contentHeight - textAreaHeight;
|
|
final cardWidth = imageSize; // Card width = image width (circle)
|
|
final itemExtent = cardWidth + 16; // width + horizontal margins
|
|
|
|
return ScrollConfiguration(
|
|
behavior: const _StretchScrollBehavior(),
|
|
child: ListView.builder(
|
|
scrollDirection: Axis.horizontal,
|
|
padding: const EdgeInsets.symmetric(horizontal: 12.0),
|
|
itemCount: _radioStations.length,
|
|
itemExtent: itemExtent,
|
|
cacheExtent: 500, // Preload ~3 items ahead for smoother scrolling
|
|
addAutomaticKeepAlives: false, // Row already uses AutomaticKeepAliveClientMixin
|
|
addRepaintBoundaries: false, // Cards already have RepaintBoundary
|
|
itemBuilder: (context, index) {
|
|
final station = _radioStations[index];
|
|
return Container(
|
|
key: ValueKey(station.uri ?? station.itemId),
|
|
width: cardWidth,
|
|
margin: const EdgeInsets.symmetric(horizontal: 8.0),
|
|
child: RadioStationCard(
|
|
radioStation: station,
|
|
heroTagSuffix: widget.heroTagSuffix,
|
|
imageCacheSize: 256,
|
|
),
|
|
);
|
|
},
|
|
),
|
|
);
|
|
}
|
|
|
|
@override
|
|
Widget build(BuildContext context) {
|
|
_logger.startBuild('RadioStationRow:${widget.title}');
|
|
super.build(context); // Required for AutomaticKeepAliveClientMixin
|
|
final textTheme = Theme.of(context).textTheme;
|
|
final colorScheme = Theme.of(context).colorScheme;
|
|
|
|
// Total row height includes title + content (same as ArtistRow)
|
|
final totalHeight = widget.rowHeight ?? 207.0; // Default: 44 title + 163 content
|
|
const titleHeight = 44.0; // 12 top padding + ~24 text + 8 bottom padding
|
|
final contentHeight = totalHeight - titleHeight;
|
|
|
|
final result = RepaintBoundary(
|
|
child: SizedBox(
|
|
height: totalHeight,
|
|
child: Column(
|
|
crossAxisAlignment: CrossAxisAlignment.start,
|
|
children: [
|
|
Padding(
|
|
padding: const EdgeInsets.fromLTRB(16.0, 12.0, 16.0, 8.0),
|
|
child: Text(
|
|
widget.title,
|
|
style: textTheme.titleLarge?.copyWith(
|
|
fontWeight: FontWeight.bold,
|
|
color: colorScheme.onBackground,
|
|
),
|
|
),
|
|
),
|
|
Expanded(
|
|
child: _buildContent(contentHeight, colorScheme),
|
|
),
|
|
],
|
|
),
|
|
),
|
|
);
|
|
_logger.endBuild('RadioStationRow:${widget.title}');
|
|
return result;
|
|
}
|
|
}
|
|
|
|
/// Custom scroll behavior that uses Android 12+ stretch overscroll effect
|
|
class _StretchScrollBehavior extends ScrollBehavior {
|
|
const _StretchScrollBehavior();
|
|
|
|
@override
|
|
Widget buildOverscrollIndicator(
|
|
BuildContext context, Widget child, ScrollableDetails details) {
|
|
return StretchingOverscrollIndicator(
|
|
axisDirection: details.direction,
|
|
child: child,
|
|
);
|
|
}
|
|
}
|