Open
Bug 1249327
Opened 9 years ago
Updated 4 years ago
Overlapping text highlights
Categories
(Core :: DOM: Selection, defect, P5)
Tracking
()
NEW
People
(Reporter: antlam, Unassigned)
References
Details
Attachments
(2 files)
Our work in bug 1097398 has overlapping text highlights when selecting multiple rows of text. In the same test sites, Chrome doesn't have this and by comparison we look and feel quite unfinished.
:TYLin suggested I check in here to see if this could be fixed.
Thanks!
https://bugzilla.mozilla.org/show_bug.cgi?id=1097398#c90
Flags: needinfo?(tlin)
Flags: needinfo?(mats)
Comment 1•9 years ago
|
||
Why are the lines overlapping in the first place?
Is that due to font-inflation in Fennec?
Flags: needinfo?(mats) → needinfo?(alam)
Reporter | ||
Comment 2•9 years ago
|
||
I have no idea why this is, but :TYLin mentioned this might need more investigation.
From a UX point of view, I just want to know if we can fix this issue :)
Flags: needinfo?(alam)
Comment 3•9 years ago
|
||
I wonder if you're actually referring to the element highlight Fennec will produce on long-press of a link that produces a contextmenu (vs a TextSelection) ... [0] ?
[0] https://www.dropbox.com/s/lqtyuxmla9t5ws7/bug_1249327_overlap.mp4?dl=0
Comment 4•9 years ago
|
||
Ah, here's more likely:
https://www.dropbox.com/s/6is6v99s1eb3b3b/bug_1249327_overlap.png?dl=0
Comment 5•9 years ago
|
||
I believe comment #4 is what Anthony concerns. Selection highlight on desktop also have this issue. However the highlight looks like without any opacity so it looks OK.
Flags: needinfo?(tlin)
Comment 6•9 years ago
|
||
The overlap there appears to be caused by the author setting 'line-height'.
Does anyone have examples of overlap with a default 'line-height'?
Reporter | ||
Comment 7•9 years ago
|
||
Thinking about this some more, Chrome might have the same issue but it might be our transparency that's actually causing this to "show" more.
I.e. where the orange gets darker cause its overlapping transparency.
:TYLin, I may have misunderstood how this works in the code. Perhaps we don't need any transparency in the text highlight? Would the text still show through though? If you have some time, could I see a screenshot of highlight opacity at 1, instead of 0.6 ?
that will help me decide what we should do here. Thanks!
Flags: needinfo?(tlin)
Comment 8•9 years ago
|
||
Here is the screenshot with text selection highlight using fennec orange #FF9500 and alpha 1. Too dark, right?
Flags: needinfo?(tlin) → needinfo?(alam)
Reporter | ||
Comment 9•9 years ago
|
||
(In reply to Ting-Yu Lin [:TYLin] (UTC+8) from comment #8)
> Created attachment 8721616 [details]
> fennec_orange_#FF9500_alpha_1.png
>
> Here is the screenshot with text selection highlight using fennec orange
> #FF9500 and alpha 1. Too dark, right?
Yeah it does seem a bit dark. I think the 0.6 opacity still creates the color we want.
My original theory was that the having no transparency (i.e. opacity = 1) would resolve the overlapping issue, but the color is too vivid without it.
Let me think about this some more. Thanks for your help :TYLin! :)
Again, I don't think this blocks us shipping these carets ATM.
Flags: needinfo?(alam)
Comment 11•4 years ago
|
||
Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.
If you have reason to believe this is wrong, please write a comment and ni :jstutte.
Severity: normal → S4
Priority: -- → P5
You need to log in
before you can comment on or make changes to this bug.
Description
•