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

Crop dialog only fits image to window width #5

Open
LRN opened this issue Jan 9, 2016 · 1 comment
Open

Crop dialog only fits image to window width #5

LRN opened this issue Jan 9, 2016 · 1 comment

Comments

@LRN
Copy link

LRN commented Jan 9, 2016

If an image being cropped has portrait orientation (lot of height, not much width; this is the default for the scanner i have), the crop dialog will still display the image by fitting its width to the window. Since most PC screens have landscape orientation, the crop window (when maximized) tends to be very wide, but not very tall. As a result, the image looks as if its top and bottom parts are clipped.
The crop window provides no way to zoom out or drag the view, no scrollbars and no indication of what's happening.
At first i thought it's a bug. Then i rotated the image before cropping it, and that fixed the problem (the image got landscape orientation, so fitting it by width fits it correctly). After that i've realized that instead of maximizing the crop window, i can resize it to be narrower, and that fixed the problem too. Still, it's not immediately obvious, and thus very confusing.

Unrelated: swingsane doesn't support the sane options for my scanner, but that's probably the expected behaviour for unsupported models. Where do i send the option xml files? The scanner.xml page of the wiki is empty.

@rquast
Copy link
Owner

rquast commented Feb 22, 2016

Yes, I'd like to rework that entire area of the program actually. I originally tried to find a library that would do all that but didn't find anything that great. Feel free to try this bit out with a third party library or enhance the existing code.

@rquast rquast self-assigned this Feb 22, 2016
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