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

Issue 889835 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jan 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression: Unwanted I-beam pointer is seen at URI field in Edit printer overlay of chrome://settings/cupsPrinters page

Project Member Reported by rkalavakuntla@chromium.org, Sep 27

Issue description

Chrome Version:71.0.3562.0/11103.0.0 dev channel Daisy,Kip,Reks
OS:Chrome OS

What steps will reproduce the problem?
(1)Launch chrome,go to chrome://settings/cupsPrinters page ->Add a Printer
(2)From More actions.. >> edit the printer >>hover the mouse on URI field and observe

Actual: Unwanted I-beam pointer is seen at URI field in Edit printer overlay
Expected:Arrow pointer should be seen at URI field in Edit printer overlay

This is a Regression issue as same is working fine in M-68

Note: Issue is seen from M-69,M-70

Attached the screencast for reference

 
Actual.mp4
10.8 MB View Download
Expected.webm
294 KB View Download
Labels: -Pri-1 -M-71 Bolton-UI Pri-2
Status: Available (was: Untriaged)

Comment 2 by jlkl...@google.com, Jan 18 (4 days ago)

Cc: jlklein@chromium.org jhawkins@chromium.org
Owner: dpa...@chromium.org
From looking at the paper-input demo[1], it seems like there's a style difference between paper-input and cr-input for the "disabled" attribute. Looking at the expected video, the behavior looks like a disabled paper-input, but the cr-input in this dialog[2] uses the "readonly" attribute, which isn't quite the same. That being said, the "disabled" attribute in cr-input doesn't look like the one in paper-input (dashed underline, etc).

It seems like this regressed when switching from paper-input to cr-input in https://chromium-review.googlesource.com/c/chromium/src/+/1089863/.

Demetrios, do you know if the style difference for "disabled" cr-inputs is intentional in Chrome?


[1] https://www.webcomponents.org/element/@polymer/paper-input/demo/demo/index.html
[2] https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/printing_page/cups_edit_printer_dialog.html?rcl=6c061c7fec21b3bb683b1adc3aa14dfa72b9a59c&l=82

Comment 3 by dpa...@chromium.org, Jan 18 (4 days ago)

>  That being said, the "disabled" attribute in cr-input doesn't look like the one in paper-input (dashed underline, etc).

> Demetrios, do you know if the style difference for "disabled" cr-inputs is intentional in Chrome?

Yes, the changes in style are intentional. As part of MD Refresh most of the original MD styles (which are used in all paper-* elements by default) are obsolete.

Regarding the I-beam pointer itself, I'll take a closer look, but at first glance, a read-only field should be able to be copied, and therefore the I-beam pointer makes sense I think.

Comment 4 by jlkl...@google.com, Jan 19 (4 days ago)

Yeah, I agree about copying, etc. for readonly. Maybe this should actually be "disabled"?

Comment 5 by dpa...@chromium.org, Jan 19 (4 days ago)

Status: WontFix (was: Available)
The "disabled" and "readonly" states, AFAICT have been designed from a UX standpoint to be standalone states, such that "readonly" does not need to be always appear along with "disabled".

Closing this as WAI.

Comment 6 by jlkl...@google.com, Jan 19 (4 days ago)

Owner: weifangsun@chromium.org
Yeah, what I meant was that maybe this field is intended to be "disabled" *instead* of "readonly", but I don't really know the history here. Weifang, are you fine with closing this out as WAI?

Sign in to add a comment