Merge 9d13634 into 0b0bfb0

nvdaes · web-flow · commit c78e95c8666a · 2024-08-13T17:16:16.000Z
diff --git a/nvdaHelper/vbufBackends/gecko_ia2/gecko_ia2.cpp b/nvdaHelper/vbufBackends/gecko_ia2/gecko_ia2.cpp
@@ -1000,10 +1000,16 @@ VBufStorage_fieldNode_t* GeckoVBufBackend_t::fillVBuf(
 	}
 
 	BSTR value=NULL;
-	if(pacc->get_accValue(varChild,&value)==S_OK) {
-		if(value&&SysStringLen(value)==0) {
-			SysFreeString(value);
-			value=NULL;
+	if (pacc->get_accValue(varChild, &value) == S_OK) {
+		if (value) {
+			if (role == ROLE_SYSTEM_LINK) {
+				// For links, store the IAccessible value to handle same page link detection.
+				parentNode->addAttribute(L"IAccessible::value", value);
+			}
+			if (SysStringLen(value)==0) {
+				SysFreeString(value);
+				value=NULL;
+			}
 		}
 	}
 
diff --git a/source/NVDAObjects/IAccessible/ia2Web.py b/source/NVDAObjects/IAccessible/ia2Web.py
@@ -213,8 +213,30 @@ def _get_states(self):
 			if popupState:
 				states.discard(controlTypes.State.HASPOPUP)
 				states.add(popupState)
+		if self.isInternalLink:
+			states.add(controlTypes.State.INTERNAL_LINK)
 		return states
 
+	@property
+	def isInternalLink(self) -> bool:
+		if self.role != controlTypes.Role.LINK:
+			return False
+		if (
+			not hasattr(self, "treeInterceptor")
+			or self.treeInterceptor is None
+			or not self.treeInterceptor.documentConstantIdentifier
+		):
+			return False
+		documentConstantIdentifier = self.treeInterceptor.documentConstantIdentifier
+		if documentConstantIdentifier.endswith("/"):
+			documentConstantIdentifier = documentConstantIdentifier[:-1]
+		queryParamCharPos = documentConstantIdentifier.find("?")
+		if queryParamCharPos > 0:
+			documentConstantIdentifier = documentConstantIdentifier[:queryParamCharPos]
+		if self.value.startswith(f"{documentConstantIdentifier}#"):
+			return True
+		return False
+
 	def _get_landmark(self):
 		xmlRoles = self.IA2Attributes.get("xml-roles", "").split(" ")
 		landmark = next((xr for xr in xmlRoles if xr in aria.landmarkRoles), None)
diff --git a/source/controlTypes/state.py b/source/controlTypes/state.py
@@ -102,6 +102,7 @@ def negativeDisplayString(self) -> str:
 	HASPOPUP_GRID = setBit(48)
 	HASPOPUP_LIST = setBit(49)
 	HASPOPUP_TREE = setBit(50)
+	INTERNAL_LINK = setBit(51)
 
 
 STATES_SORTED = frozenset([State.SORTED, State.SORTED_ASCENDING, State.SORTED_DESCENDING])
@@ -204,6 +205,9 @@ def negativeDisplayString(self) -> str:
 	State.HASPOPUP_LIST: _("opens list"),
 	# Translators: Presented when a control has a pop-up tree.
 	State.HASPOPUP_TREE: _("opens tree"),
+	# Translators: Presented when a link destination points to the page containing the link.
+	# For example, links of a table of contents of a document with different sections.
+	State.INTERNAL_LINK: _("same page"),
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -1000,10 +1000,16 @@ VBufStorage_fieldNode_t* GeckoVBufBackend_t::fillVBuf(`
`1000`	`1000`	`}`
`1001`	`1001`
`1002`	`1002`	`BSTR value=NULL;`
`1003`		`- if(pacc->get_accValue(varChild,&value)==S_OK) {`
`1004`		`- if(value&&SysStringLen(value)==0) {`
`1005`		`- SysFreeString(value);`
`1006`		`- value=NULL;`
	`1003`	`+ if (pacc->get_accValue(varChild, &value) == S_OK) {`
	`1004`	`+ if (value) {`
	`1005`	`+ if (role == ROLE_SYSTEM_LINK) {`
	`1006`	`+ // For links, store the IAccessible value to handle same page link detection.`
	`1007`	`+ parentNode->addAttribute(L"IAccessible::value", value);`
	`1008`	`+ }`
	`1009`	`+ if (SysStringLen(value)==0) {`
	`1010`	`+ SysFreeString(value);`
	`1011`	`+ value=NULL;`
	`1012`	`+ }`
`1007`	`1013`	`}`
`1008`	`1014`	`}`
`1009`	`1015`