white-space: nowrap and multiple text nodes
Reported by
carlosro...@gmail.com,
Mar 23 2016
|
|||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.87 Safari/537.36 Example URL: https://jsfiddle.net/dotpao/7hbt9v88/5/ Steps to reproduce the problem: 1. Go to the jsfiddle https://jsfiddle.net/dotpao/7hbt9v88/5/ 2. Click in together 3. Click in separate What is the expected behavior? With white-space:nowrap, the text 'abcde' in the jsfiddle must not break in two lines. Regardless the text was added as separate text nodes. What went wrong? Text 'abcde' in the jsfiddle breaks in two lines when is added as separate text nodes. Does it occur on multiple sites: Yes Is it a problem with a plugin? No Did this work before? Yes Chrome 48 Does this work in other browsers? Yes Chrome version: 49.0.2623.87 Channel: stable OS Version: OS X 10.10.5 Flash Version: Shockwave Flash 21.0 r0 It happens also on Chrome 49 for Windows
,
Mar 23 2016
Looks like a recent regression, thanks for the report!
,
Mar 24 2016
You are probably looking for a change made after 367603 (known good), but no later than 367617 (first known bad). CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/f0f095bd29216918bffaaa381d6d568d51a0e87f..4762e2dc9c5ef4f8059ff499aa2600b1abfef7a7 r367612 (510cd3fd7980c5ebb971c3ed14d6a4a99ce24277) is within the range and changes no-wrap handling. Do you have the bandwidth to take this on robhogan? If not assign it back to me and I'll take a look.
,
Mar 26 2016
,
Mar 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/67071ef5bb2ec084b4afdf5075c702ab0319d183 commit 67071ef5bb2ec084b4afdf5075c702ab0319d183 Author: robhogan <robhogan@gmail.com> Date: Tue Mar 29 20:04:42 2016 Stop tracking trailing-space when we leave it We were forgetting to reset our picture of trailing-space width when we re-entered a run of normal text. BUG= 597193 Review URL: https://codereview.chromium.org/1838723002 Cr-Commit-Position: refs/heads/master@{#383798} [add] https://crrev.com/67071ef5bb2ec084b4afdf5075c702ab0319d183/third_party/WebKit/LayoutTests/fast/text/whitespace/nowrap-previous-trailing-space-expected.txt [add] https://crrev.com/67071ef5bb2ec084b4afdf5075c702ab0319d183/third_party/WebKit/LayoutTests/fast/text/whitespace/nowrap-previous-trailing-space.html [modify] https://crrev.com/67071ef5bb2ec084b4afdf5075c702ab0319d183/third_party/WebKit/Source/core/layout/line/BreakingContextInlineHeaders.h
,
Apr 7 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by tkent@chromium.org
, Mar 23 2016Labels: Needs-Bisect