245947 2022-10-02 10:47:51 -0700 Enable justified text for white-space: pre-wrap 2022-10-04 14:51:16 -0700 1 1 1 Unclassified WebKit Layout and Rendering Safari Technology Preview Unspecified Unspecified RESOLVED WORKSFORME https://bugs.webkit.org/show_bug.cgi?id=84448 BrowserCompat P2 Normal --- 1 ahmad.saleem792 webkit-unassigned bfulgham karlcow simon.fraser zalan oldest_to_newest 1902720 0 ahmad.saleem792 2022-10-02 10:47:51 -0700 Hi Team, I was looking into another Blink Commit merge and noticed following: Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=174895 Which needs to be merged here: https://github.com/WebKit/WebKit/blob/89d57a6f366a8abe5d012e38f160f81388b46429/Source/WebCore/rendering/LegacyLineLayout.cpp#L613 and add following: whiteSpace() == WhiteSpace::Pre I will see whether it is easier to do or not but will try with PR. Although, I think it would be good to keep this for future. Thanks! 1903239 1 ahmad.saleem792 2022-10-04 14:51:16 -0700 We don't need to do this change because we already have it fixed along the way and Safari 16 behavior is consistent with Chrome Canary 108. Example test case - https://jsfiddle.net/b7j5412r/show If I do this change, which I was trying to do it in below PR: https://github.com/WebKit/WebKit/pull/4912 It reverts back us to old behavior and removed "justified". I have discussed with Alan and now closing this bug but if needed and I am not right, please let me know I will be able to turn around this patch. Marking this as "RESOLVED WORKSFORME".