New issue
Advanced search Search tips

Issue 720205 link

Starred by 4 users
Status: Available
Owner: ----
Cc:
kojii@chromium.org
dgro...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Feature

Blocked on:
issue 636993
issue 735539


Sign in to add a comment

Implement 'line-break: anywhere'

Reported by katie.ba...@gmail.com, May 10 2017 
Chrome Version       : 58.0.3029.96 (64-bit)
Other browsers tested:
    Safari: OK Version 10.1 (11603.1.30.0.34)
    Firefox: FAIL Version 53.0.2 (64-bit) (also fails to break the x's in example)

What steps will reproduce the problem?
(1) put a large amount of "."'s in a span with `word-break: break-all` styling
(2) shrink window to induce wrapping
(included example HTML file)

What is the expected result?
Periods will wrap

What happens instead?
Periods create a horizontal scroll
dots-letters-bugtest.html
990 bytes View Download
Screen Shot 2017-05-09 at 10.31.52 PM.png
158 KB View Download

Comment 1 by kavvaru@chromium.org, May 10 2017

Components: Blink
Labels: -Type-Bug -Pri-3 M-60 has-Bisect OS-Linux OS-Mac OS-Windows Pri-1 Type-Bug-Regression
Owner: kojii@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on windows 7, Ubuntu 14.04 and Mac 10.12.4 using chrome version 58.0.3029.96 and canary 60.0.3094.0.
This is regression issue broken in M41. Please find the bisect information as below
Narrow Bisect::
Good :: 41.0.2244.0  --   (build revision 307434)
Bad:: 40.0.2245.0  --    (build revision 307664)

Change Log::
https://chromium.googlesource.com/chromium/src/+log/10667afd36bc95fdeec67e0bc79c901e6d5bd333..81c64dafcde981ec4f5764ffb4c1cc5912db348f

Blink Roll::
https://chromium.googlesource.com/chromium/blink/+log/71d0c65..dd85771

Possible suspect from Blink cl
https://chromium.googlesource.com/chromium/blink/+/cefc0efc4184a1b979a664e84957ce6efa007e6b

kojii@ could you please look into this issue if it is related to your change,else please help us in finding the appropriate owner for this issue.

Thanks,

Comment 2 by dtapu...@chromium.org, May 10 2017

Components: -Blink Blink>Layout

Comment 3 by kojii@chromium.org, May 11 2017

Labels: -Pri-1 -has-Bisect -Type-Bug-Regression -M-60 Pri-3 Type-Feature
This is by design, see the discussion at
https://github.com/w3c/csswg-drafts/issues/1171

Bisect means that we fixed to match to the spec/Edge/Trident in 41, and W3C CSS WG is discussing adding a feature to break periods in the github issue above.

Comment 4 by kojii@chromium.org, Jun 25 2017

Status: Available (was: Assigned)

Comment 5 by kojii@chromium.org, Jun 25 2017

Blockedon: 735539
Cc: kojii@chromium.org
Owner: ----

Comment 6 by robho...@gmail.com, Aug 13 2017

Status: WontFix (was: Available)

Comment 7 by kojii@chromium.org, Aug 14 2017

Status: Available (was: WontFix)
Summary: Implement 'line-break: anywhere' (was: word-break: break-all not working for periods)
CSS WG resolved to add 'line-break: anywhere', see https://github.com/w3c/csswg-drafts/issues/1171 for details.

It was easy to allow break before periods, but WG wants 'line-break: anywhere' to handle spaces as well, which makes us harder to implement. This would be easy once we switched to LayoutNG, but implmeneting on the current engine is not trivial.

Comment 8 by kojii@chromium.org, Aug 14 2017

Blockedon: 636993

Comment 9 by kojii@chromium.org, Jan 2 2018

Cc: dgro...@chromium.org
 Issue 797773  has been merged into this issue.

Comment 10 by kojii@chromium.org, Jun 28 2018 

 Issue 852313  has been merged into this issue.
►

Sign in to add a comment