Closed Bug 1266715 Opened 4 years ago Closed 4 years ago

Fix the space between the blinking cursor/selection highlight and the carets

Categories

(Firefox for Android :: Text Selection, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox47 --- unaffected
firefox48 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

References

Details

(Keywords: regression)

Attachments

(3 files)

This is a follow-up of bug 1262755. I changed the style of the carets image from "bottom: 0" to "top: 0", this causes the offset in content.css [1] become malfunction. So the caret image will be too close to the blinking cursor on Fennec. Other platforms do not have this issue because they do not have these offset.

[1] https://dxr.mozilla.org/mozilla-central/source/mobile/android/themes/core/content.css#342,348,354
Keywords: regression
Attached image correct-space.png
This is the screenshot of the correct space between the blinking cursor and the caret, which is the same before fixing bug 1262755.
Comment on attachment 8744307 [details]
MozReview Request: Bug 1266715 - Fix space between blinking cursor/selection highlight and the caret.

https://reviewboard.mozilla.org/r/48449/#review45169
Attachment #8744307 - Flags: review?(snorp) → review+
https://hg.mozilla.org/mozilla-central/rev/8f0b8d0c6bc3
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.