16374 2012-03-14 17:30:39 +0000 Allow transform function list with out explicit separation 2012-03-17 01:17:29 +0000 1 1 1 Unclassified CSS Transforms unspecified PC All RESOLVED FIXED P2 normal --- 1 dschulze dschulze ayg cmarrin dino dschulze eoconnor smfr public-css-bugzilla oldest_to_newest 65526 0 dschulze 2012-03-14 17:30:39 +0000 The CSS3 Transform specification might allow transform lists, where the single transform functions are not separated at all: transform: translate(100px,100px)rotate(45deg); The same for the presentation attribute: transform="translate(100,100)rotate(45)" Neither CSS3 Transforms nor SVG Transform allow it at the moment. Nevertheless, it works on the browsers: Safari, Chrome, Firefox and Opera. Currently untested is IE, I'll test it on IE today and give feedback. 65527 1 1097 dschulze 2012-03-14 17:32:44 +0000 Created attachment 1097 CSS Transform test 65528 2 1098 dschulze 2012-03-14 17:34:43 +0000 Created attachment 1098 SVG Transform test 65529 3 dschulze 2012-03-14 17:37:43 +0000 I tested IE 10 preview, and IE 10 supports it as well. 65530 4 smfr 2012-03-14 17:40:17 +0000 Is there precendent in any other css property that takes a bare list of functions? 65533 5 dschulze 2012-03-14 17:43:59 +0000 (In reply to comment #4) > Is there precendent in any other css property that takes a bare list of > functions? I just know of CSS Filters that have a list of functions. I'll check what WebKit does there. 65535 6 dino 2012-03-14 17:46:03 +0000 It's cool that everyone supports this, but I fear that adding so much flexibility now removes any flexibility for us in the future. Imagine that we need to add some characters after the parentheses for some reason (4d transforms!!) or remove parentheses - we won't be able to do it if we explicitly allow this. On the other hand, codifying existing behaviour is generally a good thing. 65537 7 dschulze 2012-03-14 17:52:43 +0000 (In reply to comment #4) > Is there precendent in any other css property that takes a bare list of > functions? Just checked -webkit-filter: drop-shadow(10px 10px 10px blue)sepia(); and it works as well. (In reply to comment #6) > It's cool that everyone supports this, but I fear that adding so much > flexibility now removes any flexibility for us in the future. Imagine that we > need to add some characters after the parentheses for some reason (4d > transforms!!) or remove parentheses - we won't be able to do it if we > explicitly allow this. > > On the other hand, codifying existing behaviour is generally a good thing. Wow! 4D? Don't we support it already with transitions and animations? :D But I understand the concerns. Nevertheless, we would have to deal with it if we need the flexibility as well. Even if unspecified, we could potentially break existing content if we need the separation later. 65541 8 ayg 2012-03-14 18:05:55 +0000 As I'm reading CSS 3 Values and Units, optional whitespace is always allowed between tokens in values: """ Component values are specified in terms of tokens, as described in Chapter 4 of [CSS21]. As the grammar allows spaces between tokens in the components of the value production, spaces may appear between tokens in property values. """ http://dev.w3.org/csswg/css3-values/#component-whitespace So I suggest we just remove the sentence "The individual transform functions are separated by whitespace". It's already covered by other specs. I already have a test for this in 3d-transforms.html, which tests that "tranSlatex( 16px )rotatez(-90deg) rotate(100grad)\ttranslate3d(-12pt, 0, 0.0em)" is parsed as: "translateX(16px) rotateZ(-90deg) rotate(100grad) translate3d(-12pt, 0px, 0em)" 65546 9 1099 dschulze 2012-03-14 18:32:19 +0000 Created attachment 1099 Suggestion What about the attached diff? Remove the sentence like Aryeh described and change SVG part like that: If the transform list consists of more than one transform function, these functions may be separated by optional whitespace, an optional comma (‘<code class="css">,</code>’) followed by optional whitespace. The transform list can have optional whitespace characters before and after the list. 65549 10 ayg 2012-03-14 18:34:44 +0000 The CSS part of that change looks good to me (I didn't review the SVG part). 65630 11 dschulze 2012-03-15 21:44:19 +0000 Dean, do you still have concerns that we need to discuss? 65689 12 dschulze 2012-03-17 01:17:29 +0000 Committed change today. 1097 2012-03-14 17:32:44 +0000 2012-03-14 17:32:44 +0000 CSS Transform test trans.html text/html 458 dschulze PCFET0NUWVBFIGh0bWw+CjxodG1sIGxhbmc9ImVuIj4KPGhlYWQ+CiAgPG1ldGEgY2hhcnNldD0i dXRmLTgiPgogIDx0aXRsZT48L3RpdGxlPgogIDxzY3JpcHQ+CgogIDwvc2NyaXB0PgogIDxzdHls ZT4KZGl2IHsKICBiYWNrZ3JvdW5kLWNvbG9yOiBncmVlbjsKICB3aWR0aDogMTAwcHg7CiAgaGVp Z2h0OiAxMDBweDsKICAtd2Via2l0LXRyYW5zZm9ybTogdHJhbnNsYXRlKDEwMHB4LDEwMHB4KXJv dGF0ZSg0NWRlZyk7CiAgLW1vei10cmFuc2Zvcm06IHRyYW5zbGF0ZSgxMDBweCwxMDBweClyb3Rh dGUoNDVkZWcpOwogIC1vLXRyYW5zZm9ybTogdHJhbnNsYXRlKDEwMHB4LDEwMHB4KXJvdGF0ZSg0 NWRlZyk7CiAgLW1zLXRyYW5zZm9ybTogdHJhbnNsYXRlKDEwMHB4LDEwMHB4KXJvdGF0ZSg0NWRl Zyk7Cn0KICA8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5Pgo8ZGl2PjwvZGl2Pgo8L2JvZHk+CjwvaHRt bD4= 1098 2012-03-14 17:34:43 +0000 2012-03-14 17:34:43 +0000 SVG Transform test trans.svg image/svg+xml 177 dschulze PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRw Oi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIj4KPHJlY3Qgd2lkdGg9IjEwMCIgaGVpZ2h0PSIxMDAi IGZpbGw9ImdyZWVuIiB0cmFuc2Zvcm09InRyYW5zbGF0ZSgxMDAsMTAwKXJvdGF0ZSg0NSkiLz4K PC9zdmc+ 1099 2012-03-14 18:32:19 +0000 2012-03-14 18:32:19 +0000 Suggestion whitespace.patch text/plain 2962 dschulze ZGlmZiAtLWdpdCBhL2NzczMtdHJhbnNmb3Jtcy9UcmFuc2Zvcm1zLnNyYy5odG1sIGIvY3NzMy10 cmFuc2Zvcm1zL1RyYW5zZm9ybXMuc3JjLmh0bWwKLS0tIGEvY3NzMy10cmFuc2Zvcm1zL1RyYW5z Zm9ybXMuc3JjLmh0bWwKKysrIGIvY3NzMy10cmFuc2Zvcm1zL1RyYW5zZm9ybXMuc3JjLmh0bWwK QEAgLTc3NywxOSArNzc3LDE4IEBAIGRpdiB7CiAgICAgICAgICAgICAgICAgdGhlIGF0dHJpYnV0 ZSBvbmx5LgogICAgICAgICAgICAgICA8L3A+CiAgICAgICAgICAgICAgIAogICAgICAgICAgICAg ICA8aDQgaWQ9InN2Zy10cmFuc2Zvcm0tbGlzdCI+VHJhbnNmb3JtIExpc3Q8L2g0PgogICAgICAg ICAgICAgICAKICAgICAgICAgICAgICAgPHA+CiAgICAgICAgICAgICAgICAgVGhlIHZhbHVlIGZv ciB0aGUgJzxjb2RlIGNsYXNzPSJwcm9wZXJ0eSI+dHJhbnNmb3JtPC9jb2RlPicgYXR0cmlidXRl IGNvbnNpc3RzIG9mIGEgdHJhbnNmb3JtIGxpc3Qgd2l0aCB6ZXJvIG9yIG1vcmUgdHJhbnNmb3Jt IGZ1bmN0aW9ucwogICAgICAgICAgICAgICAgIHVzaW5nIDxhIGhyZWY9IiNzdmctZnVuY3Rpb25h bC1ub3RhdGlvbiI+ZnVuY3Rpb25hbCBub3RhdGlvbjwvYT4uIElmIHRoZSB0cmFuc2Zvcm0gbGlz dCBjb25zaXN0cyBvZiBtb3JlIHRoYW4gb25lIHRyYW5zZm9ybSBmdW5jdGlvbiwKLSAgICAgICAg ICAgICAgICAgIHRoZXNlIGZ1bmN0aW9ucyBhcmUgc2VwYXJhdGVkIGJ5IGVpdGhlciBhIGNvbW1h ICjigJg8Y29kZSBjbGFzcz0iY3NzIj4sPC9jb2RlPuKAmSkgd2l0aCBvcHRpb25hbCB3aGl0ZXNw YWNlIGNoYXJhY3RlcnMKLSAgICAgICAgICAgICAgICBiZWZvcmUgYW5kIGFmdGVyIHRoZSBjb21t YSwgb3IgYnkgb25lIG9yIG1vcmUgd2hpdGVzcGFjZSBjaGFyYWN0ZXJzLiBUaGUgdHJhbnNmb3Jt IGxpc3QgY2FuIGhhdmUKLSAgICAgICAgICAgICAgICBvcHRpb25hbCB3aGl0ZXNwYWNlIGNoYXJh Y3RlcnMgYmVmb3JlIGFuZCBhZnRlciB0aGUgbGlzdC4KKyAgICAgICAgICAgICAgICB0aGVzZSBm dW5jdGlvbnMgbWF5IGJlIHNlcGFyYXRlZCBieSBvcHRpb25hbCB3aGl0ZXNwYWNlLCBhbiBvcHRp b25hbCBjb21tYSAo4oCYPGNvZGUgY2xhc3M9ImNzcyI+LDwvY29kZT7igJkpIGZvbGxvd2VkIGJ5 IG9wdGlvbmFsIHdoaXRlc3BhY2UuCisgICAgICAgICAgICAgICAgVGhlIHRyYW5zZm9ybSBsaXN0 IGNhbiBoYXZlIG9wdGlvbmFsIHdoaXRlc3BhY2UgY2hhcmFjdGVycyBiZWZvcmUgYW5kIGFmdGVy IHRoZSBsaXN0LgogICAgICAgICAgICAgICA8L3A+CiAgICAgICAgICAgICAgIAogICAgICAgICAg ICAgICA8aDQgaWQ9InN2Zy1mdW5jdGlvbmFsLW5vdGF0aW9uIj5GdW5jdGlvbmFsIE5vdGF0aW9u PC9oND4KICAgICAgICAgICAgICAgCiAgICAgICAgICAgICAgIDxwPgogICAgICAgICAgICAgICAg IFRoZSBzeW50YXggc3RhcnRzIHdpdGggdGhlIG5hbWUgb2YgdGhlIGZ1bmN0aW9uIGZvbGxvd2Vk IGJ5IG9wdGlvbmFsIHdoaXRlc3BhY2UgY2hhcmFjdGVycyBmb2xsb3dlZCBieSBhIGxlZnQKICAg ICAgICAgICAgICAgICBwYXJlbnRoZXNpcyBmb2xsb3dlZCBieSBvcHRpb25hbCB3aGl0ZXNwYWNl IGZvbGxvd2VkIGJ5IHRoZSBhcmd1bWVudChzKSB0byB0aGUgbm90YXRpb24gZm9sbG93ZWQgYnkK ICAgICAgICAgICAgICAgICBvcHRpb25hbCB3aGl0ZXNwYWNlIGZvbGxvd2VkIGJ5IGEgcmlnaHQg cGFyZW50aGVzaXMuIElmIGEgZnVuY3Rpb24gdGFrZXMgbW9yZSB0aGFuIG9uZSBhcmd1bWVudCwg dGhlCkBAIC0xNTg0LDE3ICsxNTgzLDE2IEBAIGRpdiB7CiAgICAgICAgICAgICAgIDwhLS0gPT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PSAtLT4KIAogICAgICAg ICAgICAgICA8aDIgaWQ9InRyYW5zZm9ybS1mdW5jdGlvbnMiPgogICAgICAgICAgICAgICAgIFRo ZSBUcmFuc2Zvcm1hdGlvbiBGdW5jdGlvbnMKICAgICAgICAgICAgICAgPC9oMj4KICAgICAgICAg ICAgICAgPHA+CiAgICAgICAgICAgICAgICAgVGhlIHZhbHVlIG9mIHRoZSA8Y29kZSBjbGFzcz0i cHJvcGVydHkiPnRyYW5zZm9ybTwvY29kZT4gcHJvcGVydHkgaXMgYQogICAgICAgICAgICAgICAg IGxpc3Qgb2YgPHZhcj4mbHQ7dHJhbnNmb3JtLWZ1bmN0aW9ucyZndDs8L3Zhcj4gYXBwbGllZCBp biB0aGUgb3JkZXIgcHJvdmlkZWQuIFRoZQotICAgICAgICAgICAgICAgIGluZGl2aWR1YWwgdHJh bnNmb3JtIGZ1bmN0aW9ucyBhcmUgc2VwYXJhdGVkIGJ5IHdoaXRlc3BhY2UuIFRoZQogICAgICAg ICAgICAgICAgIHNldCBvZiBhbGxvd2VkIHRyYW5zZm9ybSBmdW5jdGlvbnMgaXMgZ2l2ZW4gYmVs b3cuIEluIHRoaXMgbGlzdCB0aGUKICAgICAgICAgICAgICAgICB0eXBlIDx2YXI+Jmx0O3RyYW5z bGF0aW9uLXZhbHVlJmd0OzwvdmFyPiBpcyBkZWZpbmVkIGFzIGEgPHZhcj4mbHQ7bGVuZ3RoJmd0 OzwvdmFyPiBvcgogICAgICAgICAgICAgICAgIDx2YXI+Jmx0O3BlcmNlbnRhZ2UmZ3Q7PC92YXI+ IHZhbHVlLCBhbmQgdGhlIDx2YXI+Jmx0O2FuZ2xlJmd0OzwvdmFyPiB0eXBlIGlzIGRlZmluZWQg YnkgPGEKICAgICAgICAgICAgICAgICBocmVmPSJodHRwOi8vd3d3LnczLm9yZy9UUi9jc3MzLXZh bHVlcy8iPkNTUyBWYWx1ZXMgYW5kIFVuaXRzIE1vZHVsZS48L2E+CiAgICAgICAgICAgICAgICAg V2hlcmV2ZXIgPHZhcj4mbHQ7YW5nbGUmZ3Q7PC92YXI+IGlzIHVzZWQgaW4gdGhpcyBzcGVjaWZp Y2F0aW9uLCBhIDx2YXI+Jmx0O251bWJlciZndDs8L3Zhcj4gdGhhdCBpcyBlcXVhbCB0bwogICAg ICAgICAgICAgICAgIHplcm8gaXMgYWxzbyBhbGxvd2VkLCB3aGljaCBpcyB0cmVhdGVkIHRoZSBz YW1lIGFzIGFuIGFuZ2xlIG9mIHplcm8gZGVncmVlcy4KICAgICAgICAgICAgICAgPC9wPgogCg==