drag feedback for input with placeholder showing has placeholder color

RESOLVED FIXED in mozilla15

Status

()

Core
Layout: Form Controls
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: karlt, Assigned: fryn)

Tracking

Trunk
mozilla15
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed-in-bug-673873])

(Reporter)

Description

5 years ago
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

5 years ago
There are some comments about styling in bug 673873 that may be related.
I'm not sure I understand what the bug is actually. Could you be more detailed?
(Reporter)

Comment 3

5 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.
Depends on: 737786
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

5 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.
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

5 years ago
Fixed in bug 673873.

Proper fix will be done in bug 737786.
Assignee: nobody → fryn
Status: NEW → RESOLVED
Last Resolved: 5 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.