Units dropped from random() $limit

[chriseppstein]

• Proposal

Deprecation

• Tests Tests for random-with-units deprecation sass-spec#1815
• Dart Sass Deprecate math.random() when $limit has units dart-sass#1779
• Docs Document random($limit) behavior whe $limit has units sass-site#664
• Update canonical spec Mark accepted: Random with Units Proposal #3377

Final

• Tests
• Dart Sass
• Docs

---

(Original:)

The unit passed with the limit should be the unit on the returned value. Instead the unit is dropped and never considered nor is an error raised.

Step 1:
Deprecate passing units to random() as the $limit.

Step 2:
Make passing a unit to random() as the $limit an error.

Step 3:
Allow units to be passed in, returning them with the random result.

[jcutrell]

Is the desire to have random() be redefined to use random($limit, $unit), where $unit is optional?

[nex3]

@jcutrell No, the desire is to (ultimately) have math.random(10px) return a number with the unit px.

[Goodwine]

Blocking until feedback arrives from proposal

https://github.com/sass/sass/blob/main/proposal/random-with-units.md

[Goodwine]

Updated main comment with:

Proposal

Deprecation

• Tests
• Dart Sass
• Docs
• Update canonical spec

Final

• Tests
• Dart Sass
• Docs

[nex3]

Now that the proposal has landed, we'll give this a month for public comment since it's a low-controversy breaking change.

[stof]

@nex3 shouldn't this proposal be announced on the blog if you expect comments on it ?

[Goodwine]

Per the contributing guidelines

Public comments are solicited for the feature, usually via a tweet from @SassCSS. If the feature is big enough, a blog post soliciting feedback may also be written

Assuming that because this enhancement/fix is small and uncontroversial it doesn't cross that threshold 🤔

[Goodwine]

Deprecation added, now waiting on Dart 2.0.0 to implement spec