From fa0e24b641ad241195cab24d008fed6e68983944 Mon Sep 17 00:00:00 2001 From: Michael Weghorn Date: Mon, 1 Aug 2022 14:18:05 +0200 Subject: a11y atspi: translate coords for GetOffsetAtPoint the right way MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The coordinate type for "GetOffsetAtPoint" describes the coordinate type of the input coordinates and `QAccessibleTextInterface::offsetAtPoint` expects the coordinates in screen coordinates. Therefore, if window-relative coordinates are given, the screen coordinates of the window need to be added (not subtracted) to the window-relative coordinates of the point in order to calculate the screen coordinates of the point. Fixes: QTBUG-105281 Pick-to: 6.4 6.3 6.2 Change-Id: I5517335c0c0d6a47b8c45c7e4f8f8b5a91f0fcd7 Reviewed-by: Tor Arne Vestbø --- src/gui/accessible/linux/atspiadaptor.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/gui/accessible') diff --git a/src/gui/accessible/linux/atspiadaptor.cpp b/src/gui/accessible/linux/atspiadaptor.cpp index 6f6583f814..0c27422364 100644 --- a/src/gui/accessible/linux/atspiadaptor.cpp +++ b/src/gui/accessible/linux/atspiadaptor.cpp @@ -1804,7 +1804,7 @@ bool AtSpiAdaptor::textInterface(QAccessibleInterface *interface, const QString uint coordType = message.arguments().at(2).toUInt(); if (coordType == ATSPI_COORD_TYPE_WINDOW) { QWindow *win = interface->window(); - point -= QPoint(win->x(), win->y()); + point += QPoint(win->x(), win->y()); } int offset = interface->textInterface()->offsetAtPoint(point); sendReply(connection, message, offset); -- cgit v1.2.1