Skip to content

Add image-sized window restoration feature - #220

Open
SKProCH wants to merge 5 commits into
riyasy:mainfrom
SKProCH:sizeWindowToImageOnRestore
Open

Add image-sized window restoration feature#220
SKProCH wants to merge 5 commits into
riyasy:mainfrom
SKProCH:sizeWindowToImageOnRestore

Conversation

@SKProCH

@SKProCH SKProCH commented Jul 17, 2026

Copy link
Copy Markdown

Summary

Make behavior identical with Picasa - when the user clicks outside the image:

  • Resize the window to photo's size
  • Move the window to the same position as photo was (in absolute screen coordinates), so photo visually stay in place

Motivation described in #218

Type of change

  • Bug fix
  • New feature
  • Localization (new language or translation fix)
  • Documentation
  • Other

Related issue

#128

How was this tested?

  • Built and ran locally
  • Tested with real image files

Localization (if applicable)

  • name attributes are unchanged and match en-US
  • Files saved as UTF-8

Checklist

  • I have reviewed and understand every line of this change.
  • This change is focused and avoids unrelated or low-value edits.

Animation.mp4

AI was used to write Win32 related stuff.

@SKProCH

SKProCH commented Jul 17, 2026

Copy link
Copy Markdown
Author

Probably there should be also 2 more features:

  • Double click on the image will open fullscreen back
  • Allow to size window in the way, when the controls and previews of other files will be below the image

But keeping this PR focused

@2590812378

Copy link
Copy Markdown

这是非常好的功能,期待已久

@riyasy

riyasy commented Aug 14, 2026

Copy link
Copy Markdown
Owner

@SKProCH , the PR cannot be merged to latest code as it has conflicts.
If possible, can you please correct this?
Also can you keep the new code as isolated as possible and enable this behaviour based on a setting in AppSettings.cs (like ResizeToImageBounds). (if you add in settings.xaml, add it under Advanced section for the time being)
So that I can first add it as Beta feature and after review and testing promote as a feature?

@SKProCH

SKProCH commented Aug 14, 2026

Copy link
Copy Markdown
Author

Ok, will do it a bit later today (hopefully)

@riyasy

riyasy commented Aug 14, 2026

Copy link
Copy Markdown
Owner

No hurry... thanks.
I am working only intermittently as I am on a break for few weeks.

…store

# Conflicts:
#	Src/FlyPhotos/Strings/en-US/Resources.resw
#	Src/FlyPhotos/UI/Views/PhotoDisplayWindow.xaml.cs
@SKProCH

SKProCH commented Aug 14, 2026

Copy link
Copy Markdown
Author

Resolved merge conflicts, move the settings into advanced. Anything else?

@riyasy

riyasy commented Aug 14, 2026

Copy link
Copy Markdown
Owner
  1. After resizing, there seems to be a small gap at the bottom of the window.
image
  1. can we have the resizing animation disabled in this mode?

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.

3 participants