default window size way to large on multi-monitor setup (on Win at least). Shouldn't be bigger than primary screen

• Dec 22, 2018 - 22:03
Reported version
3.0
Type
Functional
Frequency
Once
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
No
Workaround
No
Project

Currently when I start MuseScore 3.0rc factory reset on a windows multi-monitor setup (side by side, with primarily, larger monitor in middle):

Screenshot (27).png

Notice how the MuseScore window spans acorss the monitor boundary? It seems the size is set to have the width span the width of all 3 windows, cause when I drag the window all the way to the left, I get:

Screenshot (28).png

I see MuseScore::readSettings() uses
const QSize screenSize = screen->availableVirtualSize();
...and I see musescore is using this "available virtual size" minus some margin for making the default window size... I don't like that...I'd prefer using the available primary screeen size.

It seems https://doc.qt.io/qt-5/qscreen.html#availableVirtualSize-prop is "This property holds the available size of the virtual desktop to which this screen belongs" which is too much i think. I think better to use availableSize which is "The available size is the size excluding window manager reserved areas such as task bars and system menus." and I guess specific to the current monitor.


Comments

Status PR created fixed

Fixed in branch master, commit 320673460f

fix #280497 fit default window size inside primary screen

MuseScore factory reset on a multi-monitor setup produced default window size that was way too big, spanning the height and width of the entire available 'virtual' size, crossing over monitor boundaries. That is not standard default behavior for most any other program I'm aware of.

This PR instead uses only screen->availableSize(), which according to Qt doc is simply referring to primary monitor size and seems much more sane default, rather than screen->availableVirtualSize(), which according to Qt doc refers to the entire area of all monitors.

Fixed in branch master, commit 0728871538

Merge pull request #4475 from ericfont/280497-default-window-size-fit-inside-primary-screen-on-multi-monitors

fix #280497 fit default window size inside primary screen

Fix version
3.0.2