Compare commits

..

No commits in common. '9061470e9be43d605d4e39729aa342532ff80445' and '7269a56b384f15f793bfa57a81275988c2cbae47' have entirely different histories.

@ -9,18 +9,11 @@ class NostrRelay {
/// Whether the relay is enabled (should be used). /// Whether the relay is enabled (should be used).
bool isEnabled; bool isEnabled;
/// Number of consecutive connection failures.
int retryCount;
/// Maximum number of retry attempts before disabling.
static const int maxRetryAttempts = 3;
/// Creates a [NostrRelay] instance. /// Creates a [NostrRelay] instance.
NostrRelay({ NostrRelay({
required this.url, required this.url,
this.isConnected = false, this.isConnected = false,
this.isEnabled = true, this.isEnabled = true,
this.retryCount = 0,
}); });
/// Creates a [NostrRelay] from a URL string. /// Creates a [NostrRelay] from a URL string.
@ -30,7 +23,7 @@ class NostrRelay {
@override @override
String toString() { String toString() {
return 'NostrRelay(url: $url, connected: $isConnected, enabled: $isEnabled, retryCount: $retryCount)'; return 'NostrRelay(url: $url, connected: $isConnected, enabled: $isEnabled)';
} }
@override @override

@ -29,9 +29,6 @@ class NostrService {
final Map<String, StreamController<Map<String, dynamic>>> final Map<String, StreamController<Map<String, dynamic>>>
_messageControllers = {}; _messageControllers = {};
/// Retry timers for relays that are attempting to reconnect.
final Map<String, Timer> _retryTimers = {};
/// Creates a [NostrService] instance. /// Creates a [NostrService] instance.
NostrService(); NostrService();
@ -70,9 +67,6 @@ class NostrService {
orElse: () => throw NostrException('Relay not found: $relayUrl'), orElse: () => throw NostrException('Relay not found: $relayUrl'),
); );
relay.isEnabled = enabled; relay.isEnabled = enabled;
if (enabled) {
relay.retryCount = 0; // Reset retry count when enabling
}
// If disabling, also disconnect // If disabling, also disconnect
if (!enabled && relay.isConnected) { if (!enabled && relay.isConnected) {
@ -162,7 +156,6 @@ class NostrService {
// No errors occurred, connection is established // No errors occurred, connection is established
connectionConfirmed = true; connectionConfirmed = true;
relay.isConnected = true; relay.isConnected = true;
relay.retryCount = 0; // Reset retry count on successful connection
Logger.info('Connection confirmed for relay: $relayUrl (no errors after 500ms)'); Logger.info('Connection confirmed for relay: $relayUrl (no errors after 500ms)');
} }
}); });
@ -174,7 +167,6 @@ class NostrService {
if (!connectionConfirmed) { if (!connectionConfirmed) {
connectionConfirmed = true; connectionConfirmed = true;
relay.isConnected = true; relay.isConnected = true;
relay.retryCount = 0; // Reset retry count on successful connection
Logger.info('Connection confirmed for relay: $relayUrl (first message received)'); Logger.info('Connection confirmed for relay: $relayUrl (first message received)');
connectionTimer?.cancel(); connectionTimer?.cancel();
} }
@ -215,7 +207,9 @@ class NostrService {
connectionTimer?.cancel(); connectionTimer?.cancel();
Logger.error('WebSocket error for relay: $relayUrl', error); Logger.error('WebSocket error for relay: $relayUrl', error);
relay.isConnected = false; relay.isConnected = false;
_handleConnectionFailure(relayUrl, relay, controller, 'connection error'); // Automatically disable relay when connection error occurs
relay.isEnabled = false;
Logger.warning('Relay $relayUrl disabled due to connection error');
controller.addError(NostrException('Relay error: $error')); controller.addError(NostrException('Relay error: $error'));
}, },
onDone: () { onDone: () {
@ -223,7 +217,9 @@ class NostrService {
connectionTimer?.cancel(); connectionTimer?.cancel();
Logger.warning('WebSocket stream closed for relay: $relayUrl'); Logger.warning('WebSocket stream closed for relay: $relayUrl');
relay.isConnected = false; relay.isConnected = false;
_handleConnectionFailure(relayUrl, relay, controller, 'stream closure'); // Automatically disable relay when connection closes
relay.isEnabled = false;
Logger.warning('Relay $relayUrl disabled due to stream closure');
controller.close(); controller.close();
}, },
); );
@ -234,101 +230,6 @@ class NostrService {
} }
} }
/// Handles connection failure with retry logic.
///
/// [relayUrl] - The URL of the relay that failed.
/// [relay] - The relay object.
/// [controller] - The stream controller for this relay.
/// [reason] - Reason for the failure (for logging).
void _handleConnectionFailure(
String relayUrl,
NostrRelay relay,
StreamController<Map<String, dynamic>> controller,
String reason,
) {
// Cancel any existing retry timer
_retryTimers[relayUrl]?.cancel();
_retryTimers.remove(relayUrl);
// Only retry if relay is enabled
if (!relay.isEnabled) {
Logger.info('Relay $relayUrl is disabled, skipping retry');
return;
}
relay.retryCount++;
Logger.warning('Relay $relayUrl failed ($reason), retry attempt ${relay.retryCount}/${NostrRelay.maxRetryAttempts}');
if (relay.retryCount >= NostrRelay.maxRetryAttempts) {
// Max retries reached - disable the relay
relay.isEnabled = false;
relay.retryCount = 0; // Reset for future attempts
Logger.warning('Relay $relayUrl disabled after ${NostrRelay.maxRetryAttempts} failed attempts');
} else {
// Schedule retry after delay (5 seconds)
_scheduleRetry(relayUrl, relay, controller, reason);
}
}
/// Schedules a retry attempt for a failed relay connection.
///
/// [relayUrl] - The URL of the relay to retry.
/// [relay] - The relay object.
/// [controller] - The stream controller for this relay.
/// [reason] - Reason for the retry (for logging).
void _scheduleRetry(
String relayUrl,
NostrRelay relay,
StreamController<Map<String, dynamic>> controller,
String reason,
) {
Logger.info('Scheduling retry for relay $relayUrl in 5 seconds (attempt ${relay.retryCount}/${NostrRelay.maxRetryAttempts})');
_retryTimers[relayUrl] = Timer(const Duration(seconds: 5), () {
_retryTimers.remove(relayUrl);
if (relay.isEnabled && !relay.isConnected) {
Logger.info('Retrying connection to relay: $relayUrl');
try {
// Clean up old connection
disconnectRelay(relayUrl);
// Attempt to reconnect
connectRelay(relayUrl).then((stream) {
// Reset retry count on successful connection
relay.retryCount = 0;
Logger.info('Relay $relayUrl reconnected successfully');
// Cancel the stream subscription as we're just testing the connection
stream.listen(null).cancel();
}).catchError((e) {
Logger.warning('Retry failed for relay $relayUrl: $e');
// Handle retry failure - check if we should retry again or disable
if (relay.isEnabled) {
if (relay.retryCount >= NostrRelay.maxRetryAttempts) {
relay.isEnabled = false;
relay.retryCount = 0;
Logger.warning('Relay $relayUrl disabled after ${NostrRelay.maxRetryAttempts} failed attempts');
} else {
// Schedule another retry (retryCount already incremented in _handleConnectionFailure)
_scheduleRetry(relayUrl, relay, controller, 'retry failure');
}
}
});
} catch (e) {
Logger.error('Error during retry for relay $relayUrl', e);
// Handle synchronous exception
if (relay.isEnabled) {
if (relay.retryCount >= NostrRelay.maxRetryAttempts) {
relay.isEnabled = false;
relay.retryCount = 0;
Logger.warning('Relay $relayUrl disabled after ${NostrRelay.maxRetryAttempts} failed attempts');
} else {
// Schedule another retry (retryCount already incremented in _handleConnectionFailure)
_scheduleRetry(relayUrl, relay, controller, 'synchronous retry failure');
}
}
}
}
});
}
/// Disconnects from a relay. /// Disconnects from a relay.
/// ///
/// [relayUrl] - The URL of the relay to disconnect from. /// [relayUrl] - The URL of the relay to disconnect from.

