From dad2797aeefaa813fd8808a6ef63bf6a86833dda Mon Sep 17 00:00:00 2001 From: Michael Weghorn Date: Fri, 9 Sep 2022 21:53:05 +0200 Subject: [PATCH] a11y atspi: Move comment to better place MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Move the comment about setting the ATSPI_STATE_MANAGES_DESCENDANTS state to the place where the state is actually set. Change-Id: Ia6956d36db86e935b4fab03b1892cf4e8a709c0f Reviewed-by: Tor Arne Vestbø --- src/gui/accessible/linux/atspiadaptor.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gui/accessible/linux/atspiadaptor.cpp b/src/gui/accessible/linux/atspiadaptor.cpp index 0d76a5f148..10837f2538 100644 --- a/src/gui/accessible/linux/atspiadaptor.cpp +++ b/src/gui/accessible/linux/atspiadaptor.cpp @@ -1135,9 +1135,6 @@ void AtSpiAdaptor::notify(QAccessibleEvent *event) } // For now we ignore these events case QAccessible::TableModelChanged: - // For tables, setting manages_descendants should - // indicate to the client that it cannot cache these - // interfaces. case QAccessible::ParentChanged: case QAccessible::DialogStart: case QAccessible::DialogEnd: @@ -1440,6 +1437,9 @@ bool AtSpiAdaptor::accessibleInterface(QAccessibleInterface *interface, const QS } else if (function == "GetState"_L1) { quint64 spiState = spiStatesFromQState(interface->state()); if (interface->tableInterface()) { + // For tables, setting manages_descendants should + // indicate to the client that it cannot cache these + // interfaces. setSpiStateBit(&spiState, ATSPI_STATE_MANAGES_DESCENDANTS); } QAccessible::Role role = interface->role();