Bug 352555

Summary:	Display orientiation is not taken into account when looking for matching wallpapers in a package
Product:	[Plasma] plasmashell	Reporter:	Elias Probst <mail>
Component:	Image & Slideshow wallpaper plugins	Assignee:	Marco Martin <notmart>
Status:	RESOLVED FIXED
Severity:	normal	CC:	coderjoe05, kde, plasma-bugs-null, sebas
Priority:	NOR
Version First Reported In:	master
Target Milestone:	1.0
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Elias Probst 2015-09-11 13:15:04 UTC

When having a wallpaper package installed, which provides various wallpaper resolutions, Plasma might pick a wrongly oriented wallpaper if it has the same resolution.

I can reproduce this on my 1920x1080 resolution laptop this way:
1. Unpack http://vizzzion.org/stuff/stedelijk.zip to ~/.local/share/wallpapers/
2. Open the wallpaper selection dialog
3. Select the "Stedelijk" wallpaper
4. Apply

Result:
- the wallpaper is blurry because the 1080x1920 instead of 1920x1080 version is selected
- in ~/.config/plasma-org.kde.plasma.desktop-appletsrc, the wrong wallpaper is set:
Image=file:///home/elias/.local/share/wallpapers/stedelijk/contents/images/1080x1920.png

It looks like Plasma is ignoring the display orientation and just grabs the 1st matching wallpaper with the current resolution.

Running latest KF5/Plasma 5 from git + Qt 5.5.0.

Comment 1 Kai Uwe Broulik 2015-09-13 11:06:46 UTC

The distance(size, desired) method that determins which is the best image to use does not take into account orientation.

Comment 2 Kai Uwe Broulik 2016-09-22 11:58:59 UTC

Can you try whether https://phabricator.kde.org/D2834 fixes this?

Comment 3 Joe Cardenas 2022-01-24 01:56:22 UTC

I checked this, and I think this is fixed. I had to create a 1200x1920 resolution image, but the correct aspect ratio was chosen automatically. Feel free to reopen this bug if this is still an issue for you.