hhannine / superpaper

A cross-platform multi monitor wallpaper manager.
MIT License
1.12k stars 46 forks source link

Monitor positions stay at defaults when manual display sizes are set. #83

Closed D0G3L0V3R closed 2 years ago

D0G3L0V3R commented 3 years ago
  1. "Override detected sizes"
  2. "Input display sizes manually"
  3. Set desired sizes.
  4. On "Save" superpaper will move monitors on the preview screen to some positions. (Always the same for set system display configuration)
  5. "Positions"
  6. Allign monitors in the desired way and "Save"
  7. On "Apply" superpaper doesn't acknowledge changed positions and applies the ones from step 4.

Expected behavior: Superpaper applies wallpaper according to user settings.

No errors spotted when using "superpaper -d"

tomcassi commented 3 years ago

Same bug appears on Windows 10.

hhannine commented 3 years ago

Thanks for the report, and @Tom-ST thanks for helping narrowing it down that it isn't Linux specific. I think something is messing up the saving of the positions if manual sizes are set, but I couldn't find the issue yet.

beningodfrey4 commented 2 years ago

Any updates on this one? For now working around this by:

  1. "Override detected sizes"
  2. Put in the correct sizes and hit "save"
  3. The preview resets and changing positions now has no effect (the bug)
  4. "Apply manual offsets" and specify x,y offsets for each monitor as how you would if you were dragging the preview (the workaround)

Thankfully bezels still apply correctly after the manual offsetting, but the preview doesn't update. Understandable as I suppose it is intended for small post-hoc adjustments.

@hhannine, regarding the messed up saving, manually adding the correct sizes to $XDG_CONFIG_HOME/superpaper/display_systems.dat (instead of None) seems to have no effect.

hhannine commented 2 years ago

This has now been fixed (70c3fd78a3bfb58cc0a2615ade52785e9ef31f00) and will be working in the next release.