Skip to content

Commit

Permalink
Remove multicol-span-all-margin-nested-003.xht.
Browse files Browse the repository at this point in the history
It's unclear what it attempted to test. The file name suggests something
with margins, but there are no margins in the test. Furthermore, the ref
assumes a buggy intrinsic width calculation: There are four words of
column content. The max-content-width should be so that the multicol container
can hold all the four words on the same line. There are four columns too, so
the multicol container actually has to be rather wide.

If column-gap is 1em and each character and non-collapsable whitespace
also is 1em wide, the width of the line of text will be 4+1+4+1+4+1+4em
= 19em. That's the column width we want. But we have 4 columns, so
multiply that by 4 -> 76em. Add the 3 gaps between the 4 columns ->
79em. I.e. 1264px if the font-size is 16px. The ref assumes way less
than that (22em).

Bug: 788337
Change-Id: I076af29bf53914557c40b5e438b5eebfb8439e42
Reviewed-on: https://chromium-review.googlesource.com/811645
Reviewed-by: Ian Kilpatrick <[email protected]>
Reviewed-by: Emil A Eklund <[email protected]>
Commit-Queue: Morten Stenshorne <[email protected]>
Cr-Commit-Position: refs/heads/master@{#522164}
  • Loading branch information
mstensho authored and chromium-wpt-export-bot committed Dec 6, 2017
1 parent 4125a9a commit fee40e5
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 62 deletions.
39 changes: 0 additions & 39 deletions css/css-multicol/multicol-span-all-margin-nested-003.xht

This file was deleted.

23 changes: 0 additions & 23 deletions css/css-multicol/multicol-span-all-margin-nested-3-ref.xht

This file was deleted.

0 comments on commit fee40e5

Please sign in to comment.