@ -553,12 +553,9 @@ class _AddRecipeScreenState extends State<AddRecipeScreen> {
Future<void> _removeTag(String tag) async { Future<void> _removeTag(String tag) async {
if (_recipeService == null || _recipe == null) return; if (_recipeService == null || _recipe == null) return;
final currentRecipe = _currentRecipe ?? widget.recipe;
if (currentRecipe == null) return;
try { try {
final updatedTags = List<String>.from(currentRecipe.tags)..remove(tag); final updatedTags = List<String>.from(_recipe!.tags)..remove(tag);
final updatedRecipe = currentRecipe.copyWith(tags: updatedTags); final updatedRecipe = _recipe!.copyWith(tags: updatedTags);
await _recipeService!.updateRecipe(updatedRecipe); await _recipeService!.updateRecipe(updatedRecipe);
if (mounted) { if (mounted) {
@ -594,7 +591,7 @@ class _AddRecipeScreenState extends State<AddRecipeScreen> {
} }
final existingTags = allTags.toList()..sort(); final existingTags = allTags.toList()..sort();
final result = await showDialog<List<String>>( final result = await showDialog<String>(
context: context, context: context,
builder: (context) => _AddTagDialog( builder: (context) => _AddTagDialog(
tagController: tagController, tagController: tagController,
@ -607,8 +604,8 @@ class _AddRecipeScreenState extends State<AddRecipeScreen> {
tagController.dispose(); tagController.dispose();
}); });
if (mounted && result != null && result.isNotEmpty) { if (mounted && result != null && result.trim().isNotEmpty) {
await _addTags(result); await _addTag(result.trim());
} }
} catch (e) { } catch (e) {
WidgetsBinding.instance.addPostFrameCallback((_) { WidgetsBinding.instance.addPostFrameCallback((_) {
@ -617,22 +614,17 @@ class _AddRecipeScreenState extends State<AddRecipeScreen> {
} }
} }
Future<void> _addTags(List<String> tags) async { Future<void> _addTag(String tag) async {
if (_recipeService == null || _recipe == null) return; if (_recipeService == null || _recipe == null) return;
final currentRecipe = _currentRecipe ?? widget.recipe; final trimmedTag = tag.trim();
if (currentRecipe == null) return; if (trimmedTag.isEmpty) return;
final trimmedTags = tags.map((tag) => tag.trim()).where((tag) => tag.isNotEmpty).toList();
if (trimmedTags.isEmpty) return;
// Filter out tags that already exist if (_recipe!.tags.contains(trimmedTag)) {
final newTags = trimmedTags.where((tag) => !currentRecipe.tags.contains(tag)).toList();
if (newTags.isEmpty) {
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar( ScaffoldMessenger.of(context).showSnackBar(
const SnackBar( const SnackBar(
content: Text('All tags already exist'), content: Text('Tag already exists'),
duration: Duration(seconds: 1), duration: Duration(seconds: 1),
), ),
); );
@ -641,8 +633,8 @@ class _AddRecipeScreenState extends State<AddRecipeScreen> {
} }
try { try {
final updatedTags = List<String>.from(currentRecipe.tags)..addAll(newTags); final updatedTags = List<String>.from(_recipe!.tags)..add(trimmedTag);
final updatedRecipe = currentRecipe.copyWith(tags: updatedTags); final updatedRecipe = _recipe!.copyWith(tags: updatedTags);
await _recipeService!.updateRecipe(updatedRecipe); await _recipeService!.updateRecipe(updatedRecipe);
if (mounted) { if (mounted) {
@ -651,16 +643,16 @@ class _AddRecipeScreenState extends State<AddRecipeScreen> {
_tagsController.text = updatedTags.join(', '); _tagsController.text = updatedTags.join(', ');
}); });
ScaffoldMessenger.of(context).showSnackBar( ScaffoldMessenger.of(context).showSnackBar(
SnackBar( const SnackBar(
content: Text('${newTags.length} tag${newTags.length > 1 ? 's' : ''} added'), content: Text('Tag added'),
duration: const Duration(seconds: 1), duration: Duration(seconds: 1),
), ),
); );
} }
} catch (e) { } catch (e) {
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar( ScaffoldMessenger.of(context).showSnackBar(
SnackBar(content: Text('Error adding tags: $e')), SnackBar(content: Text('Error adding tag: $e')),
); );
} }
} }
@ -1789,7 +1781,7 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
late AnimationController _animationController; late AnimationController _animationController;
late Animation<double> _animation; late Animation<double> _animation;
bool _isUserScrolling = false; bool _isUserScrolling = false;
final Set<String> _selectedTags = {}; DateTime? _lastUserScrollTime;
@override @override
void initState() { void initState() {
@ -1854,47 +1846,16 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
WidgetsBinding.instance.addPostFrameCallback((_) { WidgetsBinding.instance.addPostFrameCallback((_) {
if (_scrollController.hasClients) { if (_scrollController.hasClients) {
_scrollController.jumpTo(0); _scrollController.jumpTo(0);
// Don't restart auto-scroll if user has interacted _isUserScrolling = false;
if (!_isUserScrolling) { _lastUserScrollTime = null;
_startAutoScroll(); _startAutoScroll();
}
} }
}); });
}); });
} }
void _selectTag(String tag) { void _selectTag(String tag) {
// Stop auto-scroll permanently when user selects a tag Navigator.pop(context, tag);
_isUserScrolling = true;
_animationController.stop();
setState(() {
if (_selectedTags.contains(tag)) {
_selectedTags.remove(tag);
} else {
_selectedTags.add(tag);
}
});
}
List<String> _getTagsToAdd() {
final tags = <String>[];
// Add selected tags from the list
tags.addAll(_selectedTags);
// Parse comma-separated tags from text field
final textTags = widget.tagController.text
.split(',')
.map((tag) => tag.trim())
.where((tag) => tag.isNotEmpty)
.toList();
tags.addAll(textTags);
// Remove duplicates and tags that already exist in the recipe
final uniqueTags = tags.toSet().where((tag) => !widget.currentTags.contains(tag)).toList();
return uniqueTags;
} }
@override @override
@ -1916,11 +1877,16 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
controller: widget.tagController, controller: widget.tagController,
autofocus: true, autofocus: true,
decoration: InputDecoration( decoration: InputDecoration(
hintText: 'Enter tags (comma-separated) or select from existing', hintText: 'Enter tag name or select from existing',
border: const OutlineInputBorder(), border: const OutlineInputBorder(),
prefixIcon: const Icon(Icons.search), prefixIcon: const Icon(Icons.search),
), ),
textCapitalization: TextCapitalization.none, textCapitalization: TextCapitalization.none,
onSubmitted: (value) {
if (value.trim().isNotEmpty) {
Navigator.pop(context, value.trim());
}
},
), ),
const SizedBox(height: 16), const SizedBox(height: 16),
// Existing tags scrollable list with fade hint // Existing tags scrollable list with fade hint
@ -1936,11 +1902,26 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
height: 50, height: 50,
child: NotificationListener<ScrollNotification>( child: NotificationListener<ScrollNotification>(
onNotification: (notification) { onNotification: (notification) {
// Detect when user manually scrolls - stop auto-scroll permanently // Detect when user manually scrolls
if (notification is ScrollStartNotification && notification.dragDetails != null) { if (notification is ScrollStartNotification && notification.dragDetails != null) {
_isUserScrolling = true; _isUserScrolling = true;
_animationController.stop(); _lastUserScrollTime = DateTime.now();
} _animationController.stop();
} else if (notification is ScrollEndNotification) {
// Resume auto-scroll after user stops scrolling for 3 seconds
_lastUserScrollTime = DateTime.now();
Future.delayed(const Duration(seconds: 3), () {
if (mounted && _lastUserScrollTime != null) {
final timeSinceLastScroll = DateTime.now().difference(_lastUserScrollTime!);
if (timeSinceLastScroll.inSeconds >= 3) {
setState(() {
_isUserScrolling = false;
});
_startAutoScroll();
}
}
});
}
return false; return false;
}, },
child: ListView.builder( child: ListView.builder(
@ -1950,34 +1931,11 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
itemBuilder: (context, index) { itemBuilder: (context, index) {
final tag = _filteredTags[index]; final tag = _filteredTags[index];
final isAlreadyAdded = widget.currentTags.contains(tag); final isAlreadyAdded = widget.currentTags.contains(tag);
final isSelected = _selectedTags.contains(tag);
return Padding( return Padding(
padding: const EdgeInsets.only(right: 8), padding: const EdgeInsets.only(right: 8),
child: FilterChip( child: FilterChip(
label: Text( label: Text(tag),
tag, selected: isAlreadyAdded,
style: TextStyle(
color: isSelected
? Colors.white
: (isAlreadyAdded
? (isDark ? Colors.white : Colors.black87)
: (isDark ? Colors.white : Colors.black87)),
fontWeight: isSelected
? FontWeight.w700
: (isAlreadyAdded ? FontWeight.w500 : FontWeight.normal),
fontSize: isSelected ? 14 : null,
shadows: isSelected
? [
Shadow(
color: Colors.black.withValues(alpha: 0.5),
offset: const Offset(0, 1),
blurRadius: 2,
),
]
: null,
),
),
selected: isSelected,
onSelected: isAlreadyAdded onSelected: isAlreadyAdded
? null ? null
: (selected) { : (selected) {
@ -1988,24 +1946,12 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
backgroundColor: isDark backgroundColor: isDark
? Colors.grey[800] ? Colors.grey[800]
: Colors.grey[200], : Colors.grey[200],
disabledColor: isDark disabledColor: Colors.grey[400],
? Colors.grey[600] labelStyle: TextStyle(
: Colors.grey[100], color: isAlreadyAdded
side: isSelected ? Colors.white
? BorderSide( : (isDark ? Colors.white : Colors.black87),
color: Colors.white.withValues(alpha: 0.5), ),
width: 2.5,
)
: (isAlreadyAdded
? BorderSide(
color: (isDark ? Colors.grey[500]! : Colors.grey[400]!),
width: 1,
)
: null),
elevation: isSelected ? 3 : 0,
padding: isSelected
? const EdgeInsets.symmetric(horizontal: 12, vertical: 8)
: null,
), ),
); );
}, },
@ -2029,16 +1975,12 @@ class _AddTagDialogState extends State<_AddTagDialog> with SingleTickerProviderS
), ),
FilledButton( FilledButton(
onPressed: () { onPressed: () {
final tagsToAdd = _getTagsToAdd(); final text = widget.tagController.text.trim();
if (tagsToAdd.isNotEmpty) { if (text.isNotEmpty) {
Navigator.pop(context, tagsToAdd); Navigator.pop(context, text);
} else {
Navigator.pop(context);
} }
}, },
child: Text(_selectedTags.isEmpty && widget.tagController.text.trim().isEmpty child: const Text('Add'),
? 'Add'
: 'Add (${_getTagsToAdd().length})'),
), ),
], ],
); );

@ -23,25 +23,12 @@ class _BookmarksScreenState extends State<BookmarksScreen> {
String? _errorMessage; String? _errorMessage;
RecipeService? _recipeService; RecipeService? _recipeService;
bool _wasLoggedIn = false; bool _wasLoggedIn = false;
String _searchQuery = '';
final TextEditingController _searchController = TextEditingController();
@override @override
void initState() { void initState() {
super.initState(); super.initState();
_checkLoginState(); _checkLoginState();
_initializeService(); _initializeService();
_searchController.addListener(() {
setState(() {
_searchQuery = _searchController.text.toLowerCase();
});
});
}
@override
void dispose() {
_searchController.dispose();
super.dispose();
} }
@override @override
@ -136,29 +123,6 @@ class _BookmarksScreenState extends State<BookmarksScreen> {
} }
} }
List<BookmarkCategory> _getFilteredCategories() {
if (_searchQuery.isEmpty) {
return _categories;
}
return _categories.where((category) {
final recipes = _recipesByCategory[category.id] ?? [];
return category.name.toLowerCase().contains(_searchQuery) ||
recipes.any((recipe) =>
recipe.title.toLowerCase().contains(_searchQuery) ||
(recipe.description?.toLowerCase().contains(_searchQuery) ?? false));
}).toList();
}
List<RecipeModel> _getFilteredRecipes(String categoryId) {
final recipes = _recipesByCategory[categoryId] ?? [];
if (_searchQuery.isEmpty) {
return recipes;
}
return recipes.where((recipe) =>
recipe.title.toLowerCase().contains(_searchQuery) ||
(recipe.description?.toLowerCase().contains(_searchQuery) ?? false)).toList();
}
void _viewRecipe(RecipeModel recipe) { void _viewRecipe(RecipeModel recipe) {
Navigator.push( Navigator.push(
context, context,
@ -194,36 +158,8 @@ class _BookmarksScreenState extends State<BookmarksScreen> {
if (!isLoggedIn) { if (!isLoggedIn) {
return Scaffold( return Scaffold(
appBar: AppBar(title: const Text('Bookmarks')), appBar: AppBar(title: const Text('Bookmarks')),
body: Center( body: const Center(
child: Padding( child: Text('Please log in to view bookmarks'),
padding: const EdgeInsets.all(24),
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
Icon(
Icons.bookmark_border,
size: 80,
color: Colors.grey.shade400,
),
const SizedBox(height: 24),
Text(
'Please log in to view bookmarks',
style: Theme.of(context).textTheme.titleLarge?.copyWith(
color: Colors.grey.shade700,
),
textAlign: TextAlign.center,
),
const SizedBox(height: 8),
Text(
'Sign in to save and organize your favorite recipes',
style: Theme.of(context).textTheme.bodyMedium?.copyWith(
color: Colors.grey.shade600,
),
textAlign: TextAlign.center,
),
],
),
),
), ),
); );
} }
@ -239,41 +175,16 @@ class _BookmarksScreenState extends State<BookmarksScreen> {
return Scaffold( return Scaffold(
appBar: AppBar(title: const Text('Bookmarks')), appBar: AppBar(title: const Text('Bookmarks')),
body: Center( body: Center(
child: Padding( child: Column(
padding: const EdgeInsets.all(24), mainAxisAlignment: MainAxisAlignment.center,
child: Column( children: [
mainAxisAlignment: MainAxisAlignment.center, Text(_errorMessage!),
children: [ const SizedBox(height: 16),
Icon( ElevatedButton(
Icons.error_outline, onPressed: _loadBookmarks,
size: 64, child: const Text('Retry'),
color: Colors.red.shade300, ),
), ],
const SizedBox(height: 16),
Text(
'Oops! Something went wrong',
style: Theme.of(context).textTheme.titleLarge,
textAlign: TextAlign.center,
),
const SizedBox(height: 8),
Text(
_errorMessage!,
style: Theme.of(context).textTheme.bodyMedium?.copyWith(
color: Colors.grey.shade600,
),
textAlign: TextAlign.center,
),
const SizedBox(height: 24),
ElevatedButton.icon(
onPressed: _loadBookmarks,
icon: const Icon(Icons.refresh),
label: const Text('Retry'),
style: ElevatedButton.styleFrom(
padding: const EdgeInsets.symmetric(horizontal: 24, vertical: 12),
),
),
],
),
), ),
), ),
); );
@ -282,287 +193,68 @@ class _BookmarksScreenState extends State<BookmarksScreen> {
if (_categories.isEmpty) { if (_categories.isEmpty) {
return Scaffold( return Scaffold(
appBar: AppBar(title: const Text('Bookmarks')), appBar: AppBar(title: const Text('Bookmarks')),
body: Center( body: const Center(
child: Padding( child: Text(
padding: const EdgeInsets.all(24), 'No bookmark categories yet.\nBookmark a recipe to get started!',
child: Column( textAlign: TextAlign.center,
mainAxisAlignment: MainAxisAlignment.center,
children: [
Icon(
Icons.bookmark_add_outlined,
size: 100,
color: Colors.grey.shade300,
),
const SizedBox(height: 24),
Text(
'No bookmark categories yet',
style: Theme.of(context).textTheme.headlineSmall?.copyWith(
fontWeight: FontWeight.bold,
color: Colors.grey.shade700,
),
textAlign: TextAlign.center,
),
const SizedBox(height: 12),
Text(
'Bookmark a recipe to organize your favorites into categories',
style: Theme.of(context).textTheme.bodyLarge?.copyWith(
color: Colors.grey.shade600,
),
textAlign: TextAlign.center,
),
],
),
), ),
), ),
); );
} }
final filteredCategories = _getFilteredCategories();
return Scaffold( return Scaffold(
appBar: AppBar( appBar: AppBar(
title: const Text('Bookmarks'), title: const Text('Bookmarks'),
elevation: 0, actions: [
],
), ),
body: RefreshIndicator( body: RefreshIndicator(
onRefresh: _loadBookmarks, onRefresh: _loadBookmarks,
child: CustomScrollView( child: ListView.builder(
slivers: [ padding: const EdgeInsets.all(16),
// Search bar itemCount: _categories.length,
SliverToBoxAdapter( itemBuilder: (context, index) {
child: Padding( final category = _categories[index];
padding: const EdgeInsets.fromLTRB(16, 16, 16, 8), final recipes = _recipesByCategory[category.id] ?? [];
child: TextField(
controller: _searchController, return Card(
decoration: InputDecoration( margin: const EdgeInsets.only(bottom: 16),
hintText: 'Search bookmarks...', child: ExpansionTile(
prefixIcon: const Icon(Icons.search), leading: category.color != null
suffixIcon: _searchQuery.isNotEmpty ? Container(
? IconButton( width: 32,
icon: const Icon(Icons.clear), height: 32,
onPressed: () { decoration: BoxDecoration(
_searchController.clear(); color: _parseColor(category.color!),
}, shape: BoxShape.circle,
)
: null,
filled: true,
fillColor: Theme.of(context).brightness == Brightness.dark
? Colors.grey[800]
: Colors.grey[100],
border: OutlineInputBorder(
borderRadius: BorderRadius.circular(12),
borderSide: BorderSide.none,
),
contentPadding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
),
),
),
),
// Categories list
if (filteredCategories.isEmpty && _searchQuery.isNotEmpty)
SliverFillRemaining(
hasScrollBody: false,
child: Center(
child: Padding(
padding: const EdgeInsets.all(24),
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
Icon(
Icons.search_off,
size: 64,
color: Colors.grey.shade400,
),
const SizedBox(height: 16),
Text(
'No results found',
style: Theme.of(context).textTheme.titleLarge?.copyWith(
color: Colors.grey.shade700,
),
),
const SizedBox(height: 8),
Text(
'Try a different search term',
style: Theme.of(context).textTheme.bodyMedium?.copyWith(
color: Colors.grey.shade600,
),
), ),
], )
), : const Icon(Icons.bookmark),
), title: Text(
), category.name,
) style: const TextStyle(fontWeight: FontWeight.bold),
else
SliverPadding(
padding: const EdgeInsets.all(16),
sliver: SliverList(
delegate: SliverChildBuilderDelegate(
(context, index) {
final category = filteredCategories[index];
final recipes = _getFilteredRecipes(category.id);
return _CategoryCard(
category: category,
recipes: recipes,
categoryColor: category.color != null
? _parseColor(category.color!)
: Theme.of(context).primaryColor,
onRecipeTap: _viewRecipe,
onPhotoTap: _openPhotoGallery,
);
},
childCount: filteredCategories.length,
),
), ),
), subtitle: Text('${recipes.length} recipe(s)'),
], children: recipes.isEmpty
), ? [
), const ListTile(
); title: Text('No recipes in this category'),
} )
} ]
: recipes.map((recipe) {
/// Modern category card widget
class _CategoryCard extends StatefulWidget {
final BookmarkCategory category;
final List<RecipeModel> recipes;
final Color categoryColor;
final Function(RecipeModel) onRecipeTap;
final Function(List<String>, int) onPhotoTap;
const _CategoryCard({
required this.category,
required this.recipes,
required this.categoryColor,
required this.onRecipeTap,
required this.onPhotoTap,
});
@override
State<_CategoryCard> createState() => _CategoryCardState();
}
class _CategoryCardState extends State<_CategoryCard> {
bool _isExpanded = false;
@override
Widget build(BuildContext context) {
return Card(
margin: const EdgeInsets.only(bottom: 16),
elevation: 2,
shape: RoundedRectangleBorder(
borderRadius: BorderRadius.circular(16),
),
child: Column(
children: [
InkWell(
onTap: () {
setState(() {
_isExpanded = !_isExpanded;
});
},
borderRadius: const BorderRadius.vertical(top: Radius.circular(16)),
child: Padding(
padding: const EdgeInsets.all(16),
child: Row(
children: [
// Category icon/color indicator
Container(
width: 48,
height: 48,
decoration: BoxDecoration(
color: widget.categoryColor.withOpacity(0.2),
borderRadius: BorderRadius.circular(12),
border: Border.all(
color: widget.categoryColor,
width: 2,
),
),
child: Icon(
Icons.bookmark,
color: widget.categoryColor,
size: 24,
),
),
const SizedBox(width: 16),
// Category info
Expanded(
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
Text(
widget.category.name,
style: Theme.of(context).textTheme.titleLarge?.copyWith(
fontWeight: FontWeight.bold,
),
),
const SizedBox(height: 4),
Row(
children: [
Icon(
Icons.restaurant_menu,
size: 16,
color: Colors.grey.shade600,
),
const SizedBox(width: 4),
Text(
'${widget.recipes.length} recipe${widget.recipes.length != 1 ? 's' : ''}',
style: Theme.of(context).textTheme.bodyMedium?.copyWith(
color: Colors.grey.shade600,
),
),
],
),
],
),
),
// Expand icon
Icon(
_isExpanded ? Icons.expand_less : Icons.expand_more,
color: Colors.grey.shade600,
),
],
),
),
),
// Recipes list
if (_isExpanded)
AnimatedSize(
duration: const Duration(milliseconds: 200),
child: widget.recipes.isEmpty
? Padding(
padding: const EdgeInsets.all(16),
child: Row(
children: [
Icon(
Icons.info_outline,
size: 20,
color: Colors.grey.shade400,
),
const SizedBox(width: 8),
Text(
'No recipes in this category',
style: Theme.of(context).textTheme.bodyMedium?.copyWith(
color: Colors.grey.shade600,
),
),
],
),
)
: Column(
children: widget.recipes.map((recipe) {
return _BookmarkRecipeItem( return _BookmarkRecipeItem(
recipe: recipe, recipe: recipe,
onTap: () => widget.onRecipeTap(recipe), onTap: () => _viewRecipe(recipe),
onPhotoTap: (index) => widget.onPhotoTap( onPhotoTap: (index) => _openPhotoGallery(
recipe.imageUrls, recipe.imageUrls,
index, index,
), ),
); );
}).toList(), }).toList(),
), ),
), );
], },
),
), ),
); );
} }
@ -661,130 +353,58 @@ class _BookmarkRecipeItem extends StatelessWidget {
); );
} }
Color _getRatingColor(double rating) {
if (rating >= 4.5) return Colors.green;
if (rating >= 3.5) return Colors.orange;
if (rating >= 2.5) return Colors.amber;
return Colors.red;
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
return InkWell( return ListTile(
onTap: onTap, leading: recipe.imageUrls.isNotEmpty
child: Padding( ? GestureDetector(
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12), onTap: () => onPhotoTap(0),
child: Row( child: ClipRRect(
crossAxisAlignment: CrossAxisAlignment.start, borderRadius: BorderRadius.circular(8),
children: [ child: SizedBox(
// Recipe image width: 60,
recipe.imageUrls.isNotEmpty height: 60,
? GestureDetector( child: _buildRecipeImage(recipe.imageUrls.first),
onTap: () => onPhotoTap(0), ),
child: ClipRRect(
borderRadius: BorderRadius.circular(12),
child: Container(
width: 80,
height: 80,
decoration: BoxDecoration(
color: Colors.grey.shade200,
borderRadius: BorderRadius.circular(12),
),
child: _buildRecipeImage(recipe.imageUrls.first),
),
),
)
: Container(
width: 80,
height: 80,
decoration: BoxDecoration(
color: Colors.grey.shade200,
borderRadius: BorderRadius.circular(12),
),
child: Icon(
Icons.restaurant_menu,
color: Colors.grey.shade400,
size: 32,
),
),
const SizedBox(width: 16),
// Recipe info
Expanded(
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
Text(
recipe.title,
style: Theme.of(context).textTheme.titleMedium?.copyWith(
fontWeight: FontWeight.w600,
),
maxLines: 2,
overflow: TextOverflow.ellipsis,
),
if (recipe.description != null && recipe.description!.isNotEmpty) ...[
const SizedBox(height: 6),
Text(
recipe.description!,
style: Theme.of(context).textTheme.bodySmall?.copyWith(
color: Colors.grey.shade600,
),
maxLines: 2,
overflow: TextOverflow.ellipsis,
),
],
const SizedBox(height: 8),
Row(
children: [
// Rating badge
Container(
padding: const EdgeInsets.symmetric(
horizontal: 8,
vertical: 4,
),
decoration: BoxDecoration(
color: _getRatingColor(recipe.rating.toDouble()).withOpacity(0.15),
borderRadius: BorderRadius.circular(12),
),
child: Row(
mainAxisSize: MainAxisSize.min,
children: [
const Icon(
Icons.star,
size: 14,
color: Colors.amber,
),
const SizedBox(width: 4),
Text(
recipe.rating.toString(),
style: Theme.of(context).textTheme.labelSmall?.copyWith(
fontWeight: FontWeight.bold,
color: _getRatingColor(recipe.rating.toDouble()),
),
),
],
),
),
if (recipe.isFavourite) ...[
const SizedBox(width: 8),
Icon(
Icons.favorite,
size: 16,
color: Colors.red.shade400,
),
],
],
),
],
), ),
)
: Container(
width: 60,
height: 60,
decoration: BoxDecoration(
color: Colors.grey[300],
borderRadius: BorderRadius.circular(8),
),
child: const Icon(Icons.restaurant_menu),
), ),
// Chevron icon title: Text(
Icon( recipe.title,
Icons.chevron_right, style: const TextStyle(fontWeight: FontWeight.w500),
color: Colors.grey.shade400, ),
subtitle: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
if (recipe.description != null && recipe.description!.isNotEmpty)
Text(
recipe.description!,
maxLines: 2,
overflow: TextOverflow.ellipsis,
), ),
], const SizedBox(height: 4),
), Row(
children: [
const Icon(Icons.star, size: 16, color: Colors.amber),
const SizedBox(width: 4),
Text(recipe.rating.toString()),
if (recipe.isFavourite) ...[
const SizedBox(width: 12),
const Icon(Icons.favorite, size: 16, color: Colors.red),
],
],
),
],
), ),
onTap: onTap,
); );
} }
} }

