Skip to content

Make sleep timer use a GLib timeout to avoid issues with sleeping after the end of a chapter#1011

Open
stevenleadbeater wants to merge 9 commits intogeigi:masterfrom
stevenleadbeater:master
Open

Make sleep timer use a GLib timeout to avoid issues with sleeping after the end of a chapter#1011
stevenleadbeater wants to merge 9 commits intogeigi:masterfrom
stevenleadbeater:master

Conversation

@stevenleadbeater
Copy link

@stevenleadbeater stevenleadbeater commented Jul 29, 2025

  • Added tar ball for convenience testing on phones
  • Fixed build errors with peewee
  • Use GLib timer to count past chapter ends fixes Sleep Timer Does Not Work #646
  • make methods that don't use locals static to avoid warnings

steven and others added 8 commits September 7, 2025 11:58
… always show, and when the user confirms deletion, it will check if files exist before trying to delete them. The book will be removed from the library regardless.

  2. Popup display on phones: The dialog now always shows when attempting to delete a book, instead of silently removing it when files don't exist. This ensures consistent behavior across all devices including phones.

  The changes ensure that:
  - The delete confirmation dialog always appears when trying to delete a book
  - Books are properly removed from the library even if their files are already gone
  - File deletion is only attempted if the files actually exist
# Conflicts:
#	.github/workflows/flatpak.yml
#	com.github.geigi.cozy.json
@rdbende
Copy link
Collaborator

rdbende commented Oct 19, 2025

Thanks! I reverted some of your changes, because I had fixed them on master already. The rest looks good to me, but I just realized, that with this solution, the countdown will continue even when the playback is paused. Also, I can't reproduce #646 anymore, maybe I did fix it somehow in #973?

@stevenleadbeater
Copy link
Author

Thanks! I reverted some of your changes, because I had fixed them on master already. The rest looks good to me, but I just realized, that with this solution, the countdown will continue even when the playback is paused. Also, I can't reproduce #646 anymore, maybe I did fix it somehow in #973?

Actually having the timer continue when the book is paused is kind of normal for an audiobook player, if I can't sleep and go down stairs to make a drink or something then come back and unpause, other apps usually carry the sleep timer on... can't say for definite which but it kind of feels natural to me? this is kind of airy fairy stuff though, I can see why cancelling the timer might be preferred too?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sleep Timer Does Not Work

2 participants