Bug 266090

Summary: CSS rx and ry properties not supported
Product: WebKit Reporter: James Strater <jbstrater>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: karlcow, sabouhallawa, simon.fraser, webkit-bug-importer, zalan, zimmermann
Priority: P2 Keywords: InRadar
Version: Safari 17   
Hardware: All   
OS: All   
See Also: https://bugs.webkit.org/show_bug.cgi?id=259646
Attachments:
Description Flags
Square corners - ignores rx and ry settings none

James Strater
Reported 2023-12-08 11:51:04 PST
Created attachment 468945 [details] Square corners - ignores rx and ry settings Safari supports all of the other SVG 2.0 geometry properties in CSS, but not rx and ry. This codepen tests for width, height, x, y, cy, cy, r, rx, and ry: https://codepen.io/sidewayss/pen/GRPgqvG caniuse.com says that Safari has supported this for a while now, but as you can see from the attached screenshot (and the codepen) it does not (the rectangle's corners are square). I have filed a PR for the browser-compat-data repo on github here: https://codepen.io/sidewayss/pen/GRPgqvG Obviously if/when these two properties are supported that PR becomes obsolete. This is probably the same root cause as this bug, filed earlier this year, but it has nothing to do with animation. Safari simply doesn't support these two properties in CSS (yet): https://bugs.webkit.org/show_bug.cgi?id=259646
Attachments
Square corners - ignores rx and ry settings (9.95 KB, image/png)
2023-12-08 11:51 PST, James Strater 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Karl Dubost
Comment 1 2023-12-13 22:36:21 PST
I believe Antoine has fixed it in this other bug you are citing. and as you can see in this comment, it is understood that this is not about animations. https://github.com/WebKit/WebKit/pull/21679#pullrequestreview-1778593173
James Strater
Comment 2 2023-12-14 05:26:17 PST
So that other bug sat around for months and got fixed a few days after I posted this bug, but that fix has nothing to do with my posting this bug? My timing is very odd, but glad to see that it has been fixed. When will the changes be released publicly? fyi - The browser-compat-data PR has been merged, so that will need to be reverted once this bug fix is released. That change links to this bug. That PR is here (sorry for the double-paste of the codepen instead of this link in the original comment): https://github.com/mdn/browser-compat-data/pull/21410
Radar WebKit Bug Importer
Comment 3 2023-12-15 11:52:20 PST
<rdar://problem/119731184>
Said Abou-Hallawa
Comment 4 2023-12-18 09:46:54 PST
Bug 259646 fixes this bug as well. *** This bug has been marked as a duplicate of bug 259646 ***
Note You need to log in before you can comment on or make changes to this bug.