diff options
author | Yuhang Zhao <2546789017@qq.com> | 2022-04-07 10:32:52 +0800 |
---|---|---|
committer | Yuhang Zhao <2546789017@qq.com> | 2022-05-16 22:37:38 +0800 |
commit | 75f22702933bad4f0da2b63a94ea183021771e4c (patch) | |
tree | 92927160a4f500ba3ac069daafaaebaa2d478a39 /src/sql/kernel/qsqlrecord.cpp | |
parent | 05fc3aef53348fb58be6308076e000825b704e58 (diff) |
Windows: Make QScreen::name() more user-friendly, take 2
The first attempt introduced in e9fd1c6aab28f had an unwanted
side-effect which makes the screen name non-unique and made the
screen manager think multiple different monitors with the
same name is one same monitor. This behavior is of course wrong
and unwanted but it was missed due to there's no unit test
for this property and I was not familiar enough with QPA
stuff. As a result the change was reverted in 851dc581cf9a76.
However, the original change is still a good improvement,
so we bring it back, but fixed the side-effect mentioned above,
by letting the screen manager compare the device name instead
of monitor name.
If multiple monitors have the same name, a numeric suffix
will be appended to the monitor name to let people still be
able to distinguish between them.
Change-Id: Ic03105d0825abbb3d21cb51d1da3ab2cbb4ca913
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
Reviewed-by: Morten Johan Sørvig <morten.sorvig@qt.io>
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Diffstat (limited to 'src/sql/kernel/qsqlrecord.cpp')
0 files changed, 0 insertions, 0 deletions