simplified and covered array > array case

Author: rueckstiess

src/document-list/document-list-item.jade

@@ -1,44 +1,35 @@
-mixin jsprimitive(value, _type)
-  if _type === 'Date'
-    - value = moment(value).format('LLL')
-  .document-property-value(title=value)= value
+mixin jsvalue(value, _type)
+  if _type === 'Array'
+    - value = 'Array[' + value.length + ']'
+    .document-property-type-label= value
+  else if _type === 'Object'
+    - value = 'Object'
+    .document-property-type-label= value
+  else
+    if _type === 'Date'
+      - value = moment(value).format('LLL')
+    .document-property-value(title=value)= value
 
 mixin jsarray(items)
   ol.document-property-body
     each item in items
       +jsproperty(null, item)
-
+      
 mixin jsproperty(key, value)
   - _type = getType(value)
-  - is_expandable = _type === 'Object' || _type === 'Array'
-  if !is_expandable
-    if key
-      li.document-property
-        .document-property-key= key
-        +jsprimitive(value, _type)
-    else
-      li.document-property-array-item
-        +jsprimitive(value, _type)
-  else if _type === 'Array'
-    li.document-property-array
-      .document-property-header
+  li(class='document-property #{_type.toLowerCase()}')
+    .document-property-header
+      if _type === 'Array' || _type === 'Object'
         .caret
-        if key
-          .document-property-key= key
-          | :
-          .document-property-type-label Array[#{value.length}]
+      if key
+        .document-property-key= key
+        |:
+      +jsvalue(value, _type)
+    if _type === 'Array'
       +jsarray(value)
-  else
-    li.document-property-object
-      .document-property-header
-        .caret
-        if key
-          .document-property-key= key
-          | :
-        .document-property-type-label Object
+    if _type === 'Object'
       +jsobject(value)
 
-
 mixin jsobject(obj)
   ol.document-property-body
     for value, key in obj

src/document-list/index.less

@@ -44,7 +44,7 @@ ol.document-list {
         }
       }
 
-      li.document-property-array, li.document-property-object {
+      li.document-property.array, li.document-property.object {
         display: block;
         padding-top: 4px;
 
@@ -70,13 +70,13 @@ ol.document-list {
             display: inline-block;
           }
         }
-        ol.document-property-body {
+        ol.document-property.body {
           margin-left: 5px;
           padding-left: -5px;
           border-left: 1px dotted @gray5;
           display: none;
         }
-        ol.document-property-body  > a ~ ol {
+        ol.document-property.body  > a ~ ol {
           padding-left: 12px;
           display: none;
         }
@@ -86,7 +86,7 @@ ol.document-list {
               .caret-down;
             }
           }
-          > ol.document-property-body {
+          > ol.document-property.body {
             display: block;
           }
         }