New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 751440 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Unnecessary selection of all radio button is seen in print preview page

Reported by nutan.ga...@etouch.net, Aug 2 2017

Issue description

Chrome Version: 62.0.3174.0 02ca96925ccb743e83ad34bee8595e4a6eec8443-refs/heads/master@{#491203}
OS: Windows (7,8,10), Mac (10.11.6, 10.12.3, 10.12.5),Linux (14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome, navigate to NTP and give Print command
2. On print preview page LHS, click just above second radio button of 'Pages' 
3. Observe

Actual: Unnecessary selection of all radio button is seen
Expected: Unnecessary selection of all radio button should not be seen

This is a non-regression issue, seen from M-45

Kindly review the attached video for reference.
 
Actual.mov
2.4 MB Download
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!
Correction:
Chrome Version: 60.0.3112.90 6ee4392c6cef820c64ff50b5fd5eeb8596cd0394-refs/branch-heads/3112@{#702}
Owner: rbpotter@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 10 2017

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

commit 02b1d883044393df590dfbe0864eadb2cc2d3195
Author: rbpotter <rbpotter@chromium.org>
Date: Thu Aug 10 17:12:45 2017

Print Preview: Fix radio button selection issue

Print preview relied on relatedTarget to filter for mouse clicks that
blurred the pages input by selecting the corresponding radio button.
To populate relatedTarget, the page settings wrapper div needs to
have a tabindex.

Bug:  751440 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I3475c473451a2ae15aa61f4845a38305e14445cf
Reviewed-on: https://chromium-review.googlesource.com/607452
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493437}
[modify] https://crrev.com/02b1d883044393df590dfbe0864eadb2cc2d3195/chrome/browser/resources/print_preview/settings/page_settings.html
[modify] https://crrev.com/02b1d883044393df590dfbe0864eadb2cc2d3195/chrome/browser/resources/print_preview/settings/page_settings.js

Labels: TE-Verified-62.0.3182.0 TE-Verified-M62
Note: Above issue is fixed on latest Canary version: 62.0.3182.0 (Official Build) on Windows,Linux and Mac OS

Thanks you.


Expected Result.mov
3.0 MB Download
Status: Fixed (was: Started)

Sign in to add a comment