Bug 352555 - Display orientiation is not taken into account when looking for matching wallpapers in a package
Summary: Display orientiation is not taken into account when looking for matching wall...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Image Wallpaper (show other bugs)
Version: master
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: 1.0
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-09-11 13:15 UTC by Elias Probst
Modified: 2022-01-24 01:56 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.