Steps to reproduce: 1) start NVDA screen reader on Windows 2) open attachment 178634 [details] from bug 147742 3) Click the "Run the macro" button 4) put focus on the table and move around between the cells using arrow keys Expected result: NVDA should announce the focused cell and its content. Actual result: NVDA doesn't announce anything about the newly focused cell, just what key was pressed ("down arrow", "right arrow", etc.) tested with NVDA 2023.1 Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community Build ID: 88790291ddc00bfa18d3f7bcc0825b1ce14cb490 CPU threads: 8; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win Locale: en-US (en_DE); UI: en-US Calc: threaded Note that this works fine on Linux (Debian testing) with Orca 43.1-1 and the gtk3 VCL plugin instead: Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community Build ID: 94ca402cd1fe2fd9776d08448f7216b7f638e69a CPU threads: 12; OS: Linux 6.3; UI render: default; VCL: gtk3 Locale: en-GB (en_GB.UTF-8); UI: en-US Calc: threaded
From a user perspective, this looks a bit like tdf#99609, just with a different control. Another issue I see in that scenario is that LO crashes on Windows once I close the window with the grid control when NVDA is running. Note: It might also make sense to consider how exactly announcement for tables and selected rows by NVDA should be and possibly take a look at this (either in the context of this bug, or as a follow-up, might also need looking into what NVDA does and possibly suggesting changes there), s.a. bug 99609 comment 17.
(In reply to Michael Weghorn from comment #1) > Note: It might also make sense to consider how exactly announcement for > tables and selected rows by NVDA should be and possibly take a look at this > (either in the context of this bug, or as a follow-up, might also need > looking into what NVDA does and possibly suggesting changes there), s.a. bug > 99609 comment 17. Looking at other scenarios might give some input here, e.g. how the tdf#99609 scenario is announced by Orca on Linux or how NVDA announces all row headers and values of the currently selected rows in the "Add-ins" table in "File" > "Options" > "Add-ins".
I'm looking into this.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/256926cace5819b0107e8d24579e05bcab9b5a7e tdf#156473 a11y: Don't use selection index as child index It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/de9de202a6336f3ba8da0d2deeb8e1e34ac569ab tdf#156473 a11y: Properly retrireve a11y context from XAccessible It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/18aed6fc4ff5b981ae7deb6584197b661721e7df tdf#156473 a11y: Use actual row/col index/count in grid control header It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/f7ff0468616d3ed8ded78598649a32bcb4e1ef1a tdf#156473 a11y: Implement AccessibleGridControlHeader::getAccessibleCellAt It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/2b30d37bd555188733a006e1a5796461ab11d326 tdf#156473 wina11y: Fix invalid write due to row/col mismatch It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "libreoffice-7-5": https://git.libreoffice.org/core/commit/1019104d9b2754aa8efaadb4aef046e732189b9b tdf#156473 wina11y: Fix invalid write due to row/col mismatch It will be available in 7.5.6. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "libreoffice-7-6": https://git.libreoffice.org/core/commit/b4b79640ba2acbfecd4b2aa19a8222ff5666156b tdf#156473 wina11y: Fix invalid write due to row/col mismatch It will be available in 7.6.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/cd00bc50a7425e2f58d54de80b714cb6b0362dce tdf#156473 a11y: Keep a reference to table's a11y object It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michael Weghorn committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/bf15445b7102a4e2bea2f96891093bc4bf07c4c6 tdf#99609 tdf#156473 a11y: Check whether row header exists It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
The original issues that this bug is about and some more are fixed now. (In reply to Michael Weghorn from comment #2) > (In reply to Michael Weghorn from comment #1) > > Note: It might also make sense to consider how exactly announcement for > > tables and selected rows by NVDA should be and possibly take a look at this > > (either in the context of this bug, or as a follow-up, might also need > > looking into what NVDA does and possibly suggesting changes there), s.a. bug > > 99609 comment 17. > Looking at other scenarios might give some input here, e.g. how the > tdf#99609 scenario is announced by Orca on Linux or how NVDA announces all > row headers and values of the currently selected rows in the "Add-ins" table > in "File" > "Options" > "Add-ins". I've created a separate bug report to keep track of this now: tdf#156968 @Thorsten: I've added you in CC there as well.