search

doubleSlashde / KeepTime

Application to track your time spent on different projects each day.
GNU General Public License v3.0
8 stars 3 forks source link

Feature/#38 remember last position on screen #49

Closed JSchraff closed 4 years ago

JSchraff commented 4 years ago

saves position on the screen

Death111 commented 4 years ago

I need to test with multiple screens, also disconnecting them at runtime and see if it works correctly.

mplieske commented 4 years ago

@JSchraff @Death111 I tested this feature in various ways and it seems to be very stable :).

List of tests I made:

mplieske commented 4 years ago

I have found a bug and it could be from this feature. I have setup my KeepTime with this version. Setting for save window position is enabled. After a reboot from a windows update, my KeepTime starts itself from "shell:startup" (the autostart folder, not the taskmanagers autostart) and aparently is not on any screen, but on the taskbar.

Death111 commented 4 years ago

if minimized position will be set to -32000 for x and y 22:53:06.240 [JavaFX Application Thread] INFO de.doubleslash.keeptime.Main - Pos has changed -32000.0 22:53:06.269 [JavaFX Application Thread] INFO de.doubleslash.keeptime.Main - Pos has changed -32000.0 opening the application again will put it there making it not seeable

@mplieske guess this was your issue

JSchraff commented 4 years ago

@JSchraff @Death111 I tested this feature in various ways and it seems to be very stable :).

List of tests I made:

  • drag and restart application
  • drag to display B, remove display B
  • drag to display A, close application, remove display B, start application
  • drag to display B, close application, remove display B, start application
  • drag on display A, close application, add display B, start application
  • drag on display A, shutdown system, boot system, start application
  • drag on display A, close application, shutdown system, add display B, boot system, start application
  • drag on display A, shutdown system, add dispay B, boot system, start application
  • drag to display B, shutdown system, remove display B, boot system, start application
  • set display As resolution to 800x600, drag to (bottom right corner) display B (1080p), shutdown system, remove display B, boot system, start application

test 6,8,9 does not work anymore because of 6313a59 schould be mostly solved with autosave #53

JSchraff commented 4 years ago

tested on ubuntu working with one and two monitors did not test edgecases.