Bug 176925

Summary: Center wallpaper positioning of images smaller than screen is incorrect
Product: [Plasma] plasma4 Reporter: Marian Kyral <mkyral>
Component: desktopAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Screenshot when the centered positiong is selected
When scalled, keep proportions is selected
Gwenview fullscreen preview, how it should looks like

Description Marian Kyral 2008-12-04 22:33:34 UTC 
Version:           4.1.80 (using Devel)
Compiler:          gcc-4.3.2 x86_64-pc-linux-gnu-4.3.2 
OS:                Linux
Installed from:    Compiled sources

I have an image 1000x708, screen resolution is 1280x800.

When I select this image as a wallpaper and change the wallpaper position to "Centered", the image is incorrectly scaled to the whole screen.
Also Tiled and Tiled center positioning looks very bad. The image is always scaled and is bigger than the screen resolution.

Expected result: The image is in the center of the screen and there is a color border around.
Comment 1 Marian Kyral 2008-12-04 22:34:55 UTC 
Created attachment 29044 [details]
Screenshot when the centered positiong is selected
Comment 2 Marian Kyral 2008-12-04 22:36:06 UTC 
Created attachment 29045 [details]
When scalled, keep proportions is selected
Comment 3 Marian Kyral 2008-12-04 22:37:26 UTC 
Created attachment 29046 [details]
Gwenview fullscreen preview, how it should looks like
Comment 4 Marian Kyral 2008-12-04 23:22:17 UTC 
I've played with this for a while and it seems that setting to positioning to "Centered" has the same effect as "Scaled & Cropped"
Comment 5 Aaron J. Seigo 2008-12-05 21:23:30 UTC 
SVN commit 893037 by aseigo:

only scale the size of the image by the ratio when we are going to actually scale the final image
BUG:176925


 M  +4 -0      image.cpp  
 M  +24 -23    renderthread.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=893037