Multiline values for Patternslib data attributes.

[thet]

Split multiple Patternslib data attribute values into multiple lines. Keep it single-lined, if there is only one value.

[coveralls]

Coverage is 96.878% — thet/patternslib-multiline into master. No base build found for master.

[ale-rt]

I like the idea, but it needs some refinements.

I tested it on the ploneintranet code based and spotted this:

$ git --no-pager diff --staged  imagebank/browser/templates/app_image_bank_navigation.pt
diff --git a/src/ploneintranet/imagebank/browser/templates/app_image_bank_navigation.pt b/src/ploneintranet/imagebank/browser/templates/app_image_bank_navigation.pt
index 4d261a6a6..8d132ed3f 100644
--- a/src/ploneintranet/imagebank/browser/templates/app_image_bank_navigation.pt
+++ b/src/ploneintranet/imagebank/browser/templates/app_image_bank_navigation.pt
@@ -19,9 +19,12 @@
         <a class="pat-inject ${python: 'current' if tab.get('current') else ''}"
            href="${tab/url}"
            data-pat-inject="
-                        history: record; source: #directory-image-bank; target: #directory-image-bank &amp;&amp;
-                        source: #toolbar-functions-area-${view/app_name}::element; target: #toolbar-functions-area-${view/app_name}::element
-                     "
+             history: record;
+             source: #directory-image-bank;
+             target: #directory-image-bank &amp;&amp;
+                        source: #toolbar-functions-area-${view/app_name}::element;
+             target: #toolbar-functions-area-${view/app_name}::element
+           "
            tal:repeat="tab view/tabs"
         >${tab/title}</a>
       </nav>

As you can see there is too much indentation before the second source parameter.

Also I am wondering:

1. if && would deserve a line by its own (I am not really sure about this).
2. how this will react to json encoded attributes (IIRC Patternslib supports that).