New issue
Advanced search Search tips

Issue 805492 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Long press on "Welcome!" should not select text

Project Member Reported by zalcorn@chromium.org, Jan 24 2018

Issue description

When accessing touch accessible OOBE options with long press on "Welcome!" , the Welcome text should not be selectable.

Additionally, the Welcome text should announce itself as a button to Chromevox.
 
Cc: alemate@chromium.org
Labels: -Pri-2 Pri-1
Owner: tbarzic@chromium.org
Toni can you grab this one for M-65? Thanks!
Status: Started (was: Assigned)
Are you sure the Welcome text should announce itself as a button, considering that it
 * only supports long tap (so it might be unexpected that click does not work)
 * it is not displayed as a button in UI - the long press functionality would be no less discoverable using screen reader than looking at the UI


Though, the Advanced settings dialog doesn't seem to be reachable by keyboard - we might want to fix that.
Everything in the advanced settings dialog is already keyboard accessible via keyboard shortcuts, so we're good in that regard.
I'm okay not announcing as button as long as the text isn't selectable.
OK, sounds good
Project Member

Comment 6 by bugdroid1@chromium.org, Feb 8 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2b3970a2d688a88ab06dd8967acfd7eb90c6a28b

commit 2b3970a2d688a88ab06dd8967acfd7eb90c6a28b
Author: Toni Barzic <tbarzic@google.com>
Date: Thu Feb 08 01:25:22 2018

Add icon to OOBE advanced options dialog

Also, makes "Welcome!" text in welcome dialog not selectable.

BUG=797021,  805492 

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id8ac18d8a49c923fb0bd79a5f081c963ab4c0ae7
Reviewed-on: https://chromium-review.googlesource.com/907728
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#535236}
[modify] https://crrev.com/2b3970a2d688a88ab06dd8967acfd7eb90c6a28b/chrome/browser/resources/chromeos/login/oobe_welcome.css
[modify] https://crrev.com/2b3970a2d688a88ab06dd8967acfd7eb90c6a28b/chrome/browser/resources/chromeos/login/oobe_welcome.html
[modify] https://crrev.com/2b3970a2d688a88ab06dd8967acfd7eb90c6a28b/chrome/browser/resources/chromeos/login/oobe_welcome_dialog.css

Project Member

Comment 7 by bugdroid1@chromium.org, Feb 14 2018

Labels: merge-merged-3325
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/02b6463f6646db325aa62883383e7f83122eabb9

commit 02b6463f6646db325aa62883383e7f83122eabb9
Author: Toni Barzic <tbarzic@google.com>
Date: Wed Feb 14 02:32:29 2018

Add icon to OOBE advanced options dialog

Also, makes "Welcome!" text in welcome dialog not selectable.

BUG=797021,  805492 
TBR=tbarzic@google.com

(cherry picked from commit 2b3970a2d688a88ab06dd8967acfd7eb90c6a28b)

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id8ac18d8a49c923fb0bd79a5f081c963ab4c0ae7
Reviewed-on: https://chromium-review.googlesource.com/907728
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Toni Barzic <tbarzic@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#535236}
Reviewed-on: https://chromium-review.googlesource.com/917539
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/branch-heads/3325@{#459}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}
[modify] https://crrev.com/02b6463f6646db325aa62883383e7f83122eabb9/chrome/browser/resources/chromeos/login/oobe_welcome.css
[modify] https://crrev.com/02b6463f6646db325aa62883383e7f83122eabb9/chrome/browser/resources/chromeos/login/oobe_welcome.html
[modify] https://crrev.com/02b6463f6646db325aa62883383e7f83122eabb9/chrome/browser/resources/chromeos/login/oobe_welcome_dialog.css

Status: Fixed (was: Started)

Sign in to add a comment