Closed
Bug 737273
Opened 12 years ago
Closed 12 years ago
drag feedback for input with placeholder showing has placeholder color
Categories
(Core :: Layout: Form Controls, defect)
Core
Layout: Form Controls
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: karlt, Assigned: fryn)
References
Details
(Whiteboard: [fixed-in-bug-673873])
STR:
1. Load attachment 590476 [details].
2. Drag text over the text input control.
Expected results:
Regular caret at drop location.
Actual results
Drag feedback caret is faint.
Reporter | ||
Comment 1•12 years ago
|
||
There are some comments about styling in bug 673873 that may be related.
Comment 2•12 years ago
|
||
I'm not sure I understand what the bug is actually. Could you be more detailed?
Reporter | ||
Comment 3•12 years ago
|
||
For text inputs without the placeholder showing, the drag feedback caret is black, as expected. When the placeholder is showing, the drag feedback is the same color as the placeholder - grey.
Comment 4•12 years ago
|
||
Indeed. I misunderstood your first comment actually. I hope this could be fixed if we use a pseudo-element instead of a pseudo-class to style the placeholder.
Should we really be showing the placeholder while there's a drag feedback caret? It seems to me that should hide the placeholder anyway.
Reporter | ||
Comment 6•12 years ago
|
||
Hiding the placeholder when showing drag feedback sounds reasonable to me as it gives an indication of what would happen if the drop is made at that mouse position.
Comment 7•12 years ago
|
||
Given that bug 673873 wants to make the placeholder stays when the element has the focus but no value, I think the behavior described in comment 5 would be inconsistent. Also, it might be interesting to see what native UI do here.
Assignee | ||
Comment 8•12 years ago
|
||
Fixed in bug 673873. Proper fix will be done in bug 737786.
Assignee: nobody → fryn
Status: NEW → RESOLVED
Closed: 12 years ago
Depends on: 673873
OS: Linux → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Whiteboard: [fixed-in-bug-673873]
Target Milestone: --- → mozilla15
Version: Other Branch → Trunk
You need to log in
before you can comment on or make changes to this bug.
Description
•