1792608 - text-emphasis-position does not support optional left / right values

Status: RESOLVED FIXED

(Firefox :: Untriaged, defect)

Description

[Anurag Kalia]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:105.0) Gecko/20100101 Firefox/105.0

Steps to reproduce:

Logged here: https://wpt.fyi/results/css/css-text-decor/parsing/text-emphasis-position-computed.html?label=master&label=experimental&aligned&view=subtest&q=text%20emphasis%20position

Actual results:

It does not support text-emphasis-position: over or text-emphasis-position: under

Expected results:

It should have worked just like text-emphasis-position: over right / text-emphasis-position: under right, respectively.

Comment 1

[Anurag Kalia]

Related discussion:
https://bugzilla.mozilla.org/show_bug.cgi?id=1784022#c3 and https://bugzilla.mozilla.org/show_bug.cgi?id=1784022#c5

Comment 2

[Anurag Kalia]

I will work on this as per the discussion in the attached links.

Comment 3

[Anurag Kalia]

Attachment: Bug 1792608 - Make vertical writing mode (left/right) in text-emphasis-position optional r=emilio!

Comment 4

[Anurag Kalia]

Hi emilio! I am back with a new question. I made the changes as you suggested for making left/right in text-emphasis-position optional. I followed the blueprint in your comment and the changes can be seen in the phabricator.

Now, a potential problem I am facing is by reproducing the steps as follows:

1. In the locally built browser with the changes, navigate to https://developer.mozilla.org/en-US/docs/Web/CSS/text-emphasis-position. You will see the interactive example with text-emphasis-position: over right; selected by default and "happy than right" having blue dots over it.
2. Open developer console and highlight the element "happy than right" (span#example-element).

Expected: Value of text-emphasis-position for span#example-element should be over right.

Observed: Value of text-emphasis-position for span#example-element is over.

Further Research

I digged into it and figured out that it is happening on using style.cssText = 'text-emphasis-position: over right;' for it and both over right and over are parsed to the same numeric value.

When I checked the docs for style.cssText, I found the following steps (cssom-1/#dom-cssstyledeclaration-csstext):

Setting the cssText attribute must run these steps:

1. If the computed flag is set, then throw a NoModificationAllowedError exception.
2. Empty the declarations.
3. Parse the given value and, if the return value is not the empty list, insert the items in the list into the declarations, in specified order.
4. Update style attribute for the CSS declaration block.

Basically, we have to "Parse the given value" where over right becomes equivalent to over and when we serialize it back during "Update the style attribute", it turns into over because that it cannot distinguish between the two.

My question is, is it okay? I tried to drill down into the definitions of parsing and could not figure out if this is allowed. Is there any other similar example for optional values for other CSS properties? Moreover, I could not see tests for the same for cssText so that also could not help me here.

Comment 5

[Emilio Cobos Álvarez (:emilio)]

Replied to your question in phabricator, but yes, it's ok (and in fact generally the right thing to do) to serialize to the shortest form. See all the #[css(skip_if...)] usage which also behaves the same way.

Comment 6

[Anurag Kalia]

Hi Emilio! No hurry, but I was worrying if my request on phabricator did not reach you. So, asking for information here too. Thanks! It was about running tests which passed but I am not sure if it was the correct query. So needed confirmation from you for that.

Comment 7

[Emilio Cobos Álvarez (:emilio)]

Yeah, sorry, I replied there. Thanks!

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/186bc4e56d14
Make vertical writing mode (left/right) in text-emphasis-position optional r=emilio

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/81300558527a
Annotate a missing passing test.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6e67171775b3
Simplify other passing test annotations.

Comment 11

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/186bc4e56d14
https://hg.mozilla.org/mozilla-central/rev/81300558527a
https://hg.mozilla.org/mozilla-central/rev/6e67171775b3