groove/Vaani
1
0
Fork 0
mirror of https://github.com/Dr-Blank/Vaani.git synced 2026-01-03 16:59:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: run dart fix
Some checks are pending
Flutter CI & Release / Test (push) Waiting to run
Details
Flutter CI & Release / Build Android APKs (push) Blocked by required conditions
Details
Flutter CI & Release / build_linux (push) Blocked by required conditions
Details
Flutter CI & Release / Create GitHub Release (push) Blocked by required conditions
Details

Browse source
This commit is contained in:
Dr.Blank 2025-04-23 00:29:02 +05:30
parent 2cb00c451e
commit ad0cd6e2ad
No known key found for this signature in database
GPG key ID: BA5F87FF0560C57B
5 changed files with 49 additions and 41 deletions
Download patch file Download diff file Expand all files Collapse all files

43
lib/features/item_viewer/view/library_item_page.dart
View file

@ -33,29 +33,32 @@ class LibraryItemPage extends HookConsumerWidget {
final showFab = useState(false);
// Effect to listen to scroll changes and update FAB visibility
useEffect(() {
void listener() {
if (!scrollController.hasClients) {
return; // Ensure controller is attached
useEffect(
() {
void listener() {
if (!scrollController.hasClients) {
return; // Ensure controller is attached
}
final shouldShow = scrollController.offset > _showFabThreshold;
// Update state only if it changes and widget is still mounted
if (showFab.value != shouldShow && context.mounted) {
showFab.value = shouldShow;
}
}
final shouldShow = scrollController.offset > _showFabThreshold;
// Update state only if it changes and widget is still mounted
if (showFab.value != shouldShow && context.mounted) {
showFab.value = shouldShow;
}
}
scrollController.addListener(listener);
// Initial check in case the view starts scrolled (less likely but safe)
WidgetsBinding.instance.addPostFrameCallback((_) {
if (scrollController.hasClients && context.mounted) {
listener();
}
});
scrollController.addListener(listener);
// Initial check in case the view starts scrolled (less likely but safe)
WidgetsBinding.instance.addPostFrameCallback((_) {
if (scrollController.hasClients && context.mounted) {
listener();
}
});
// Cleanup: remove the listener when the widget is disposed
return () => scrollController.removeListener(listener);
}, [scrollController],); // Re-run effect if scrollController changes
// Cleanup: remove the listener when the widget is disposed
return () => scrollController.removeListener(listener);
},
[scrollController],
); // Re-run effect if scrollController changes
// --- FAB Scroll-to-Top Logic ---
void scrollToTop() {

40
lib/features/item_viewer/view/library_item_sliver_app_bar.dart
View file

@ -21,25 +21,28 @@ class LibraryItemSliverAppBar extends HookConsumerWidget {
final showTitle = useState(false);
useEffect(() {
void listener() {
final shouldShow = scrollController.hasClients &&
scrollController.offset > _showTitleThreshold;
if (showTitle.value != shouldShow) {
showTitle.value = shouldShow;
useEffect(
() {
void listener() {
final shouldShow = scrollController.hasClients &&
scrollController.offset > _showTitleThreshold;
if (showTitle.value != shouldShow) {
showTitle.value = shouldShow;
}
}
}
scrollController.addListener(listener);
// Trigger listener once initially in case the view starts scrolled
// (though unlikely for this specific use case, it's good practice)
WidgetsBinding.instance.addPostFrameCallback((_) {
if (scrollController.hasClients) {
listener();
}
});
return () => scrollController.removeListener(listener);
}, [scrollController],);
scrollController.addListener(listener);
// Trigger listener once initially in case the view starts scrolled
// (though unlikely for this specific use case, it's good practice)
WidgetsBinding.instance.addPostFrameCallback((_) {
if (scrollController.hasClients) {
listener();
}
});
return () => scrollController.removeListener(listener);
},
[scrollController],
);
return SliverAppBar(
elevation: 0,
@ -62,7 +65,8 @@ class LibraryItemSliverAppBar extends HookConsumerWidget {
key: const ValueKey('title-text'),
item?.media.metadata.title ?? '',
overflow: TextOverflow.ellipsis,
style: Theme.of(context).textTheme.bodyMedium,)
style: Theme.of(context).textTheme.bodyMedium,
)
: const SizedBox(
// Also give it a key for differentiation
key: ValueKey('empty-title'),