search

projecthamster / hamster

GNOME time tracker
http://projecthamster.org
GNU General Public License v3.0
1.08k stars 249 forks source link

Selected date is deselected by typing in time input field #723

Open mwilck opened 1 year ago

mwilck commented 1 year ago

Steps to reproduce:

Screencast

  1. open "add activity" dialog
  2. select past date with mouse
  3. type backspace in "start" input field to change start time

When the "minutes" part is void, the selected date changes back to the current date (unselected).

The same happens if I "select all and cut" in the "start" field. It seems to happen whenever the time in the start field temporarily assumes an "invalid" value. It doesn't happen if I set the "start" input field to overwrite mode and just overwrite the digits of the field.

This was tested with the openSUSE hamster package, which was built from 9b557ab (@matthijskooijman's simplify-facttree-scrolling branch), which contains the fix for #674.

GeraldJansen commented 1 year ago

I confirm this behavior in current master (hamster version v3.0.2-53-gd10ae12c). It also happens when editing an activity from a previous date. This could be serious because if one is not careful one could inadvertently clobber other activities on the current date.

DarkRedman commented 1 year ago

I just had the same issue with hamster 3.0.2-5 using Gnome 44.44 and Linux 6.1.53-1-MANJARO.

Similarly it seems even if I register an event today for any duration, if the start hour is below the start hour of a day (see settings) Hamster overrides the display of stats and shows the task belong to the day before. So I did reset the start of day at 00:00 to avoid any bugs.