search

projectM-visualizer / frontend-sdl2

Standalone application based on SDL2 that turns your desktop audio into awesome visuals. Available on most platforms supported by both libprojectM and libSDL2.
GNU General Public License v3.0
58 stars 32 forks source link

Window always opens at monitor's max res. (latest master, Win x64 build) #3

Closed chaosophy closed 2 years ago

chaosophy commented 3 years ago

Hi,

I downloaded the latest master branch and did a x64 release build for Windows using MS visual studio and projectM seems to ignore the 'Window Height' and 'Window Width settings from the 'config.inp' settings files.

The window always opens at the monitors max resolution.

If you resize the window and then ctrl+q to quit it updates the config.inp file with the new values for the window size but on re-running projectM the same thing happens and it ignores them and opens at max res.

I downloaded the 'projectm-3.1.12' release and built that the same way and it does not have this issue.

Looks like there has been a number of commits to master since the last release and my knowledge doesn't extend beyond downloading and running a build, so no idea when this happened or what might be causing it.

thanks.

kblaschke commented 3 years ago

Currently, the SDL frontend ignores the width and height settings from the configuration file. Instead, it always uses the current display size, just as you report. This has recently been changed in commit f9cfdfe, probably not intentionally.

chaosophy commented 3 years ago

Hi, thanks for taking a look and confirming the issue.

kblaschke commented 2 years ago

The rewritten app uses the configured default size (currently 1024x768) in windowed mode when it opens. Future versions might get an option to remember the last window size, position and fullscreen setting.

Current window size is also honored when switching to exclusive fullscreen mode, which will try to set a best-match fullscreen resolution for the current window dimensions.

Closing this issue as the problem has been addressed in the rewrite. If you have suggestions on more configuration options or find there's still an issue with window management in the new app, feel free to open a new issue.