New issue
Advanced search Search tips

Issue 697052 link

Starred by 3 users
Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

transform-origin does not apply correctly to group until re-set in DevTools

Reported by atkinch...@gmail.com, Feb 28 2017 
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36

Steps to reproduce the problem:
1. Observe position of red square
2. Open DevTools
3. Select SVG group (<g>) with class 'container'
4. Change the `transform-origin` to any other value, via DevTools
5. Observe position change
6. Change the `transform-origin` back to `center`
7. Observe new position, with same values, is not the same as initial position.

What is the expected behavior?
The position of the red square should respect `transform-origin`.

What went wrong?
The position of the red square does not respect `transform-origin` until it is changed by DevTools.

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 56.0.2924.87  Channel: stable
OS Version: OS X 10.12.1
Flash Version: 

https://jsbin.com/govavayuya/edit?html,output

Comment 1 by f...@opera.com, Feb 28 2017

Labels: -OS-Mac
Status: Available (was: Unconfirmed)
This is caused by the transform (and hence *-origin) being computed before the oBB is computed.

Comment 2 by schenney@chromium.org, Feb 28 2017

Labels: PaintTeamTriaged-20170228 BugSource-User
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 15 2017 

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

commit 6ea24ace180634d03e7407279b27f24a02101d23
Author: Fredrik Söderquist <fs@opera.com>
Date: Fri Dec 15 15:17:52 2017

[PE] SVG <use> 'x' and 'y' are presentation attributes

'x' and 'y' were turned into presentation attributes in
https://codereview.chromium.org/896773002 (r190180), but the property
values were not actually applied.
Apply the property values from ComputedStyle, cleaning up the
surrounding code a bit in the process by moving the "is a <use> element"
check into a method. Also fix the FIXME in SVGUseElement::ToClipPath by
using the computed transform from the LayoutObject, rather than
"manually" recomputing it. This code-path should already require a
LayoutObject for the element in question.

Bug: 400725, 697052
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Iff261e468c4476cfb98efbadd6989ada027e47ab
Reviewed-on: https://chromium-review.googlesource.com/829314
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#524385}
[add] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/LayoutTests/svg/clip-path/clip-path-use-xy-properties-expected.html
[add] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/LayoutTests/svg/clip-path/clip-path-use-xy-properties.html
[add] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/LayoutTests/svg/custom/use-xy-presentation-attributes-expected.html
[add] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/LayoutTests/svg/custom/use-xy-presentation-attributes.html
[add] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/LayoutTests/svg/custom/use-xy-property-mutation-expected.html
[add] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/LayoutTests/svg/custom/use-xy-property-mutation.html
[modify] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/Source/core/layout/svg/LayoutSVGTransformableContainer.cpp
[modify] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/Source/core/layout/svg/LayoutSVGTransformableContainer.h
[modify] https://crrev.com/6ea24ace180634d03e7407279b27f24a02101d23/third_party/WebKit/Source/core/svg/SVGUseElement.cpp
Project Member

Comment 4 by sheriffbot@chromium.org, Dec 17

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by schenney@chromium.org, Dec 17

Status: Fixed (was: Untriaged)

Comment 6 by f...@opera.com, Dec 18

Status: Available (was: Fixed)
This issue is still at large - it however require an explicit 'transform-box: fill-box' to reproduce now. TC attached.
issue697052.html
330 bytes View Download
►

Sign in to add a comment