Opened 9 years ago

Closed 9 years ago

#6226 closed enhancement (invalid)

OverlayImage app doesn't remember current position

Reported by: jscipione Owned by: humdinger
Priority: normal Milestone: R1
Component: Applications Version: R1/Development
Keywords: OverlayImage Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

The OverlayImage app doesn't remember the current window position when you quit the application and reopen it. The following patch corrects that by saving the current position to a settings file and then recalling that position from the same file. If the window is closed off screen or their is not settings file then the window opens in the middle of the screen. Design shamelessly stolen from deskcalc.

Attachments (2)

OverlayImageSavePosition.diff (7.8 KB ) - added by jscipione 9 years ago.
Patch to save the current postion on quit
OverlayImageSavePosition2.diff (7.7 KB ) - added by jscipione 9 years ago.
Patch to save the current postion on quit. (Good)

Download all attachments as: .zip

Change History (9)

by jscipione, 9 years ago

Patch to save the current postion on quit

comment:1 by mmadia, 9 years ago

Has a Patch: set

comment:2 by stippi, 9 years ago

Owner: changed from nobody to humdinger
Status: newassigned

comment:3 by humdinger, 9 years ago

It's a bit awkward because you already have a finished patch... But do you really consider remembering the window position is a needed feature for this app? I don't see this demo app being used frequently enough for anyone even noticing that the window position is kept. :)

by jscipione, 9 years ago

Patch to save the current postion on quit. (Good)

comment:4 by humdinger, 9 years ago

The reason I wouldn't like to save settings for this little tool is that it's another file "polluting" ~/config/settings without much benefit. I've been working on using a BTextView instead of the tool tip for instructions. I'll commit that soon and would of course work in your patch should we decide to apply it.
So... what should we do with the patch?

comment:5 by humdinger, 9 years ago

Hey John. Have you thought about my argument for not remembering the window position? I'd like to close this ticket, either by applying your patch or (preferably :) ) without it. Thanks!

comment:6 by jscipione, 9 years ago

It just peeved me that the window did not save its position (spelled correctly this time) when I closed the windows and reopened the app. I understand your argument against littering the ~/config/settings directory with yet another settings file. The best solution is to close this bug and open another one that is more general. All windows should remember their positions automatically, it shouldn't be something the application programmer has to take care of, rather it is something that the OS should just provide for all applications.

comment:7 by humdinger, 9 years ago

Resolution: invalid
Status: assignedclosed

I agree. I'll open a new ticket for that. If you have an idea how that system should work (we don't want a registry, right?), please comment there. It's #6348.

Note: See TracTickets for help on using tickets.