Bug 133150 - REGRESSION transform-origin doesn't take zoom into account correctly for SVG content
Summary: REGRESSION transform-origin doesn't take zoom into account correctly for SVG ...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://jsfiddle.net/cpkSp/
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-21 01:58 PDT by Dirk Schulze
Modified: 2023-12-18 09:42 PST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schulze 2014-05-21 01:58:59 PDT
In the following fiddle, the transform-origin doesn't apply the value of the zoom property (or CMD+/CMD-) correctly. It seems that the origin stays at the same position as without zoom.

Note: it works correctly in Safari 7. I don't know the regression window yet. Must be a problem in 2013 or earlier though.

http://jsfiddle.net/cpkSp/
Comment 1 Dirk Schulze 2014-05-21 02:02:17 PDT
Additional information: This problem just occurs for percentage values. Pixel values are scaled correctly. Which, on the other hand, doesn't work correctly in Safari 7.
Comment 2 Ahmad Saleem 2023-01-07 15:26:25 PST
It is still happening in Safari 16.2 & STP160 using JSFiddle from Comment 0 and it is fixed in following commit by Blink:

Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=174907

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/svg/SVGGraphicsElement.cpp#98

There are fixes in LBSE so I don't know whether it is already fixed in LBSE or not? Appreciate if @Nikolaz can confirm. Thanks!
Comment 3 Ahmad Saleem 2023-05-14 03:32:44 PDT
(In reply to Ahmad Saleem from comment #2)
> It is still happening in Safari 16.2 & STP160 using JSFiddle from Comment 0
> and it is fixed in following commit by Blink:
> 
> Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=174907
> 
> WebKit Source -
> https://searchfox.org/wubkat/source/Source/WebCore/svg/SVGGraphicsElement.
> cpp#98
> 
> There are fixes in LBSE so I don't know whether it is already fixed in LBSE
> or not? Appreciate if @Nikolaz can confirm. Thanks!

I applied this locally and this does fix the attached test case but the attached testcase with the commit does not work.