summaryrefslogtreecommitdiffstats
path: root/src/sql/kernel/qsqlrecord.cpp
diff options
context:
space:
mode:
authorYuhang Zhao <2546789017@qq.com>2022-04-07 10:32:52 +0800
committerYuhang Zhao <2546789017@qq.com>2022-05-16 22:37:38 +0800
commit75f22702933bad4f0da2b63a94ea183021771e4c (patch)
tree92927160a4f500ba3ac069daafaaebaa2d478a39 /src/sql/kernel/qsqlrecord.cpp
parent05fc3aef53348fb58be6308076e000825b704e58 (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