1606305 - Wrong intrinsic width for pre element with font-size: 0.

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Enrique]

Attachment: bugExample.tar.gz

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:71.0) Gecko/20100101 Firefox/71.0

Steps to reproduce:

Rendering a -something like weird/non-standard- layout (See attached example)

Actual results:

The calculated width for pre elements is not correct.

The widh is calculated properly "most-of-the-times", but depending on the internal content of the pre element it suddenly "explodes" in width. This looks to happen in latest versions of firefox 69+

Expected results:

The calculated width should adjust properly.
See attached example with input html and rendered output.
Same content is rendered as expected on Chrome.

Comment 1

[Enrique]

It looks to be related to the script making the font-size equal to ZERO. By just increasing to 0.0001 it renders properly.

Comment 2

[Seba S.]

Hi, Enrique!

Thanks for your contribution!

I couldn't reproduce the described behavior in:

Ubuntu 18.04 LT
Firefox 71.0
Nightly 73.0a1 (2020-01-03) (64-bit)

Will add product and component to keep on track, it seems like an easy issue to reproduce.

Please let us know if this issue is reproduced in the latest Nightly edition. You can download it from here: https://nightly.mozilla.org/

If you still have the issue please create a new profile, you have the steps here: https://support.mozilla.org/en-US/kb/profile-manager-create-and-remove-firefox-profiles?redirectlocale=en-US&redirectslug=Managing-profiles#w_starting-the-profile-manager
support.mozilla.orgsupport.mozilla.org

Once you have all this information, please let us know so we can continue investigating.

Regards,

Comment 3

[Enrique]

I tried with Nightly. The bug persists. Digging further, it looks to be related to the parsing of the CSS.

"font-size: 0em;" fails.

"font-size: 0.em;" works (Adding a dot after the 0)

"font-size: 0.0em;" fails. (Adding a second 0 after the dot)

Comment 4

[Seba S.]

Hi, Enrique!

Thanks for clarifying this, I am going to add product and component to this issue in order to move on. If you have some other details or specifications you can dd it on a thread below.

Regards,

Comment 5

[Emilio Cobos Álvarez (:emilio)]

Attachment: Test-case from reporter.

Comment 6

[Emilio Cobos Álvarez (:emilio)]

(In reply to Enrique from comment #3)
Can you elaborate? I don't see any different style for the <pre> element involved or such due to different font-size parsing.

Comment 7

[Enrique]

Not sure how to elaborate. Just another online example. This time it works when font-size is greater or equal to 0.0006rem:

http://www.oficina24x7.com/Blockchain/ethereum_map.html

Edit style rule using rem units:

[zoom] { font-size:0.0001rem; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0002rem; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0003rem; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0004rem; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0005rem; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0006rem; } / OK! */
[zoom] { font-size:0.0007rem; } / OK! */

---

Edit style rule using em units:
[zoom] { font-size:0.00em; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.001em; } / OK! */
[zoom] { font-size:0.000em; } / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0001em; / ERROR! . The element width "explodes". */
[zoom] { font-size:0.0006em; } / OK! */

I've not been able to find a common error-pattern, but it looks "something somewhere" has problems to handle zero or "close-to-zero" values.

Comment 8

[Emilio Cobos Álvarez (:emilio)]

Ok, so I see what happens, mostly.

The <pre> element is floating, so it gets sized at its intrinsic size. It seems that when we're computing intrinsic sizes, we ignore the font-size: 0 content altogether, and thus don't break...

Comment 9

[Emilio Cobos Álvarez (:emilio)]

Attachment: Reduced test-case.

The border should be just around the text

Comment 10

[Jonathan Kew [:jfkthame]]

Ah, yes... we short-circuit textrun initialization when the font size is zero (or close enough to zero that it rounds down). But unfortunately we then fail to record the locations of important characters like <newline> in the textrun, and that's what leads to this issue.

<tab> is similarly affected; if you try an example like

data:text/html,<pre style="font-size:0px;-moz-tab-size:100px"><span style="font-size:12px">foo</span>&%239;<span style="font-size:12px">bar</span>

we fail to properly tab the word "bar" over to start at 100px from the left margin.

Looks like a straightforward fix...

Comment 11

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1606305 - Don't forget to record the location of newline and tab characters when creating a textrun for a zero-size font. r=emilio

Comment 12

[Jonathan Kew [:jfkthame]]

This fixes the bug reported here, as well as the related <tab> example; we should also add a couple of simple reftests (but I don't have time tonight).

Comment 13

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1606305 - Add WPT reftests for handling of <newline> and <tab> when font-size is zero. r=emilio

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e42bc01188e0
Don't forget to record the location of newline and tab characters when creating a textrun for a zero-size font. r=emilio
https://hg.mozilla.org/integration/autoland/rev/a4caf68a8938
Add WPT reftests for handling of <newline> and <tab> when font-size is zero. r=emilio

Comment 15

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/21145 for changes under testing/web-platform/tests

Comment 16

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 17

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/e42bc01188e0
https://hg.mozilla.org/mozilla-central/rev/a4caf68a8938

Comment 18

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot