Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: 8d41f722d967fb2d3fd042265f76ba2bff51b6bf https://github.com/WebKit/WebKit/commit/8d41f722d967fb2d3fd042265f76ba2bff51b6bf Author: Alan Baradlay <za...@apple.com> Date: 2022-12-20 (Tue, 20 Dec 2022)
Changed paths: M Source/WebCore/layout/formattingContexts/inline/InlineContentBreaker.cpp M Source/WebCore/layout/formattingContexts/inline/InlineContentBreaker.h M Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.cpp Log Message: ----------- [IFC][hanging punctuation] InlineContentBreaker should not care if hanging content is leading or trailing https://bugs.webkit.org/show_bug.cgi?id=249652 Reviewed by Antti Koivisto. Hanging content is not supposed to take up space in the line box. It simply means that InlineContentBreaker should ignore it when checking for fit computation. From InlineContentBreaker's point of view it does not matter whether the hanging content is at the beginning or at the end of the continuous candidate content. * Source/WebCore/layout/formattingContexts/inline/InlineContentBreaker.cpp: (WebCore::Layout::InlineContentBreaker::ContinuousContent::reset): (WebCore::Layout::InlineContentBreaker::ContinuousContent::appendTrailingHangingContent): Deleted. (WebCore::Layout::InlineContentBreaker::ContinuousContent::appendLeadingHangingContent): Deleted. * Source/WebCore/layout/formattingContexts/inline/InlineContentBreaker.h: (WebCore::Layout::InlineContentBreaker::ContinuousContent::hangingContentWidth const): (WebCore::Layout::InlineContentBreaker::ContinuousContent::setHangingContentWidth): * Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.cpp: (WebCore::Layout::LineCandidate::InlineContent::appendInlineItem): Canonical link: https://commits.webkit.org/258181@main _______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes