|
39 | 39 |
|
40 | 40 | $value: map.get($values, $variation); |
41 | 41 |
|
42 | | - // TODO(): this can be removed when foreground is required |
| 42 | + // TODO(FW-6417): this can be removed when foreground is required |
43 | 43 | // Fallback to "base" variant when "foreground" variant is undefined |
44 | 44 | @if ($variation == foreground and $value == null) { |
45 | 45 | $variation: base; |
|
143 | 143 |
|
144 | 144 | $prefix: if($hue == "subtle", "-subtle", ""); |
145 | 145 |
|
146 | | - // TODO this @if can be removed if we add subtle colors for ios and md |
| 146 | + // TODO(FW-6417) this @if can be removed if we add subtle colors for ios and md |
147 | 147 | // Only proceed if the color variants exist |
148 | 148 | @if $color-variants { |
149 | 149 | // Grab the individual color variants |
|
165 | 165 | ) !important; |
166 | 166 | --ion-color#{$prefix}-shade: var(--ion-color-#{$color-name}#{$prefix}-shade, #{$shade}) !important; |
167 | 167 | --ion-color#{$prefix}-tint: var(--ion-color-#{$color-name}#{$prefix}-tint, #{$tint}) !important; |
168 | | - // TODO(): remove the fallback variable when the foreground variable is |
| 168 | + // TODO(FW-6417): remove the fallback variable when the foreground variable is |
169 | 169 | // required by all palettes for all themes: |
170 | 170 | // --ion-color#{$prefix}-foreground: var(--ion-color-#{$color-name}#{$prefix}-foreground, #{$foreground}) !important; |
171 | 171 | --ion-color#{$prefix}-foreground: var( |
|
241 | 241 | --ion-color-#{$color-name}#{$prefix}-contrast-rgb: #{map.get($colors, contrast-rgb)}; |
242 | 242 | --ion-color-#{$color-name}#{$prefix}-shade: #{map.get($colors, shade)}; |
243 | 243 | --ion-color-#{$color-name}#{$prefix}-tint: #{map.get($colors, tint)}; |
244 | | - // TODO(): this "if" can be removed when foreground is defined for ios/md |
| 244 | + // TODO(FW-6417): this "if" can be removed when foreground is defined for ios/md |
245 | 245 | // themes. It should not be added until we want foreground to be required for |
246 | 246 | // ios and md because this will be a breaking change, requiring users to add |
247 | 247 | // `--ion-color-{color}-foreground` in order to override the default colors |
|
0 commit comments