Unnecessary selection of all radio button is seen in print preview page
Reported by
nutan.ga...@etouch.net,
Aug 2 2017
|
||||
Issue descriptionChrome 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.
,
Aug 2 2017
Correction: Chrome Version: 60.0.3112.90 6ee4392c6cef820c64ff50b5fd5eeb8596cd0394-refs/branch-heads/3112@{#702}
,
Aug 9 2017
,
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
,
Aug 11 2017
Note: Above issue is fixed on latest Canary version: 62.0.3182.0 (Official Build) on Windows,Linux and Mac OS Thanks you.
,
Aug 11 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by rbasuvula@chromium.org
, Aug 2 2017