search

vajrasky / wallpapoz

Wallpapoz -- Gnome and XFCE Desktop Wallpapers Configuration Tool
http://vajrasky.wordpress.com/wallpapoz
GNU General Public License v2.0
22 stars 15 forks source link

Method "span" missing #6

Open pjkersten opened 12 years ago

pjkersten commented 12 years ago

When using multiple monitor setups, the "span" method is used to fill all monitors with a single background image. Wallpapoz is missing this method, which makes it unsuitable as a replacement for manual labor (change background) in a multi monitor setting.

vajrasky commented 12 years ago

I am sorry for the inconvenience.

I developed Wallpapoz without considering multi monitor setup. I do not have multi monitor.

pjkersten commented 12 years ago

Never mind. You could add the placement method just the same: scale/zoom/fill are valid settings for a single monitor setup. Adding one non testable method (scale) on top of that should be no sweat. Other people can give feedback on that. It's called "open source" for a reason :-)

eToThePiIPower commented 10 years ago

I know this issue is a bit old, but I have a quick fix for this on my fork. I'll send a pull request once I can spin up some VMs and test it on other window managers besides cinnamon.

pjkersten commented 10 years ago

This fixes this issue for the Gnome(3) backend.

When using other backends (I tried Xfce4) it crashes daemon_wallpapoz. (KeyError on available_style)

Xfce can be fixed by introducing a dummy style (I used {'5': '6'}), which must then set /backdrop/screen0/xinerama-stretch to true, or false for the other styles. I didn't look at the others.