Update stylelint rule shorthand-property-no-redundant-values to be consistent with Drupal's CSS standards

Created on 4 April 2017, over 7 years ago
Updated 30 January 2023, almost 2 years ago

Problem/Motivation

Follow-up to #2865971: Use stylelint as opposed to csslint in core β†’ . Configure the shorthand-property-no-redundant-values to be consistent with https://www.drupal.org/docs/develop/standards/css/css-coding-standards β†’

Proposed resolution

Brief instructions on running stylelint - you'll need npm...

All the commands below take place in DRUPAL_ROOT/core
To install stylelint

npm install

This will install Drupal 8's npm dependencies of which stylelint is one.

To run it on all core css files. Apply this issue's patch and do the following command from DRUPAL_ROOT/core

npm run lint:css

Remaining tasks

User interface changes

None

API changes

None

πŸ“Œ Task
Status

Needs work

Version

10.1 ✨

Component
CSSΒ  β†’

Last updated 5 days ago

Created by

πŸ‡¬πŸ‡§United Kingdom alexpott πŸ‡ͺπŸ‡ΊπŸŒ

Live updates comments and jobs are added and updated live.
  • CSS

    It involves the content or handling of Cascading Style Sheets.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • The Needs Review Queue Bot β†’ tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

    Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

    Consult the Drupal Contributor Guide β†’ to find step-by-step guides for working with issues.

Production build 0.71.5 2024