Summary: | [LFC][IFC] Create inline boxes for hard line breaks(<br>) and word wrap opportunities (<wbr>) | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | zalan <zalan> | ||||||||
Component: | Layout and Rendering | Assignee: | zalan <zalan> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bfulgham, koivisto, simon.fraser, webkit-bug-importer, zalan | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
zalan
2020-09-26 20:16:10 PDT
Created attachment 409808 [details]
Patch
Created attachment 409811 [details]
Patch
Committed r267657: <https://trac.webkit.org/changeset/267657> All reviewed patches have been landed. Closing bug and clearing flags on attachment 409811 [details]. Reopening to attach new patch. Created attachment 409837 [details]
Patch
Committed r267666: <https://trac.webkit.org/changeset/267666> All reviewed patches have been landed. Closing bug and clearing flags on attachment 409837 [details]. Comment on attachment 409811 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=409811&action=review > Source/WebCore/layout/FormattingState.cpp:55 > + // Anonymous text wrappers do not need display boxes. You mean "do not need box geometry"? This is not about Display::Box. (In reply to Simon Fraser (smfr) from comment #8) > Comment on attachment 409811 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=409811&action=review > > > Source/WebCore/layout/FormattingState.cpp:55 > > + // Anonymous text wrappers do not need display boxes. > > You mean "do not need box geometry"? This is not about Display::Box. Yeah, you are right. I should have rephrased it (this comment is from before when box geometry was really just a Display::Box) |