Contrib.social

Add CSS logical properties to properties-order within .stylelintrc.json, and do some cleanup of unneeded properties

Open on Drupal.org →
Created on 12 October 2022, over 1 year ago
Updated 24 June 2024, 4 days ago

Related to 📌 Enforce the use of CSS Logical Properties in core Needs work

Problems

  1. Core's .stylelintrc.json does not include CSS logical properties within the order/properties-order section. This means that our linting does not enforce properties order for those properties.
  2. There's a lot of unneeded properties that have browser prefixes in .stylelintrc.json (examples: -ms-overflow-x, -o-transition-duration).

Solutions

  1. Add CSS logical properties to .stylelintrc.json.
  2. Remove unneeded CSS properties from .stylelintrc.json.
📌 Task
Status

Needs work

Version

11.0 🔥

Component
CSS  →

Last updated 4 days ago

Created by

🇺🇸United States mherchel Gainesville, FL, US

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

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

  • Comment about 1 month ago →
    🇮🇳India akashdab

    - Added css logical properties
    - removed unneeded properties that have browser prefixes
    - did not alter unit-allowed-list and ignoreFiles

  • Comment 29 days ago →
    🇮🇳India ehsann_95

    ahsannazir → made their first commit to this issue’s fork.

  • Merge request !8225Issue #3314921: Add css logical props-order. → (Open) created by Unnamed author
  • Pipeline finished with Failed
    29 days ago
    Total: 172s
    #185653
  • Pipeline finished with Failed
    28 days ago
    Total: 194s
    #186759
  • Pipeline finished with Failed
    28 days ago
    Total: 196s
    #186776
  • Pipeline finished with Failed
    28 days ago
    Total: 1080s
    #186780
  • Pipeline finished with Failed
    28 days ago
    #186786
  • Comment 4 days ago →
    🇮🇳India Gauravvv Delhi, India

    Gauravvvv → made their first commit to this issue’s fork.

  • Status changed to Needs review 4 days ago
  • Comment 4 days ago →
    🇮🇳India Gauravvv Delhi, India
  • Pipeline finished with Failed
    4 days ago
    Total: 523s
    #206413
  • Status changed to Needs work 4 days ago
  • Comment 4 days ago →
    🇺🇸United States smustgrave

    MR has failures

    Issue summary should be updated to standard template and mention about adding a new package, how it was determined which properties to drop/add

  • Pipeline finished with Failed
    3 days ago
    Total: 191s
    #207380
  • Pipeline finished with Failed
    2 days ago
    Total: 180s
    #208370
contrib.social Blog FAQ Discussions
Production build 0.69.0 2024