-webkit-order doesn't work with css()

[alex.latchford@…]

Hello,

Mocked up a demo: ​http://jsfiddle.net/BsYqC/

Click the third div, you'll see it change colour but not order.

Thanks, Alex

[jasonmerino]

Here is a test case that I think is a little more accurate for testing.

​http://jsfiddle.net/jasonmerino/BsYqC/1/

It looks like internally the vendorPropName() method handles the vendor prefixing well enough, so you don't have to include the "-webkit-" portion, but the value for the CSS order property (-2 in this case) gets "px" added to it because the cssNumber object doesn't contain "order": true.

Being a jQuery core newb I may be missing something, but I feel like this is a pretty simple fix and would be happy to make the change and submit a pull request.

[dmethvin]

Seems like your diagnosis is correct. I don't think we'd want to add the exception inside jQuery until or unless this property is a CSS standard one. It looks like it's part of the flexbox standard. Do you know if it's part of the latest official standard, or is it part of one of the older revs?

[alex.latchford@…]

Replying to dmethvin:

Seems like your diagnosis is correct. I don't think we'd want to add the exception inside jQuery until or unless this property is a CSS standard one. It looks like it's part of the flexbox standard. Do you know if it's part of the latest official standard, or is it part of one of the older revs?

Hi Dave,

As far as I'm aware it is the correct syntax (albeit browser prefixed) that is in the latest revision of the specification for flexbox. ​http://dev.w3.org/csswg/css-flexbox/#order-property ​http://www.w3.org/TR/css3-flexbox/#order-property

Thanks, Alex

[timmywil]

This seems valid and easy to fix.

[jasonmerino]

Hope I'm following correct protocol here. Just submitted a pull request with a fix for this issue: ​https://github.com/jquery/jquery/pull/1300

[Jason Merino]

Fixes #14049: don't append px to CSS order value. Close gh-1300.

Changeset: ec6eb38c64aadd6860c402cc0894c77b82b1a65d