Skip to content

Latest commit

 

History

History
209 lines (177 loc) · 3.69 KB

README.md

File metadata and controls

209 lines (177 loc) · 3.69 KB
Raw

declaration-block-no-redundant-longhand-properties

Disallow redundant longhand properties within declaration-block.

  a {
    padding-top: 1px;
    padding-right: 2px;
    padding-bottom: 3px;
    padding-left: 4px; }
/** ↑
 *  These longhand properties */

The longhand properties in the example above can be more concisely written as:

a {
  padding: 1px 2px 3px 4px;
}

This rule will only complain if you've used the longhand equivalent of all the properties that the shorthand will set.

This rule complains when the following shorthand properties can be used:

  • margin
  • padding
  • background
  • font
  • border
  • border-top
  • border-bottom
  • border-left
  • border-right
  • border-width
  • border-style
  • border-color
  • list-style
  • border-radius
  • transition
  • animation
  • border-block-end
  • border-block-start
  • border-image
  • border-inline-end
  • border-inline-start
  • column-rule
  • columns
  • flex
  • flex-flow
  • grid
  • grid-area
  • grid-column
  • grid-gap
  • grid-row
  • grid-template
  • outline
  • text-decoration
  • text-emphasis
  • mask

Please note that properties are considered to be redundant if they may be written shorthand according to the specification, regardless of the behavior of any individual browser. For example, due to Internet Explorer's implementation of Flexbox, it may not be possible to use the shorthand property flex, but the longhand form is still considered a problem.

Flexbox-related properties can be ignored using ignoreShorthands: ["/flex/"] (see below).

Options

true

The following patterns are considered problems:

a {
  margin-top: 1px;
  margin-right: 2px;
  margin-bottom: 3px;
  margin-left: 4px;
}
a {
  font-style: italic;
  font-variant: normal;
  font-weight: bold;
  font-stretch: normal;
  font-size: 14px;
  line-height: 1.2;
  font-family: serif;
}
a {
  -webkit-transition-property: top;
  -webkit-transition-duration: 2s;
  -webkit-transition-timing-function: ease;
  -webkit-transition-delay: 0.5s;
}

The following patterns are not considered problems:

a {
  margin: 1px 2px 3px 4px;
}
a {
  font: italic normal bold normal 14px/1.2 serif;
}
a {
  -webkit-transition: top 2s ease 0.5s;
}
a {
  margin-top: 1px;
  margin-right: 2px;
}
a {
  margin-top: 1px;
  margin-right: 2px;
  margin-bottom: 3px;
}

You should use a longhand property to have only the specific value inherited. So, the following patterns are also not considered problems:

a {
  margin-top: 1px;
  margin-right: 2px;
  margin-bottom: 3px;
  margin-left: inherit;
}

Optional secondary options

ignoreShorthands: ["/regex/", /regex/, "string"]

Given:

["padding", "/border/"]

The following patterns are not considered problems:

a {
  padding-top: 20px;
  padding-right: 10px;
  padding-bottom: 30px;
  padding-left: 10px;
}
a {
  border-top-width: 1px;
  border-bottom-width: 1px;
  border-left-width: 1px;
  border-right-width: 1px;
}
a {
  border-top-width: 1px;
  border-bottom-width: 1px;
  border-left-width: 1px;
  border-right-width: 1px;
}
a {
  border-top-color: green;
  border-top-style: double;
  border-top-width: 7px;
}