Commit b14cbf9d authored by Akihiro Ota's avatar Akihiro Ota Committed by Commit Bot

ChromeVox Tutorial: Remove automatic reading behavior for some lessons.

UXR reported that users were confused by the automatic reading
behavior for tutorial lessons that allowed them to move freely. This is
because they would hear the lesson read automatically, but hear the same
content again when navigating.

Instead of reading the entire lesson, we now give a hint for navigating
when these types of lessons are shown.

Fixed: 1142041
AX-Relnotes: N/A
Change-Id: I35ccb209d7ad542df965d99b92b2e3921abe38b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2519038Reviewed-by: default avatarDavid Tseng <dtseng@chromium.org>
Commit-Queue: Akihiro Ota <akihiroota@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825060}
parent 7e860b7e
......@@ -558,9 +558,6 @@ Polymer({
// Read the title since initial focus gets placed on the first piece of
// text content.
this.readCurrentLessonTitle();
} else {
// Otherwise, automatically read current lesson content.
setTimeout(this.readCurrentLessonContent.bind(this), 1000);
}
},
......
......@@ -437,8 +437,8 @@ TEST_F('ChromeVoxTutorialTest', 'AutoReadTitle', function() {
});
});
// Tests that the content of a non-interactive lesson is read when shown.
TEST_F('ChromeVoxTutorialTest', 'AutoReadLesson', function() {
// Tests that we read a hint for navigating a lesson when it is shown.
TEST_F('ChromeVoxTutorialTest', 'LessonHint', function() {
const mockFeedback = this.createMockFeedback();
this.runWithLoadedTree(this.simpleDoc, async function(root) {
await this.launchAndWaitForTutorial();
......@@ -454,10 +454,7 @@ TEST_F('ChromeVoxTutorialTest', 'AutoReadLesson', function() {
tutorial.showLesson(0);
})
.expectSpeech('On, Off, and Stop', 'Heading 1')
.expectSpeech(
'To temporarily stop ChromeVox from speaking, ' +
'press the Control key.')
.expectSpeech('To turn ChromeVox on or off, use Control+Alt+Z.')
.expectSpeech('Press Search + left/right arrow to navigate the lesson')
.replay();
});
});
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment