slgobinath / SafeEyes

Protect your eyes from eye strain using this simple and beautiful, yet extensible break reminder
http://slgobinath.github.io/SafeEyes/
GNU General Public License v3.0
1.45k stars 163 forks source link

"Disable Safe Eyes for" not working correctly #648

Closed aceArt-GmbH closed 6 days ago

aceArt-GmbH commented 1 week ago

Describe the bug

When using the system tray to disable safe eyes for i.e. 30min the shown next break times are wrong. Also the time the next break happens is wrong too (probably same cause)

To Reproduce

  1. Check system clock. i.e. 12:57
  2. Click tray icon
  3. Click on "Disable Safe Eyes for"
  4. Click i.e. "30 minutes"
  5. Click tray icon again and see "Disabled until 15:57

Expected behavior

It should be disabled for the selected duration

Desktop (please complete the following information):

» timedatectl status
               Local time: Di 2024-09-17 12:58:53 CEST
           Universal time: Di 2024-09-17 10:58:53 UTC
                 RTC time: Di 2024-09-17 10:58:53
                Time zone: Europe/Berlin (CEST, +0200)
System clock synchronized: yes
              NTP service: active
          RTC in local TZ: no

Debug Log

safeeyes.log

Screenshots

image


Looks like on_disable_clicked() always gets time_to_wait=180

archisman-panigrahi commented 6 days ago

Thank you for reporting this bug and fixing it!