Ensure that role label for landmark is once again abbreviated in brai…

…lle (PR #13160) Fixes #13158 PR #12814 replaced usages of deprecated controlTypes.roleLabels with role.displayString. However braille has its own mapping of roles to labels which should be used for abbreviations of roles for braille. The role label used for speech has been mistakenly used for a role of landmark.
nvaccess · Dec 13, 2021 · ded5ef5 · ded5ef5
1 parent 3ff29a1
commit ded5ef5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/source/braille.py b/source/braille.py
@@ -719,7 +719,7 @@ def getControlFieldBraille(  # noqa: C901
 	roleText = field.get('roleTextBraille', field.get('roleText'))
 	landmark = field.get("landmark")
 	if not roleText and role == controlTypes.Role.LANDMARK and landmark:
-		roleText = f"{controlTypes.Role.LANDMARK.displayString} {landmarkLabels[landmark]}"
+		roleText = f"{roleLabels[controlTypes.Role.LANDMARK]} {landmarkLabels[landmark]}"
 
 	content = field.get("content")
 

diff --git a/user_docs/en/changes.t2t b/user_docs/en/changes.t2t
@@ -9,6 +9,8 @@ What's New in NVDA
 
 
 == Bug Fixes ==
+- Landmark is once again abbreviated in braille. #13158
+-
 
 
 = 2021.3 =