RESOLVED FIXED 209872
[css-flexbox] flex-grow property should be animatable 
https://bugs.webkit.org/show_bug.cgi?id=209872
Summary [css-flexbox] flex-grow property should be animatable
Carlos Alberto Lopez Perez
Reported 2020-04-01 13:07:41 PDT
The flex-grow property must be animatable https://drafts.csswg.org/css-flexbox/#flex-grow-property Here its a simple test case: https://codepen.io/cl0p3z/pen/yLNrNyN Works on Chrome and Firefox. Fails on WebKit. This its tested by WPT css/css-flexbox/animation/flex-grow-interpolation.html
Attachments
Patch (32.06 KB, patch)
2020-10-14 05:49 PDT, Sergio Villar Senin 		graouts: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch for landing (39.74 KB, patch)
2020-10-14 07:21 PDT, Sergio Villar Senin 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sergio Villar Senin
Comment 1 2020-10-14 05:49:01 PDT
Created attachment 411314 [details] Patch
Antoine Quint
Comment 2 2020-10-14 05:58:20 PDT
Comment on attachment 411314 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=411314&action=review r+ assuming EWS bots are happy as well, thanks for adding this. > Source/WebCore/rendering/style/RenderStyle.h:1104 > + void setFlexGrow(float f) { float v = std::max<float>(f, 0.f); SET_NESTED_VAR(m_rareNonInheritedData, flexibleBox, flexGrow, v); } Perhaps spell out what `v` stands for here. Since you're clamping, perhaps call that out in the variable name.
Sergio Villar Senin
Comment 3 2020-10-14 07:21:06 PDT
Created attachment 411321 [details] Patch for landing
Sergio Villar Senin
Comment 4 2020-10-15 00:55:04 PDT
Committed r268516: <https://trac.webkit.org/changeset/268516>
Radar WebKit Bug Importer
Comment 5 2020-10-15 00:56:17 PDT
<rdar://problem/70325825>
Note You need to log in before you can comment on or make changes to this bug.