Francesco149 / weebp

free and open-source wallpaper engine, set any window as your wallpaper
The Unlicense
547 stars 60 forks source link

Fullscreen with multi-screen works not properly #2

Closed chs040701 closed 5 years ago

chs040701 commented 5 years ago

I have 2 screens, one is 19201080, another is 38402160. the first screen coordinates is [-1920,0],[0,1080], and the second is [0,3840],[0,2160] Now the full screen position is [-1920,1920],[0,2160]

Francesco149 commented 5 years ago

--fullscreen is supposed to fill one of the monitors, the one the window is currently in. if you want a background that spans both monitors you will have to set coordinates manually with the move command for now. maybe i'll consider adding another kind of fullscreen, like maybe --multimonitor or something that spans all monitors

chs040701 commented 5 years ago

Thanks. But it seems --fullscreen does not behavior as expected. Wherever I put the window, fullscreen will fill 1080p screen with 2160p size.

Francesco149 commented 5 years ago

hm, interesting, I remember testing on my dual monitor (1280x1024 and 1920x1080) and it was working as expected, but I guess i'll re-test --fullscreen as well

Francesco149 commented 5 years ago

ok i see the problem with --fullscreen, the coordinates used in wp mv seem to be always positive even on multimonitor setup, which doesn't match the coordinates returned when i query the monitors (for example my 2nd monitor is monitor: -1280 0 0 1024). i'm currently testing if MapWindowPoints from the desktop window to the root window is what i want and it seems like it behaves as it should

but there's another issue, for some reason it always wants to put it on my 2nd monitor, so maybe I'll use the cursor position to figure out which monitor instead of using MonitorFromWindow which doesn't seem to work

Francesco149 commented 5 years ago

ok i fixed it and also added a --panoramic option to make a window fill all monitors