Problem/Motivation

Running CSSLint on HEAD fails:

$ yarn run lint:css
yarn run v1.21.1
$ stylelint "**/*.css"

themes/seven/css/theme/update-report.css
 20:3  ✖  Expected "margin" to come before "border"   order/properties-order

Proposed resolution

Fix the CSSLint failure

CommentFileSizeAuthor
#2 3118958-2.patch512 byteslauriii
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lauriii created an issue. See original summary.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Status: Active » Reviewed & tested by the community
FileSize
3118958-2.patch512 bytes

I run yarn run lint:css --fix and generated a patch. Since this is such a trivial patch, I feel like I can mark this directly as RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed c5b2a892a9 to 9.0.x and 7ce03c20ec to 8.9.x and 9c19684fae to 8.8.x. Thanks!

I ran yarn run lint:css --fix locally and can confirm the fix. Thanks.

  • alexpott committed c5b2a89 on 9.0.x 
    Issue #3118958 by lauriii: Follow-up to #3102724: CSSLint failure

  • alexpott committed 7ce03c2 on 8.9.x 
    Issue #3118958 by lauriii: Follow-up to #3102724: CSSLint failure

(...

  • alexpott committed 9c19684 on 8.8.x 
    Issue #3118958 by lauriii: Follow-up to #3102724: CSSLint failure

(...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.