Bug 157698 - REGRESSION(r189567): text-overflow: ellipsis doesn't work with display: flex
Summary: REGRESSION(r189567): text-overflow: ellipsis doesn't work with display: flex
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-05-13 17:42 PDT by Ryosuke Niwa
Modified: 2016-12-15 15:53 PST (History)
5 users (show)

See Also:

Attachments
Reduction (282 bytes, text/html)
2016-05-13 17:42 PDT, Ryosuke Niwa 		no flags Details
Test case rendered by Blink,WebKit and Firefox (63.63 KB, image/png)
2016-05-19 04:10 PDT, Sergio Villar Senin 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2016-05-13 17:42:51 PDT 
Created attachment 278901 [details]
Reduction

text-overflow: ellipsis doesn't work when it's specified with display: flex.
Comment 1 Ryosuke Niwa 2016-05-13 17:43:01 PDT 
<rdar://problem/26075120>
Comment 2 Ryosuke Niwa 2016-05-13 19:35:43 PDT 
This is a regression from http://trac.webkit.org/changeset/189567.
Comment 3 Ryosuke Niwa 2016-05-13 19:36:09 PDT 
Sergio, would you be able to look into this regression?
Comment 4 Jon Lee 2016-05-16 14:05:15 PDT 
Might this be correct behavior? I don't see an ellipsis in Chrome or Firefox.
Comment 5 Sergio Villar Senin 2016-05-19 04:10:29 PDT 
Created attachment 279378 [details]
Test case rendered by Blink,WebKit and Firefox

Are you sure this is a regression Ryosuke?

It looks like we output the same outcome as Chrome and Firefox. I am far from a Flexbox expert but it looks like the three web engines agree on the expected results.

I'm attaching a screenshot rendering your test case (I forced an Arial 18px font to get exactly the same output) using Blink ToT, WK ToT and FF ToT.