Closed Bug 1607313 Opened 5 years ago Closed 5 years ago

In translate composition treating 'none' as '0px'

Categories

(Core :: DOM: Animation, defect)

73 Branch
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1506746

People

(Reporter: haozhes, Unassigned)

Details

Attachments

(2 files)

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.88 Safari/537.36

Steps to reproduce:

Run a local http server in the same fold with the test html filed (eg. python SimpleHttpSever: python -m SimpleHTTPServer 8000). Then use the firefox nightly version to open the html attached.

Actual results:

See the attachment

Expected results:

According to the spec(https://drafts.csswg.org/css-transforms-2/#individual-transform-serialization), when translate from "none", the computed value should serialize to 'none' instead of "0px". Firefox shows the correct behavior for rotate and scale.

Hi, Hao!

Thanks for your contribution!

Sorry for the late response, I couldn't reproduce this but I'll be adding product and component to keep track of it.

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,

Component: Untriaged → DOM: Core & HTML
Flags: needinfo?(haozhes)
Product: Firefox → Core

(In reply to Seba S. from comment #2)

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

These failures are recorded here:
https://searchfox.org/mozilla-central/source/testing/web-platform/meta/css/css-transforms/animation/translate-composition.html.ini

Status: UNCONFIRMED → NEW
Component: DOM: Core & HTML → Layout
Ever confirmed: true

Fwiw, it's easier to just link to http://w3c-test.org/css/css-transforms/animation/translate-composition.html.

I'm not sure I agree with the test though. Sure, none should serialize as none and not 0px, but the question is more whether interpolating at offset 0 does actually interpolate, right?

And I think we behave consistently with scale and rotate here, looking at the code... Boris, Brian, thoughts?

Component: Layout → DOM: Animation
Flags: needinfo?(brian)
Flags: needinfo?(boris.chiou)

I'm thinking is this related to https://bugzilla.mozilla.org/show_bug.cgi?id=1506746?
The spec issue: https://github.com/w3c/csswg-drafts/issues/3290

We return the values the animation is working on while the animation is going and that includes the endpoints per the definition in web animation

I just updated this tests in css/css-transforms/animation/translate-interpolation.html (wpt.fyi), and it seems it's Blink's bug. In other words, I think Gecko is correct because we have to promote none to 0px at 0%.

Flags: needinfo?(boris.chiou)

Yes, Boris is correct. Gecko is right and Blink has a bug here.

Marking as a duplicate of bug 1506746 since that's where the test was fixed.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(haozhes)
Flags: needinfo?(brian)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: