See #290 - allows `height` overriding.

Why:

* Another step to move away from aggressive merging.

diff --git a/lib/properties/compactable.js b/lib/properties/compactable.js
index ca0657d..ec5549d 100644 (file)
--- a/lib/properties/compactable.js
+++ b/lib/properties/compactable.js
@@ -125,6 +125,11 @@ var compactable = {
     shortestValue: '0',
     shorthand: true
   },
+  'height': {
+    canOverride: canOverride.unit,
+    defaultValue: 'auto',
+    shortestValue: '0'
+  },
   'list-style': {
     components: [
       'list-style-type',

diff --git a/test/optimizer/advanced-test.js b/test/optimizer/advanced-test.js
index d89a81c..f934131 100644 (file)
--- a/test/optimizer/advanced-test.js
+++ b/test/optimizer/advanced-test.js
@@ -88,6 +88,10 @@ vows.describe('advanced optimizer')
   )
   .addBatch(
     optimizerContext('unit compacting', {
+      'height': [
+        'div{height:1rem;height:16px}',
+        'div{height:16px}'
+      ],
       'width': [
         'div{width:1rem;width:16px}',
         'div{width:16px}'