@ -7,7 +7,6 @@ import '../../data/recipes/recipe_service.dart';
import '../../data/recipes/models/recipe_model.dart'; import '../../data/recipes/models/recipe_model.dart';
import '../add_recipe/add_recipe_screen.dart'; import '../add_recipe/add_recipe_screen.dart';
import '../photo_gallery/photo_gallery_screen.dart'; import '../photo_gallery/photo_gallery_screen.dart';
import '../navigation/material3_page_route.dart';
/// Favourites screen displaying user's favorite recipes. /// Favourites screen displaying user's favorite recipes.
class FavouritesScreen extends StatefulWidget { class FavouritesScreen extends StatefulWidget {
@ -166,9 +165,8 @@ class _FavouritesScreenState extends State<FavouritesScreen> {
void _viewRecipe(RecipeModel recipe) async { void _viewRecipe(RecipeModel recipe) async {
await Navigator.of(context).push( await Navigator.of(context).push(
Material3PageRoute( MaterialPageRoute(
child: AddRecipeScreen(recipe: recipe, viewMode: true), builder: (context) => AddRecipeScreen(recipe: recipe, viewMode: true),
useEmphasized: true, // Use emphasized easing for more fluid feel
), ),
); );
// Reload favourites after viewing (in case favorite was toggled) // Reload favourites after viewing (in case favorite was toggled)
@ -179,9 +177,8 @@ class _FavouritesScreenState extends State<FavouritesScreen> {
void _editRecipe(RecipeModel recipe) async { void _editRecipe(RecipeModel recipe) async {
final result = await Navigator.of(context).push( final result = await Navigator.of(context).push(
Material3PageRoute( MaterialPageRoute(
child: AddRecipeScreen(recipe: recipe), builder: (context) => AddRecipeScreen(recipe: recipe),
useEmphasized: true, // Use emphasized easing for more fluid feel
), ),
); );
if (result == true || mounted) { if (result == true || mounted) {

@ -6,7 +6,6 @@ import '../../data/recipes/models/recipe_model.dart';
import '../shared/primary_app_bar.dart'; import '../shared/primary_app_bar.dart';
import '../add_recipe/add_recipe_screen.dart'; import '../add_recipe/add_recipe_screen.dart';
import '../navigation/main_navigation_scaffold.dart'; import '../navigation/main_navigation_scaffold.dart';
import '../navigation/material3_page_route.dart';
import 'package:video_player/video_player.dart'; import 'package:video_player/video_player.dart';
/// Home screen showing recipes overview, stats, tags, and favorites. /// Home screen showing recipes overview, stats, tags, and favorites.
@ -70,30 +69,29 @@ class _HomeScreenState extends State<HomeScreen> {
final recent = allRecipes.take(6).toList(); final recent = allRecipes.take(6).toList();
if (mounted) { if (mounted) {
setState(() { setState(() {
_allRecipes = allRecipes; _allRecipes = allRecipes;
_favoriteRecipes = favorites; _favoriteRecipes = favorites;
_recentRecipes = recent; _recentRecipes = recent;
_tagCounts = Map.fromEntries(sortedTags); _tagCounts = Map.fromEntries(sortedTags);
_isLoading = false; _isLoading = false;
}); });
} }
} catch (e) { } catch (e) {
Logger.error('Failed to load home data', e); Logger.error('Failed to load home data', e);
if (mounted) { if (mounted) {
setState(() { setState(() {
_isLoading = false; _isLoading = false;
}); });
} }
} }
} }
void _navigateToRecipe(RecipeModel recipe) async { void _navigateToRecipe(RecipeModel recipe) async {
final result = await Navigator.push( final result = await Navigator.push(
context, context,
Material3PageRoute( MaterialPageRoute(
child: AddRecipeScreen(recipe: recipe, viewMode: true), builder: (context) => AddRecipeScreen(recipe: recipe, viewMode: true),
useEmphasized: true, // Use emphasized easing for more fluid feel
), ),
); );
// Reload data if recipe was edited // Reload data if recipe was edited
@ -422,7 +420,7 @@ class _HomeScreenState extends State<HomeScreen> {
], ],
if (recipe.isFavourite) ...[ if (recipe.isFavourite) ...[
if (recipe.rating > 0) const SizedBox(width: 8), if (recipe.rating > 0) const SizedBox(width: 8),
Icon( Icon(
Icons.favorite, Icons.favorite,
size: 16, size: 16,
color: Colors.red, color: Colors.red,
@ -445,8 +443,8 @@ class _HomeScreenState extends State<HomeScreen> {
child: Icon( child: Icon(
Icons.restaurant_menu, Icons.restaurant_menu,
size: 48, size: 48,
color: Colors.grey, color: Colors.grey,
), ),
), ),
); );
} }
@ -585,14 +583,14 @@ class _HomeScreenState extends State<HomeScreen> {
), ),
], ],
), ),
), ),
Icon( Icon(
Icons.favorite, Icons.favorite,
color: Colors.red, color: Colors.red,
size: 20, size: 20,
), ),
], ],
), ),
), ),
), ),
); );
@ -682,7 +680,7 @@ class _VideoThumbnailPreviewState extends State<_VideoThumbnailPreview> {
), ),
), ),
], ],
); );
} }
return Stack( return Stack(

@ -1,74 +0,0 @@
import 'package:flutter/material.dart';
/// Material Design 3 easing curves
/// Reference: https://m3.material.io/styles/motion/easing-and-duration/applying-easing-and-duration
class Material3Easing {
/// Standard easing curve for entering animations
/// Cubic(0.2, 0, 0, 1)
static const Curve standardEnter = Cubic(0.2, 0, 0, 1);
/// Standard easing curve for exiting animations
/// Cubic(0, 0, 0.2, 1)
static const Curve standardExit = Cubic(0, 0, 0.2, 1);
/// Emphasized easing curve for entering animations
/// Cubic(0.2, 0, 0, 1)
static const Curve emphasizedEnter = Cubic(0.2, 0, 0, 1);
/// Emphasized easing curve for exiting animations
/// Cubic(0.05, 0.7, 0.1, 1)
static const Curve emphasizedExit = Cubic(0.05, 0.7, 0.1, 1);
}
/// Material Design 3 page route with custom easing animations.
///
/// Uses Material Design 3 motion guidelines for smooth, fluid transitions.
/// Reference: https://m3.material.io/styles/motion/easing-and-duration/applying-easing-and-duration
class Material3PageRoute<T> extends PageRouteBuilder<T> {
final Widget child;
final bool useEmphasized;
Material3PageRoute({
required this.child,
this.useEmphasized = false,
RouteSettings? settings,
}) : super(
pageBuilder: (context, animation, secondaryAnimation) => child,
transitionDuration: useEmphasized
? const Duration(milliseconds: 400)
: const Duration(milliseconds: 300),
reverseTransitionDuration: useEmphasized
? const Duration(milliseconds: 400)
: const Duration(milliseconds: 300),
settings: settings,
transitionsBuilder: (context, animation, secondaryAnimation, child) {
// Use Material Design 3 easing curves
final enterCurve = useEmphasized
? Material3Easing.emphasizedEnter
: Material3Easing.standardEnter;
final exitCurve = useEmphasized
? Material3Easing.emphasizedExit
: Material3Easing.standardExit;
// Apply easing curves to animations
final curvedAnimation = CurvedAnimation(
parent: animation,
curve: enterCurve,
reverseCurve: exitCurve,
);
// Slide transition with fade
return SlideTransition(
position: Tween<Offset>(
begin: const Offset(0.0, 0.05), // Slight upward slide
end: Offset.zero,
).animate(curvedAnimation),
child: FadeTransition(
opacity: curvedAnimation,
child: child,
),
);
},
);
}

@ -9,7 +9,6 @@ import '../../data/local/models/item.dart';
import '../add_recipe/add_recipe_screen.dart'; import '../add_recipe/add_recipe_screen.dart';
import '../photo_gallery/photo_gallery_screen.dart'; import '../photo_gallery/photo_gallery_screen.dart';
import '../navigation/main_navigation_scaffold.dart'; import '../navigation/main_navigation_scaffold.dart';
import '../navigation/material3_page_route.dart';
import 'bookmark_dialog.dart'; import 'bookmark_dialog.dart';
import '../../data/recipes/models/bookmark_category_model.dart'; import '../../data/recipes/models/bookmark_category_model.dart';
@ -29,7 +28,7 @@ class RecipesScreen extends StatefulWidget {
} }
} }
class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserver, SingleTickerProviderStateMixin { class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserver {
List<RecipeModel> _recipes = []; List<RecipeModel> _recipes = [];
List<RecipeModel> _filteredRecipes = []; List<RecipeModel> _filteredRecipes = [];
bool _isLoading = false; bool _isLoading = false;
@ -41,12 +40,6 @@ class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserv
bool _isSearching = false; bool _isSearching = false;
DateTime? _lastRefreshTime; DateTime? _lastRefreshTime;
Set<String> _selectedTags = {}; // Currently selected tags for filtering Set<String> _selectedTags = {}; // Currently selected tags for filtering
// Auto-scroll animation for tag filter bar
ScrollController? _tagScrollController;
AnimationController? _tagAnimationController;
Animation<double>? _tagAnimation;
bool _isUserScrollingTags = false;
@override @override
void initState() { void initState() {
@ -56,53 +49,15 @@ class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserv
_checkLoginState(); _checkLoginState();
_loadPreferences(); _loadPreferences();
_searchController.addListener(_onSearchChanged); _searchController.addListener(_onSearchChanged);
// Initialize tag scroll animation
_tagScrollController = ScrollController();
_tagAnimationController = AnimationController(
vsync: this,
duration: const Duration(seconds: 8),
);
_tagAnimation = Tween<double>(begin: 0, end: 0.3).animate(
CurvedAnimation(parent: _tagAnimationController!, curve: Curves.easeInOut),
);
_startTagAutoScroll();
_tagAnimation?.addListener(_animateTagScroll);
} }
@override @override
void dispose() { void dispose() {
WidgetsBinding.instance.removeObserver(this); WidgetsBinding.instance.removeObserver(this);
_searchController.dispose(); _searchController.dispose();
_tagAnimationController?.dispose();
_tagScrollController?.dispose();
super.dispose(); super.dispose();
} }
void _startTagAutoScroll() {
if (!_isUserScrollingTags && _tagAnimationController != null) {
_tagAnimationController!.repeat(reverse: true);
}
}
void _animateTagScroll() {
if (!_isUserScrollingTags &&
_tagScrollController != null &&
_tagScrollController!.hasClients &&
_tagAnimation != null) {
final allTags = _getFilteredTags().toList();
if (allTags.length > 3) {
final maxScroll = _tagScrollController!.position.maxScrollExtent;
if (maxScroll > 0) {
final targetScroll = _tagAnimation!.value * maxScroll;
if ((_tagScrollController!.offset - targetScroll).abs() > 1) {
_tagScrollController!.jumpTo(targetScroll);
}
}
}
}
}
@override @override
void didChangeAppLifecycleState(AppLifecycleState state) { void didChangeAppLifecycleState(AppLifecycleState state) {
super.didChangeAppLifecycleState(state); super.didChangeAppLifecycleState(state);
@ -214,10 +169,6 @@ class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserv
/// Handles tag selection for filtering /// Handles tag selection for filtering
void _selectTag(String tag) { void _selectTag(String tag) {
// Stop auto-scroll permanently when user selects a tag
_isUserScrollingTags = true;
_tagAnimationController?.stop();
setState(() { setState(() {
if (_selectedTags.contains(tag)) { if (_selectedTags.contains(tag)) {
_selectedTags.remove(tag); _selectedTags.remove(tag);
@ -383,9 +334,8 @@ class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserv
void _viewRecipe(RecipeModel recipe) async { void _viewRecipe(RecipeModel recipe) async {
await Navigator.of(context).push( await Navigator.of(context).push(
Material3PageRoute( MaterialPageRoute(
child: AddRecipeScreen(recipe: recipe, viewMode: true), builder: (context) => AddRecipeScreen(recipe: recipe, viewMode: true),
useEmphasized: true, // Use emphasized easing for more fluid feel
), ),
); );
// Always reload recipes after viewing/editing to ensure UI is up-to-date // Always reload recipes after viewing/editing to ensure UI is up-to-date
@ -541,54 +491,34 @@ class _RecipesScreenState extends State<RecipesScreen> with WidgetsBindingObserv
), ),
), ),
) )
: NotificationListener<ScrollNotification>( : ListView.builder(
onNotification: (notification) { scrollDirection: Axis.horizontal,
// Detect when user manually scrolls - stop auto-scroll permanently padding: const EdgeInsets.symmetric(horizontal: 16),
if (notification is ScrollStartNotification && notification.dragDetails != null) { itemCount: allTags.length,
_isUserScrollingTags = true; itemBuilder: (context, index) {
_tagAnimationController?.stop(); final tag = allTags[index];
} final isSelected = _selectedTags.contains(tag);
return false; return Padding(
}, padding: const EdgeInsets.only(right: 8),
child: ListView.builder( child: FilterChip(
controller: _tagScrollController ?? ScrollController(), label: Text(tag),
scrollDirection: Axis.horizontal, selected: isSelected,
padding: const EdgeInsets.symmetric(horizontal: 16), onSelected: (selected) {
itemCount: allTags.length, _selectTag(tag);
itemBuilder: (context, index) { },
final tag = allTags[index]; selectedColor: theme.primaryColor,
final isSelected = _selectedTags.contains(tag); checkmarkColor: Colors.white,
return Padding( backgroundColor: isDark
padding: const EdgeInsets.only(right: 8), ? Colors.grey[800]
child: FilterChip( : Colors.grey[200],
label: Text( labelStyle: TextStyle(
tag, color: isSelected
style: TextStyle( ? Colors.white
color: isSelected : (isDark ? Colors.white : Colors.black87),
? Colors.white
: (isDark ? Colors.white : Colors.black87),
fontWeight: isSelected ? FontWeight.w600 : FontWeight.normal,
),
),
selected: isSelected,
onSelected: (selected) {
_selectTag(tag);
},
selectedColor: theme.primaryColor,
checkmarkColor: Colors.white,
backgroundColor: isDark
? Colors.grey[800]
: Colors.grey[200],
side: isSelected
? BorderSide(
color: theme.primaryColor.withValues(alpha: 0.8),
width: 1.5,
)
: null,
), ),
); ),
}, );
), },
), ),
), ),
); );
@ -1173,10 +1103,7 @@ class _RecipeCard extends StatelessWidget {
Widget _buildRecipeImage(String imageUrl) { Widget _buildRecipeImage(String imageUrl) {
final mediaService = ServiceLocator.instance.mediaService; final mediaService = ServiceLocator.instance.mediaService;
if (mediaService == null) { if (mediaService == null) {
return _FadeInImageWidget( return Image.network(imageUrl, fit: BoxFit.cover);
imageUrl: imageUrl,
isNetwork: true,
);
} }
// Try to extract asset ID from Immich URL (format: .../api/assets/{id}/original) // Try to extract asset ID from Immich URL (format: .../api/assets/{id}/original)
@ -1208,16 +1135,41 @@ class _RecipeCard extends StatelessWidget {
); );
} }
return _FadeInImageWidget( return Image.memory(
imageBytes: snapshot.data!, snapshot.data!,
fit: BoxFit.cover,
width: double.infinity,
height: double.infinity,
); );
}, },
); );
} }
return _FadeInImageWidget( return Image.network(
imageUrl: imageUrl, imageUrl,
isNetwork: true, fit: BoxFit.cover,
width: double.infinity,
height: double.infinity,
errorBuilder: (context, error, stackTrace) {
return Container(
color: Colors.grey.shade200,
child: const Icon(Icons.image_not_supported, size: 32, color: Colors.grey),
);
},
loadingBuilder: (context, child, loadingProgress) {
if (loadingProgress == null) return child;
return Container(
color: Colors.grey.shade200,
child: Center(
child: CircularProgressIndicator(
value: loadingProgress.expectedTotalBytes != null
? loadingProgress.cumulativeBytesLoaded /
loadingProgress.expectedTotalBytes!
: null,
),
),
);
},
); );
} }
@ -1232,124 +1184,6 @@ class _RecipeCard extends StatelessWidget {
} }
} }
/// Widget that displays an image with a smooth fade-in animation.
class _FadeInImageWidget extends StatefulWidget {
final String? imageUrl;
final Uint8List? imageBytes;
final bool isNetwork;
const _FadeInImageWidget({
this.imageUrl,
this.imageBytes,
this.isNetwork = false,
}) : assert(imageUrl != null || imageBytes != null, 'Either imageUrl or imageBytes must be provided');
@override
State<_FadeInImageWidget> createState() => _FadeInImageWidgetState();
}
class _FadeInImageWidgetState extends State<_FadeInImageWidget>
with SingleTickerProviderStateMixin {
late AnimationController _controller;
late Animation<double> _fadeAnimation;
bool _imageLoaded = false;
@override
void initState() {
super.initState();
_controller = AnimationController(
duration: const Duration(milliseconds: 500),
vsync: this,
);
_fadeAnimation = CurvedAnimation(
parent: _controller,
curve: Curves.easeOut,
);
// Start with opacity 0
_controller.value = 0.0;
}
@override
void dispose() {
_controller.dispose();
super.dispose();
}
void _onImageLoaded() {
if (!_imageLoaded && mounted) {
_imageLoaded = true;
_controller.forward();
}
}
@override
Widget build(BuildContext context) {
Widget imageWidget;
if (widget.imageBytes != null) {
// For memory images, use frameBuilder to detect when image is ready
imageWidget = Image.memory(
widget.imageBytes!,
fit: BoxFit.cover,
width: double.infinity,
height: double.infinity,
frameBuilder: (context, child, frame, wasSynchronouslyLoaded) {
if ((wasSynchronouslyLoaded || frame != null) && !_imageLoaded) {
WidgetsBinding.instance.addPostFrameCallback((_) => _onImageLoaded());
}
return child;
},
);
} else if (widget.imageUrl != null) {
// For network images, use frameBuilder to detect when image is ready
imageWidget = Image.network(
widget.imageUrl!,
fit: BoxFit.cover,
width: double.infinity,
height: double.infinity,
errorBuilder: (context, error, stackTrace) {
return Container(
color: Colors.grey.shade200,
child: const Icon(Icons.image_not_supported, size: 32, color: Colors.grey),
);
},
loadingBuilder: (context, child, loadingProgress) {
if (loadingProgress == null) {
// Image is loaded, return child and let frameBuilder handle fade-in
return child;
}
return Container(
color: Colors.grey.shade200,
child: Center(
child: CircularProgressIndicator(
value: loadingProgress.expectedTotalBytes != null
? loadingProgress.cumulativeBytesLoaded /
loadingProgress.expectedTotalBytes!
: null,
),
),
);
},
frameBuilder: (context, child, frame, wasSynchronouslyLoaded) {
if ((wasSynchronouslyLoaded || frame != null) && !_imageLoaded) {
WidgetsBinding.instance.addPostFrameCallback((_) => _onImageLoaded());
}
return child;
},
);
} else {
return Container(
color: Colors.grey.shade200,
child: const Icon(Icons.image_not_supported, size: 32, color: Colors.grey),
);
}
return FadeTransition(
opacity: _fadeAnimation,
child: imageWidget,
);
}
}
/// Widget that displays a video thumbnail preview with a few seconds of playback. /// Widget that displays a video thumbnail preview with a few seconds of playback.
class _VideoThumbnailPreview extends StatefulWidget { class _VideoThumbnailPreview extends StatefulWidget {

@ -56,7 +56,6 @@ class RelayManagementController extends ChangeNotifier {
url: relay.url, url: relay.url,
isConnected: relay.isConnected, isConnected: relay.isConnected,
isEnabled: relay.isEnabled, isEnabled: relay.isEnabled,
retryCount: relay.retryCount,
)).toList(); )).toList();
notifyListeners(); notifyListeners();
} }

File diff suppressed because it is too large Load Diff
Loading…
Cancel
Save

Powered by TurnKey Linux.