Turpial icon indicating copy to clipboard operation
Turpial copied to clipboard

Inline preview does not resize pics, just crops them

Open Majki opened this issue 10 years ago • 7 comments

Just compiled libturpial 1.7 and turpial from git. Inline previews are now broken partially.

Example:

http://i.imgur.com/tIAZCLY.png

Majki avatar May 08 '14 09:05 Majki

Hi, that's not an issue, it's the expected behavior. Turpial will crop the image to show you just a little preview (like the Twitter website). Then you can click the image to see it in full size

satanas avatar May 08 '14 14:05 satanas

Oh my - previous feature was so much better imo and much more user friendly.

Majki avatar May 08 '14 14:05 Majki

Yes I agree, but that feature was breaking out the whole layout of the app, so I had to remove it and just show a cropped version of the image :(

satanas avatar May 08 '14 14:05 satanas

@satanas While I agree that the images should be cropped, the cropped size images are too small (in height). So basically I have to open each image just to understand what exactly it is about. Little more visible height doesn't harm.

Twitter (Web) shows full size image inline if the image size is relatively small. May be we can have similar mechanism here.

khurshid-alam avatar Oct 17 '14 11:10 khurshid-alam

Ok, let's consider giving more size to the crop to improve the user experience

satanas avatar Oct 18 '14 00:10 satanas

+1 khurshid-alam

Having to click each time for pic is annoying.

Majki avatar Oct 20 '14 13:10 Majki

I think it's reasonable to preview a smaller version of the image so we don't break the layout, however, I think the problem is the crop as opposed to resize. I suspect it will take more CPU to resize an image vs just cropping it, but I think the best solution would be to evenly resize it so you can get the whole image, just a smaller version of it.

jmcclelland avatar Mar 24 '15 19:03 jmcclelland