New issue
Advanced search Search tips

Issue 811142 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

CSS: Individual Transform Properties can be 2D or 3D

Project Member Reported by ericwilligers@chromium.org, Feb 11 2018

Issue description

translate: 10px 20px indicates a 2D translate
translate: 10px 20px 0px indicates a 3D translate

rotate: 90deg indicates a 2D rotate
rotate: 0 0 1 90deg indicates a 3D rotate

scale: 3 2 indicates a 2D scale
scale: 3 2 1 indicates a 3D scale

We currently represent each of these pairs equivalently, so getComputedStyle gives the same results. We should preserve the distinction, in serialization results and getComputedStyle results.

We need more web platform tests for these properties' getComputedStyle.

We also need to test smooth animation between 2D and 3D versions.

 

Comment 1 by nainar@chromium.org, Feb 12 2018

Labels: Hotlist-Interop
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 5 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f6d74e9b7978f3bce6f8e75fb8af0a3a0b550425

commit f6d74e9b7978f3bce6f8e75fb8af0a3a0b550425
Author: Eric Willigers <ericwilligers@chromium.org>
Date: Mon Mar 05 16:56:11 2018

CSS: Public methods for Scale,Translate transform

The GetType()/PrimitiveType() methods were private for
Scale,Translate TransformOperation, but public on the
base class and for Rotate.

Making the methods public allows clients to determine if
scale and translate transforms are 3D without needing to upcast.

BUG=811142

Change-Id: I5a3b7c69c7df847ca10b6f92331dae2d53343dc9
Reviewed-on: https://chromium-review.googlesource.com/947834
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540848}
[modify] https://crrev.com/f6d74e9b7978f3bce6f8e75fb8af0a3a0b550425/third_party/WebKit/Source/platform/transforms/ScaleTransformOperation.h
[modify] https://crrev.com/f6d74e9b7978f3bce6f8e75fb8af0a3a0b550425/third_party/WebKit/Source/platform/transforms/TranslateTransformOperation.h

Labels: Needs-Feedback
@ericwilligers: Please provide sample testcase /URL to verify this issue from TE end. Hence adding Needs-Feedback label.

Thanks!

Sign in to add a comment