Closed
Bug 1322352
Opened 8 years ago
Closed 6 years ago
Incorrect word wrapping when using pseudo elements
Categories
(Core :: Layout: Text and Fonts, defect, P3)
Tracking
()
RESOLVED
DUPLICATE
of bug 1283222
People
(Reporter: alexander, Unassigned)
References
()
Details
(Keywords: parity-chrome, parity-edge, Whiteboard: [pseudo-elements before inline word-wrap])
Attachments
(1 file)
420 bytes,
text/html
|
Details |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36
Steps to reproduce:
Used SPAN with absolutely positioned ::before pseudo-element withing text.
See this code example:
http://codepen.io/waterplea/pen/pNVxqE
Actual results:
Firefox started seeing this SPAN as a separate word, i.e. incorrect word wrapping occurred.
Expected results:
This SPAN should be a part of the word it's within, if there are no spaces around it. The issue is only reproducible in Firefox, works fine in IE, Chrome and Safari.
Severity: normal → minor
OS: Unspecified → Windows 7
Priority: -- → P4
Hardware: Unspecified → x86_64
Whiteboard: pseudo-elements before inline word-wrap
Updated•8 years ago
|
Component: Layout → Layout: Text
Updated•8 years ago
|
status-firefox50:
--- → affected
status-firefox51:
--- → affected
status-firefox52:
--- → affected
status-firefox53:
--- → affected
Keywords: regressionwindow-wanted
Priority: P4 → P3
Whiteboard: pseudo-elements before inline word-wrap → [pseudo-elements before inline word-wrap][parity-chrome][parity-edge]
Comment 1•8 years ago
|
||
TY, looks like the positioned pseudo element will break the work-break rule here. Any idea before we figure out the regression window?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(tlin)
Comment 3•8 years ago
|
||
Yeah, I did the bisec (mozregression) this afternoon. It turns out that we seem have this problem from the very beginning in 2009-01-01, which is the earliest version in mozregression. Might be the case that we didn't take into consideration while implementing word-break rule or something.
Here's the document that I used for the bisec (which should be exactly the same as listed in comment 0):
data:text/html, <style>div { width: 130px; border: 1px solid black; } span { color: red; } span::before { content: ''; position: absolute; }</style> <div>Firefox has a word<span>wrapping</span> issue</div>
Flags: needinfo?(jeremychen)
Comment 4•8 years ago
|
||
Thanks, Jeremy. Let's figure it out further.
Here's another case of similar issue, it is reproducible in Chrome too this time, I'll submit a bug there as well.
http://codepen.io/waterplea/pen/zombor
Comment 6•8 years ago
|
||
(In reply to alexander from comment #5)
> Here's another case of similar issue, it is reproducible in Chrome too this
> time, I'll submit a bug there as well.
> http://codepen.io/waterplea/pen/zombor
Looks like this link is no longer available. Could you provide another one? Or, maybe an raw html document (like comment 3) would be great!
Also, could you provide the link to the Chrome bug as well? Thank you.
Hi Jeremy. Here's a bin replicating that codepen that I used for Chrome bug:
http://output.jsbin.com/meyoxagufu/3
And here's that Chrome bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=674097
Comment 8•8 years ago
|
||
Busy with stylo, so unassign myself for now.
Assignee: jeremychen → nobody
Status: ASSIGNED → NEW
Comment 9•7 years ago
|
||
This sounds a lot like bug 1283222; I'm going to mark it "depends on" for now. I suspect fixing that bug might be all that's needed here, too.
Depends on: 1283222
Comment 10•7 years ago
|
||
![]() |
||
Comment 11•7 years ago
|
||
Mass bug change to replace various 'parity' whiteboard flags with the new canonical keywords. (See bug 1443764 comment 13.)
Keywords: parity-chrome,
parity-edge
Whiteboard: [pseudo-elements before inline word-wrap][parity-chrome][parity-edge] → [pseudo-elements before inline word-wrap]
Comment 12•6 years ago
|
||
The testcases here do appear to have been fixed by the patch in bug 1283222.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•