Merge 65b491e into a3a5fbb

SaschaCowley · web-flow · commit e45c984413e5 · 2024-01-11T03:27:13.000Z
diff --git a/source/NVDAObjects/IAccessible/__init__.py b/source/NVDAObjects/IAccessible/__init__.py
@@ -1,5 +1,5 @@
 # A part of NonVisual Desktop Access (NVDA)
-# Copyright (C) 2006-2023 NV Access Limited, Babbage B.V., Cyrille Bougot
+# Copyright (C) 2006-2024 NV Access Limited, Babbage B.V., Cyrille Bougot, Sascha Cowley
 # This file is covered by the GNU General Public License.
 # See the file COPYING for more details.
 
@@ -1348,8 +1348,18 @@ def _tableHeaderTextHelper(self, axis):
 			# Each header must be fetched from the headers array once and only once,
 			# as it gets released when it gets garbage collected.
 			for i in range(nHeaders):
+				header = headers[i]
+				# Chromium exposes cells as their own headers, so exclude cells with the same `Iaccessible2::uniqueID`.
+				if self.IA2UniqueID is not None:  # No point checking if we don't have this cell's UID
+					try:
+						headerUniqueID = header.QueryInterface(IA2.IAccessible2).uniqueID
+					except COMError:
+						log.debugWarning("could not get IAccessible2::uniqueID to use as headerUniqueID",exc_info=True)
+						headerUniqueID = None
+					if self.IA2UniqueID == headerUniqueID:
+						continue
 				try:
-					text = headers[i].QueryInterface(IA2.IAccessible2).accName(0)
+					text = header.QueryInterface(IA2.IAccessible2).accName(0)
 				except COMError:
 					continue
 				if not text:
diff --git a/source/compoundDocuments.py b/source/compoundDocuments.py
@@ -1,7 +1,7 @@
 # A part of NonVisual Desktop Access (NVDA)
 # This file is covered by the GNU General Public License.
 # See the file COPYING for more details.
-# Copyright (C) 2010-2022 NV Access Limited, Bram Duvigneau
+# Copyright (C) 2010-2023 NV Access Limited, Bram Duvigneau, Sascha Cowley
 from typing import (
 	Optional,
 	Dict,
@@ -172,6 +172,8 @@ def _getControlFieldForObject(self, obj: NVDAObject, ignoreEditableText=True):
 			field["table-id"] = 1 # FIXME
 			field["table-rownumber"] = obj.rowNumber
 			field["table-columnnumber"] = obj.columnNumber
+			field["table-rowheadertext"] = obj.rowHeaderText
+			field["table-columnheadertext"] = obj.columnHeaderText
 			# Row/column span is not supported by all implementations (e.g. LibreOffice)
 			try:
 				field['table-rowsspanned']=obj.rowSpan
diff --git a/user_docs/en/changes.t2t b/user_docs/en/changes.t2t
@@ -6,10 +6,13 @@ What's New in NVDA
 = 2024.2 =
 
 == New Features ==
+
+- Reporting row and column headers is now supported in contenteditable HTML elements. (#14113)
 - In Windows 11, NVDA will announce alerts from voice typing and suggested actions including the top suggestion when copying data such as phone numbers to the clipboard (Windows 11 2022 Update and later). (#16009, @josephsl)
 -
 
 
+
 == Changes ==
 
 

Original file line number	Diff line number	Diff line change
`@@ -6,10 +6,13 @@ What's New in NVDA`
`6`	`6`	`= 2024.2 =`
`7`	`7`
`8`	`8`	`== New Features ==`
	`9`	`+`
	`10`	`+- Reporting row and column headers is now supported in contenteditable HTML elements. (#14113)`
`9`	`11`	`- In Windows 11, NVDA will announce alerts from voice typing and suggested actions including the top suggestion when copying data such as phone numbers to the clipboard (Windows 11 2022 Update and later). (#16009, @josephsl)`
`10`	`12`	`-`
`11`	`13`
`12`	`14`
	`15`	`+`
`13`	`16`	`== Changes ==`
`14`	`17`
`15`	`18`