:placeholder-shown matches an INPUT with non-empty value
Reported by
jeremia....@gmail.com,
Sep 17
|
|||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/69.0.3497.81 Chrome/69.0.3497.81 Safari/537.36 Steps to reproduce the problem: 1. I use an external stylesheet to define a placeholder color for the input element 2. I render the input element in a React component 3. The placeholder attribute of the input bases on the local state of that component What is the expected behavior? The text color of the input is the default one, because it was never defined in a stylesheet What went wrong? The text color of the input is equal to the placeholder color Did this work before? N/A Chrome version: 69.0.3497.81 Channel: n/a OS Version: 69.0.3497.81 Flash Version: I created a minimal code snippet on Stackoverflow: https://stackoverflow.com/questions/52369649/text-input-color-equals-placeholder-color-in-react-app
,
Sep 17
,
Sep 18
Able to reproduce the issue on Mac 10.13.3, Win-10 and Ubuntu 14.04 using chrome reported version #69.0.3497.81 and latest canary #71.0.3553.2. This is a non-regression issue as it is observed from M60 old builds. Hence, marking it as untriaged to get more inputs from dev team. Thanks...!!
,
Sep 26
,
Sep 27
,
Sep 28
,
Oct 15
,
Oct 15
,
Oct 21
Hi, Can I try this issue?
,
Oct 22
> Can I try this issue? Please go ahead.
,
Nov 1
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4410851849fa63e584853c41f4ad7755ce4f7445 commit 4410851849fa63e584853c41f4ad7755ce4f7445 Author: HyeockJinKim <kherootz@gmail.com> Date: Thu Nov 01 09:27:47 2018 Always update placeholder's visibility Always call SetPlaceholderVisibility if !placeholder to prevent :placeholder-shown style from being applied Bug: 884696 Change-Id: I5f078dad825de91253bc99d133dc7978a59bbda3 Reviewed-on: https://chromium-review.googlesource.com/c/1292666 Reviewed-by: Kent Tamura <tkent@chromium.org> Commit-Queue: HyeockJin Kim <kherootz@gmail.com> Cr-Commit-Position: refs/heads/master@{#604529} [add] https://crrev.com/4410851849fa63e584853c41f4ad7755ce4f7445/third_party/WebKit/LayoutTests/external/wpt/css/selectors/selector-placeholder-shown-emptify-placeholder.html [modify] https://crrev.com/4410851849fa63e584853c41f4ad7755ce4f7445/third_party/blink/renderer/core/html/forms/text_control_element.cc
,
Nov 1
Assuming this is fixed by the landed CL.
,
Nov 1
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by jeremia....@gmail.com
, Sep 17