Bug 138846 – REGRESSION(r152313): Inline-block element doesn't wrap properly

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Dave Hyatt]

Inline-block element doesn't wrap properly because of regression regarding empty inline break position testing.

[Dave Hyatt]

Created attachment 241814 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 241814 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=241814&action=review

> Source/WebCore/ChangeLog:8
> +        No new tests (OOPS!).

OOPS!

> LayoutTests/fast/inline-block/inline-block-empty-spans-expected.html:4
> +  <META http-equiv="Content-Type" content="text/html; charset=UTF-8">

No need for this.

> LayoutTests/fast/inline-block/inline-block-empty-spans-expected.html:5
> +  <title>FileInput</title>

Or this.

> LayoutTests/fast/inline-block/inline-block-empty-spans-expected.html:17
> +    table {
> +    	width: 130px;
> +    	border-collapse:collapse;
> +    }
> +    .button {
> +    	display: inline-block;
> +      padding: 5px;
> +      margin: 3px;
> +      background: green;
> +      min-width: 100px;
> +    }

Weird spacing.

> LayoutTests/fast/inline-block/inline-block-empty-spans-expected.html:31
> +			<td>
> +			  <div>
> +			    <span class="button">One</span>
> +                <span class="button">Two</span>
> +                <span class="button">Three</span>
> +			  </div>
> +			</td>
> +		</tr>

Weird whitespace. Tabs?

> LayoutTests/fast/inline-block/inline-block-empty-spans.html:5
> +  <META http-equiv="Content-Type" content="text/html; charset=UTF-8">
> +  <title>FileInput</title>

Boo

> LayoutTests/fast/inline-block/inline-block-empty-spans.html:17
> +    table {
> +    	width: 130px;
> +    	border-collapse:collapse;
> +    }
> +    .button {
> +    	display: inline-block;
> +      padding: 5px;
> +      margin: 3px;
> +      background: green;
> +      min-width: 100px;
> +    }

Weird spacing.

> LayoutTests/fast/inline-block/inline-block-empty-spans.html:29
> +		<tr>
> +			<td>
> +			  <div>
> +			    <span class="button">One</span><span class="button">Two</span><span class="button">Three</span><span> </span>
> +			  </div>
> +			</td>
> +		</tr>

Yeah.

[WebKit Commit Bot]

Attachment 241814 [details] did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:8:  You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 5 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Dave Hyatt]

Fixed in r176287.