diff options
| author | Jan-Marek Glogowski <glogow@fbihome.de> | 2022-06-05 08:55:43 +0200 |
|---|---|---|
| committer | Jan-Marek Glogowski <glogow@fbihome.de> | 2022-06-05 12:17:45 +0200 |
| commit | 3f8d3fd4649ef09e86c735617383a4bda0425540 (patch) | |
| tree | 2b704867df7b83e246b9b65bd0982af449b3e90a /vcl/qt5 | |
| parent | e0e944a42059271d082f8bd5121f30a4536fb8f3 (diff) | |
tdf#137471 Qt return frame pos + client area size
My code comment about "drawable area" and Michaels bug comment
12 about the "frameGeometry()" usage were both half right. LO
expects the window's frame position and the drawable client area
size almost everywhere when "geometry" is involved. The frame's
border is stored in the decorations members of SalFrameGeometry.
Change-Id: Ic00ad1f1d74d7afadcaca0c01e1a41ea7f2833de
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135434
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Diffstat (limited to 'vcl/qt5')
| -rw-r--r-- | vcl/qt5/QtFrame.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vcl/qt5/QtFrame.cxx b/vcl/qt5/QtFrame.cxx index 4b94273e1cff..3c63060855ab 100644 --- a/vcl/qt5/QtFrame.cxx +++ b/vcl/qt5/QtFrame.cxx @@ -741,8 +741,8 @@ bool QtFrame::GetWindowState(SalFrameState* pState) } else { - // geometry() is the drawable area, which is wanted here - QRect rect = scaledQRect(asChild()->geometry(), devicePixelRatioF()); + // we want the frame position and the client area size + QRect rect = scaledQRect({ asChild()->pos(), asChild()->size() }, devicePixelRatioF()); pState->mnX = rect.x(); pState->mnY = rect.y(); pState->mnWidth = rect.width(); |