From e1d88d3878cec151cdaeebbcc2d836755c5ec3de Mon Sep 17 00:00:00 2001 From: Jakub Pawlowicz Date: Sun, 15 Jan 2017 12:14:26 +0100 Subject: [PATCH] See #290 - disables `font` shorthand overriding. Why: * It's a shorthand property, so not straightforward to implement in shorthand/override compacting; * pending in #254. --- lib/optimizer/level-2/compacting/optimize.js | 5 ----- .../optimizer/level-2/compacting/longhand-overriding-test.js | 5 ----- 2 files changed, 10 deletions(-) diff --git a/lib/optimizer/level-2/compacting/optimize.js b/lib/optimizer/level-2/compacting/optimize.js index e3a383636..8f071d966 100644 --- a/lib/optimizer/level-2/compacting/optimize.js +++ b/lib/optimizer/level-2/compacting/optimize.js @@ -33,11 +33,6 @@ var shorthands = { 'border-top-color': ['border-top', 'border-color', 'border'], 'border-top-style': ['border-top', 'border-style', 'border'], 'border-top-width': ['border-top', 'border-width', 'border'], - 'font-family': ['font'], - 'font-size': ['font'], - 'font-style': ['font'], - 'font-variant': ['font'], - 'font-weight': ['font'] }; function _optimize(properties, mergeAdjacent, aggressiveMerging, validator) { diff --git a/test/optimizer/level-2/compacting/longhand-overriding-test.js b/test/optimizer/level-2/compacting/longhand-overriding-test.js index 495ce2faf..38f050b40 100644 --- a/test/optimizer/level-2/compacting/longhand-overriding-test.js +++ b/test/optimizer/level-2/compacting/longhand-overriding-test.js @@ -125,11 +125,6 @@ vows.describe(optimize) 'border-top-color': ['border-top', 'border-color', 'border'], 'border-top-style': ['border-top', 'border-style', 'border'], 'border-top-width': ['border-top', 'border-width', 'border'], - 'font-family': ['font'], - 'font-size': ['font'], - 'font-style': ['font'], - 'font-variant': ['font'], - 'font-weight': ['font'], 'list-style-image': ['list-style'], 'list-style-position': ['list-style'], 'list-style-type': ['list-style'],