New issue
Advanced search Search tips

Issue 872251 link

Starred by 1 user
Status: Verified
Owner:
rbpotter@chromium.org
Closed: Aug 17
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

[Print Preview] Unable to scroll the 'Settings section' page via keyboard.

Reported by dchau...@etouch.net, Aug 8 
Chrome Version: 68.0.3440.106 (Official Build) Revision	1c32c539ce0065a41cb79da7bfcd2c71af1afe62-refs/branch-heads/3440@{#794} (32/64-bit)
OS: Win(7,8,8.1,10), Mac(10.12.6 , 10.13.1 , 10.13.6, 10.14) and Linux(14.04 LTS).

Precondition : Enable #enable-new-print-preview flag from chrome://flags

What steps will reproduce the problem?
1. Launch Chrome and give print command on any webpage using "Ctrl/Command + P" key from keyboard.
2. Keep focus on 'More settings' button using 'Tab' key and press 'Enter' key from keyboard.
3. Now press 'Down arrow' key from keyboard and observe.

Actual: 'Settings section' doesn't scroll on pressing 'Down/Up arrow' key from keyboard.
Expected: 'Settings section' should scroll on pressing 'Down/Up arrow' key from keyboard.

This is a non-regression issue, seen from M-68 series as 'More settings' button is introduced from build # 68.0.3410.0 in new print preview UI.

NOTE: This issue is also reproducible on Stable #68.0.3440.84, Beta #69.0.3497.23 and Canary #70.0.3516.0

Kindly review the attached screen-cast for reference.

Thank you.
Actual behavior.mp4
729 KB View Download
Expected behavior.mp4
867 KB View Download

Comment 1 by rbasuvula@chromium.org, Aug 8

Status: Untriaged (was: Unconfirmed)
As this being a Non-Regression issue, changing the status to Untriaged so that the issue would get addressed.

Thank You!

Comment 2 by thestig@chromium.org, Aug 8

Owner: rbpotter@chromium.org
Status: Assigned (was: Untriaged)
Same thing happens after opening "More Settings" and using shift + Tab to put the keyboard focus on the "Change..." button.
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 16 

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

commit 0af13b1919674af8aec2212b504c40c6df026f7d
Author: rbpotter <rbpotter@chromium.org>
Date: Thu Aug 16 15:50:51 2018

Print Preview: Correctly detect scrollbars

In Polymerized Print Preview, the event target is always
print-preview-app, which does not have an ancestor with a scrollbar
even if one exists on the page (e.g. in the settings sections).
Examine the event path instead of the target and its ancestors to
determine if there is a scrollbar that should receive the arrow key
events instead of the plugin.

Bug:  872251 
Change-Id: I034542d04644e1b47fd1961a9b984a48843b2b40
Reviewed-on: https://chromium-review.googlesource.com/1174995
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Scott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583671}
[modify] https://crrev.com/0af13b1919674af8aec2212b504c40c6df026f7d/chrome/browser/resources/print_preview/new/preview_area.js

Comment 4 by dchau...@etouch.net, Aug 17

Labels: TE-Verified-70.0.3525.0 TE-Verified-M70
Update:
Rechecked the above issue on Win(7,8,8.1,10), Mac(10.12.6 , 10.13.1 , 10.13.6, 10.14) and Linux(14.04 LTS) machines using latest canary #70.0.3525.0 and issue is fixed. Hence adding TE-Verified labels.

please refer below attached screen-cast for reference.

Thank You.
Fixed_behavior.mp4
712 KB View Download

Comment 5 by rbpotter@chromium.org, Aug 17

Status: Verified (was: Assigned)
►

Sign in to add a comment