Skip to content

Commit

Permalink
See #290 - disables font shorthand overriding.
Browse files Browse the repository at this point in the history
Why:

* It's a shorthand property, so not straightforward to implement
  in shorthand/override compacting;
* pending in #254.
  • Loading branch information
jakubpawlowicz committed Jan 18, 2017
1 parent 50a0278 commit e1d88d3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
5 changes: 0 additions & 5 deletions lib/optimizer/level-2/compacting/optimize.js
Expand Up @@ -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) {
Expand Down
5 changes: 0 additions & 5 deletions test/optimizer/level-2/compacting/longhand-overriding-test.js
Expand Up @@ -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'],
Expand Down

0 comments on commit e1d88d3

Please sign in to comment.