Skip to content

Commit 181787f

Browse files
nex3nex3
authored
[Module System] Remove adjust-hue() from sass:color (sass#2746)
Closes sass#2740
1 parent 4df504f commit 181787f

File tree

2 files changed

+35
-30
lines changed

2 files changed

+35
-30
lines changed

‎accepted/module-system.changes.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
## Draft 8
2+
3+
* Remove `adjust-hue()` from the `sass:color` module. `color.adjust($hue: ...)`
4+
should be used instead.
5+
16
## Draft 7
27

38
* Change the syntax for namespaced variables from `$namespace.name` to

‎accepted/module-system.md

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -1536,31 +1536,31 @@ The built-in functions will be organized as follows:
15361536
| `hue` | | sass:color | | `comparable` | `compatible` | sass:math |
15371537
| `saturation` | | sass:color | | | | |
15381538
| `lightness` | | sass:color | | `length` | | sass:list |
1539-
| `adjust-hue` | | sass:color | | `nth` | | sass:list |
1540-
| `complement` | | sass:color | | `set-nth` | | sass:list |
1541-
| `invert` | | sass:color | | `join` | | sass:list |
1542-
| `alpha` | | sass:color | | `append` | | sass:list |
1543-
| `adjust-color` | `adjust` | sass:color | | `zip` | | sass:list |
1544-
| `scale-color` | `scale` | sass:color | | `index` | | sass:list |
1545-
| `change-color` | `change` | sass:color | | `list-separator` | `separator` | sass:list |
1546-
| `ie-hex-str` | | sass:color | | | | |
1547-
| | | | | `feature-exists` | | sass:meta |
1548-
| `map-get` | `get` | sass:map | | `variable-exists` | | sass:meta |
1549-
| `map-merge` | `merge` | sass:map | | `global-variable-exists` | | sass:meta |
1550-
| `map-remove` | `remove` | sass:map | | `function-exists` | | sass:meta |
1551-
| `map-keys` | `keys` | sass:map | | `mixin-exists` | | sass:meta |
1552-
| `map-values` | `values` | sass:map | | `inspect` | | sass:meta |
1553-
| `map-has-key` | `has-key` | sass:map | | `get-function` | | sass:meta |
1554-
| | | | | `type-of` | | sass:meta |
1555-
| `unquote` | | sass:string | | `call` | | sass:meta |
1556-
| `quote` | | sass:string | | `content-exists` | | sass:meta |
1557-
| `str-length` | `length` | sass:string | | `keywords` | | sass:meta
1558-
| `str-insert` | `insert` | sass:string | | | `module-variables` | sass:meta |
1559-
| `str-index` | `index` | sass:string | | | `module-functions` | sass:meta |
1560-
| `str-slice` | `slice` | sass:string | | | | |
1561-
| `to-upper-case` | | sass:string | | `selector-nest` | `nest` | sass:selector |
1562-
| `to-lower-case` | | sass:string | | `selector-append` | `append` | sass:selector |
1563-
| `unique-id` | | sass:string | | `selector-replace` | `replace` | sass:selector |
1539+
| `complement` | | sass:color | | `nth` | | sass:list |
1540+
| `invert` | | sass:color | | `set-nth` | | sass:list |
1541+
| `alpha` | | sass:color | | `join` | | sass:list |
1542+
| `adjust-color` | `adjust` | sass:color | | `append` | | sass:list |
1543+
| `scale-color` | `scale` | sass:color | | `zip` | | sass:list |
1544+
| `change-color` | `change` | sass:color | | `index` | | sass:list |
1545+
| `ie-hex-str` | | sass:color | | `list-separator` | `separator` | sass:list |
1546+
| | | | | | | |
1547+
| `map-get` | `get` | sass:map | | `feature-exists` | | sass:meta |
1548+
| `map-merge` | `merge` | sass:map | | `variable-exists` | | sass:meta |
1549+
| `map-remove` | `remove` | sass:map | | `global-variable-exists` | | sass:meta |
1550+
| `map-keys` | `keys` | sass:map | | `function-exists` | | sass:meta |
1551+
| `map-values` | `values` | sass:map | | `mixin-exists` | | sass:meta |
1552+
| `map-has-key` | `has-key` | sass:map | | `inspect` | | sass:meta |
1553+
| | | | | `get-function` | | sass:meta |
1554+
| `unquote` | | sass:string | | `type-of` | | sass:meta |
1555+
| `quote` | | sass:string | | `call` | | sass:meta |
1556+
| `str-length` | `length` | sass:string | | `content-exists` | | sass:meta |
1557+
| `str-insert` | `insert` | sass:string | | `keywords` | | sass:meta
1558+
| `str-index` | `index` | sass:string | | | `module-variables` | sass:meta |
1559+
| `str-slice` | `slice` | sass:string | | | `module-functions` | sass:meta |
1560+
| `to-upper-case` | | sass:string | | | | |
1561+
| `to-lower-case` | | sass:string | | `selector-nest` | `nest` | sass:selector |
1562+
| `unique-id` | | sass:string | | `selector-append` | `append` | sass:selector |
1563+
| | | | | `selector-replace` | `replace` | sass:selector |
15641564
| | | | | `selector-unify` | `unify` | sass:selector |
15651565
| | | | | `is-superselector` | | sass:selector |
15661566
| | | | | `simple-selectors` | | sass:selector |
@@ -1573,11 +1573,11 @@ In addition, one built-in mixin will be added:
15731573
| ---------- | --------- |
15741574
| `load-css` | sass:meta |
15751575
1576-
The existing built-in functions `lighten()`, `darken()`, `saturate()`,
1577-
`desaturate()`, `opacify()`, `fade-in()`, `transparentize()`, and `fade-out()`
1578-
will not be added to any module. Instead, functions with the same names will be
1579-
added to the `sass:color` module that will always emit errors suggesting that
1580-
the user use `color.adjust()` instead.
1576+
The existing built-in functions `adjust-hue()`, `lighten()`, `darken()`,
1577+
`saturate()`, `desaturate()`, `opacify()`, `fade-in()`, `transparentize()`, and
1578+
`fade-out()` will not be added to any module. Instead, functions with the same
1579+
names will be added to the `sass:color` module that will always emit errors
1580+
suggesting that the user use `color.adjust()` instead.
15811581
15821582
> These functions are shorthands for `color.adjust()`. However, `color.adjust()`
15831583
> generally produces less useful results than `color.scale()`, so having

0 commit comments

Comments
 (0)