Skip to content

Commit

Permalink
See #290 - disables animation 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 #860.
  • Loading branch information
jakubpawlowicz committed Jan 18, 2017
1 parent 517b4f1 commit 2fe18ee
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 40 deletions.
32 changes: 0 additions & 32 deletions lib/optimizer/level-2/compacting/optimize.js
Expand Up @@ -14,38 +14,6 @@ var OptimizationLevel = require('../../../options/optimization-level').Optimizat
var serializeProperty = require('../../../writer/one-time').property;

var shorthands = {
'animation-delay': ['animation'],
'animation-direction': ['animation'],
'animation-duration': ['animation'],
'animation-fill-mode': ['animation'],
'animation-iteration-count': ['animation'],
'animation-name': ['animation'],
'animation-play-state': ['animation'],
'animation-timing-function': ['animation'],
'-moz-animation-delay': ['-moz-animation'],
'-moz-animation-direction': ['-moz-animation'],
'-moz-animation-duration': ['-moz-animation'],
'-moz-animation-fill-mode': ['-moz-animation'],
'-moz-animation-iteration-count': ['-moz-animation'],
'-moz-animation-name': ['-moz-animation'],
'-moz-animation-play-state': ['-moz-animation'],
'-moz-animation-timing-function': ['-moz-animation'],
'-o-animation-delay': ['-o-animation'],
'-o-animation-direction': ['-o-animation'],
'-o-animation-duration': ['-o-animation'],
'-o-animation-fill-mode': ['-o-animation'],
'-o-animation-iteration-count': ['-o-animation'],
'-o-animation-name': ['-o-animation'],
'-o-animation-play-state': ['-o-animation'],
'-o-animation-timing-function': ['-o-animation'],
'-webkit-animation-delay': ['-webkit-animation'],
'-webkit-animation-direction': ['-webkit-animation'],
'-webkit-animation-duration': ['-webkit-animation'],
'-webkit-animation-fill-mode': ['-webkit-animation'],
'-webkit-animation-iteration-count': ['-webkit-animation'],
'-webkit-animation-name': ['-webkit-animation'],
'-webkit-animation-play-state': ['-webkit-animation'],
'-webkit-animation-timing-function': ['-webkit-animation'],
'border-color': ['border'],
'border-style': ['border'],
'border-width': ['border'],
Expand Down
8 changes: 0 additions & 8 deletions test/optimizer/level-2/compacting/longhand-overriding-test.js
Expand Up @@ -98,14 +98,6 @@ function overrideContext(longhands) {
vows.describe(optimize)
.addBatch(
overrideContext({
'animation-delay': ['animation'],
'animation-direction': ['animation'],
'animation-duration': ['animation'],
'animation-fill-mode': ['animation'],
'animation-iteration-count': ['animation'],
'animation-name': ['animation'],
'animation-play-state': ['animation'],
'animation-timing-function': ['animation'],
'background-attachment': ['background'],
'background-clip': ['background'],
'background-color': ['background'],
Expand Down

0 comments on commit 2fe18ee

Please sign in to comment.