Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DESKTOP] Image scaling issues #90

Closed
abishai opened this issue Nov 17, 2015 · 4 comments
Closed

[DESKTOP] Image scaling issues #90

abishai opened this issue Nov 17, 2015 · 4 comments

Comments

@abishai
Copy link

abishai commented Nov 17, 2015

Images on virtually all tabs have rather nasty effect -they are scaled to width, so height can exceed resolution of device or monitor, forcing a user to scroll the image vertically. It can be observed in full screen mode.
Can be fixed with a check to scale to height or to width according program window size.

@pliablepixels
Copy link
Member

@abishai I am not seeing this problem at all. I use a widescreen monitor as well. By default, it stretches the image to fill the screen but crops it - does not change aspect ratio. When you tap on fit image it ensures the image is fit to screen again without changing aspect ratio. This is consistent in both full screen and normal screen mode. I tried this with 1280x960 and 1280x720 sized screens

@abishai
Copy link
Author

abishai commented Nov 17, 2015

But it shouldn't crop image, no ? It should fit the screen by whatever axis fits it to ensure all image can be observed.

  1. Image fits http://i.imgur.com/SOgDIfM.png
  2. Full screen - image is cropped http://i.imgur.com/inOZdBj.png Expected behaviour - image is fitted by height, kept aspect ratio, some borders on left and right side.
    BTW, is it nesessary to upscale images? My cameras are 1.3 mp, I just using 5% for security reasons, but imagine cheap camera on 4k monitor - the result would be blurred as this one. Maybe add that nice horisontal scroll from monitor window and let user to choose?

BTW, buttons seems to be dead - only exit works.

@pliablepixels
Copy link
Member

Ok I see the problem. I'll look at fixing it

@pliablepixels
Copy link
Member

closing so I can make a new release - please re-open if this is still an issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants