Page MenuHomePhabricator

Sass compiler warning regarding BouncingDots mixin
Closed, ResolvedPublic

Description

While transpiling code from sass to css in the Mismatch Finder project, the compiler emits the following warning:

: Using / for division is deprecated and will be removed in Dart Sass 2.0.0.

Recommendation: math.div(1em, 4)

More info and automated migrator: https://sass-lang.com/d/slash-div

    ╷
658 │ $wikit-BouncingDots-margin-left: 1em/4; // not sure the operation is going to work
    │                                  ^^^^^
    ╵
    node_modules/@wmde/wikit-tokens/dist/_variables.scss 658:34  @import
    resources/sass/app.scss 5:9                                  root stylesheet

We believe this is caused by the division symbol included within the BouncingDots token file.

Event Timeline

The underlying issue of style dictionary not being able to handle operations in the token values is tracked here https://phabricator.wikimedia.org/T287389

Removing Mismatch finder tag, as this has already been done, and should be managed for bookkeeping only in the Design System board, as it was a DS issue.

ItamarWMDE moved this task from Needs review to Done on the Wikidata Design System board.

Setting task status to resolved as it seems there is no work left in this task. (Please update the task status once a task is done - thanks a lot!)