Closes #293 - review: Clarify a11y hack comment.

mozilla-mobile · Jan 16, 2018 · 82a8db5 · 82a8db5
1 parent 421464f
commit 82a8db5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/app/src/main/res/layout/fragment_new_settings.xml b/app/src/main/res/layout/fragment_new_settings.xml
@@ -43,7 +43,8 @@
                  on the state of the checkbox child. However, unlike for TextViews, VoiceView does not read the
                  checked state of child CheckBoxes so we have to work around this:
 
-                 When VoiceView is enabled, we dynamically change focusability so the parent layout is
+                 When VoiceWiev is enabled, we sacrifice consistency in visual highlighting in order to get the
+                 correct content description by dynamically changing focusability so that the parent layout is
                  unfocusable and we focus the CheckBox instead, which announces the correct data. Since the
                  TextView is no longer spoken, we dynamically set the contentDescription of the CheckBox to the
                  text of the TextView to cause them to have the same announcement.