New issue
Advanced search Search tips

Issue 919826 link

Starred by 1 user
Status: WontFix
Owner:
rbpotter@chromium.org
Closed: Jan 8
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: [Print Preview] Preview options doesn't get disabled on deleting the 'Scale' value

Project Member Reported by khush...@virtusa.com, Jan 8 
Chrome version: 73.0.3664.3 (Official Build) cac0e5b7d469aad82d6fd21ee684caeec1c5cf46-refs/branch-heads/3664@{#5} (32/64 bit)

OS: Windows (7, 8, 8.1, 10), Mac (10.13.1, 10.13.6, 10.14.3) and Linux (14.04 LTS)

What steps will reproduce the problem?
(1) Launch chrome and press 'Ctrl+P' to open Print Preview page.
(2) Click on 'More Settings' to expand.
(3) Traverse the focus over 'Scale' option using Tab key (value will get selected).
(4) Now delete the selected value and Observe.

Actual Result: Preview options doesn't get disabled on deleting the 'Scale' value.

Expected Result: Preview options should get disabled on deleting the 'Scale' value.

This is a regression issue broken in ‘M-71’, below is the Manual Regression range:
Good build: 71.0.3562.0 (Revision: 594164)
Bad build:  71.0.3563.0 (Revision: 594642)

You are probably looking for a change made after 594207 (known good), but no later than 594208 (first known bad).

CHANGE-LOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/5c8a9d48cba89e25c0a8c3610438b90c1ffa2100..957e18ed8c32465c693652d80e6166cb331e3770

Suspect: https://chromium.googlesource.com/chromium/src/+/957e18ed8c32465c693652d80e6166cb331e3770

@rbpotter: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

NOTE: Issue is also seen on M-71 Stable (build #71.0.3578.98), M-72 Beta (build #72.0.3626.28) and M-73 Canary (build #73.0.3665.0).

Kindly refer the attached screen-cast.

Thank You..!!
Actual Video.mp4
597 KB View Download
Expected Video.mp4
607 KB View Download

Comment 1 by rbpotter@chromium.org, Jan 8

Status: WontFix (was: Assigned)
Working as intended. Per UX guidance, options should not be disabled for an empty input that still has focus as the user may still be entering a value.

Comment 2 by khush...@virtusa.com, Jan 9 

With respect to comment #1,

@rbpotter: According to the said condition in comment 1, it should be followed same when user enters a wrong value and then clears slowly in the Scale field (here options get disabled even when there is focus seen in scale field).

Please refer the attached screen-cast for the same.

Thank You..!!
Disable Behaviour.mp4
827 KB View Download

Comment 3 by rbpotter@chromium.org, Jan 9 

No, there is a difference between clearing something valid to enter a new value and pausing on an invalid value, in which case we have to disable the other options, and then clearing. In other words, clearing the value doesn't change the state of the other options regardless of what that state is. Still WontFix, per UX guidance.
►

Sign in to add a comment