#14049 closed bug (fixed)
-webkit-order doesn't work with css()
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | low | Milestone: | 1.10.2/2.0.3 |
Component: | css | Version: | 1.10.1 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Hello,
Mocked up a demo: http://jsfiddle.net/BsYqC/
Click the third div, you'll see it change colour but not order.
Thanks, Alex
Change History (8)
comment:1 Changed 10 years ago by
comment:2 follow-up: 4 Changed 10 years ago by
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?
comment:3 Changed 10 years ago by
Owner: | set to [email protected]… |
---|---|
Status: | new → pending |
comment:4 Changed 10 years ago by
Status: | pending → new |
---|
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
comment:5 Changed 10 years ago by
Component: | unfiled → css |
---|---|
Milestone: | None → 1.11/2.1 |
Priority: | undecided → low |
Status: | new → pending |
This seems valid and easy to fix.
comment:6 Changed 10 years ago by
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
comment:7 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | pending → closed |
Fixes #14049: don't append px to CSS order value. Close gh-1300.
Changeset: ec6eb38c64aadd6860c402cc0894c77b82b1a65d
comment:8 Changed 10 years ago by
Milestone: | 1.11/2.1 → 1.10.2/2.0.3 |
---|
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.