Issue metadata
Sign in to add a comment
|
Space is missing when using word-wrap: break-word and floated 100% width images and two-letter words
Reported by
mflo...@gmail.com,
Aug 22 2016
|
||||||||||||||||||||
Issue descriptionChrome Version : 52.0.2743.116 (Official Build) (64-bit) OS: Mac OS X Blink: 537.36 (@9115ecad1cae66fd5fe52bd9120af643384fd6f3) URLs (if applicable) : http://jsbin.com/zepapo/edit?html,css,output Other browsers tested: Add OK or FAIL, along with the version, after other browsers where you have tested this issue: Safari: OK 9.1.2 (11601.7.7) Firefox: OK 48.0.1 Chrome canary: FAIL 54.0.2832.2 (Official Build) canary (64-bit) What steps will reproduce the problem? (1) Open chrome (2) Goto http://jsbin.com/zepapo/edit?html,css,output What is the expected result? There should be a space between It and is under the first picture. What happens instead? The space is missing making it look like "Itis". Please provide any additional information below. Attach a screenshot if possible. It only happens when the first word is two characters long. One letter works fine, as well as more (tested 3-5). The minimum code to reproduce is something like this: <style> figure { width: 100%; float: left; } img { width: 100%; } .container { word-wrap: break-word; } </style> <div class="container"> <figure> <img src="http://placekitten.com/800/50" alt="kitten" /> </figure> <span>It is missing a space when first word is two letters long</span> </div>
,
Sep 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d5bcb937b6dd019905eb7bbd9834b1eb26213f2a commit d5bcb937b6dd019905eb7bbd9834b1eb26213f2a Author: kojii <kojii@chromium.org> Date: Thu Sep 15 04:16:20 2016 Fix BreakingContext::rewindToMidWordBreak to set m_currentCharacterIsSpace This patch fixes BreakingContext::rewindToMidWordBreak to set m_currentCharacterIsSpace correctly when it rewinded m_current. Not doing so sets previousCharacterIsSpace incorrectly, and can collapse whitespaces that should not be collapsed. BUG= 639781 Review-Url: https://codereview.chromium.org/2334143002 Cr-Commit-Position: refs/heads/master@{#418776} [add] https://crrev.com/d5bcb937b6dd019905eb7bbd9834b1eb26213f2a/third_party/WebKit/LayoutTests/fast/text/break-word-with-floats-expected.html [add] https://crrev.com/d5bcb937b6dd019905eb7bbd9834b1eb26213f2a/third_party/WebKit/LayoutTests/fast/text/break-word-with-floats.html [modify] https://crrev.com/d5bcb937b6dd019905eb7bbd9834b1eb26213f2a/third_party/WebKit/Source/core/layout/line/BreakingContextInlineHeaders.h
,
Sep 16 2016
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by tkonch...@chromium.org
, Aug 24 2016Labels: -Type-Bug -Pri-3 M-54 OS-Linux OS-Mac OS-Windows Pri-1 Type-Bug-Regression
Owner: kojii@chromium.org
Status: Assigned (was: Unconfirmed)