Skip to content

Commit 3e8037c

Browse files
committed
Merge branch 'next' into FW-6252-chip
2 parents fd300b2 + 207d485 commit 3e8037c

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

core/src/themes/functions.color.scss

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939

4040
$value: map.get($values, $variation);
4141

42-
// TODO(): this can be removed when foreground is required
42+
// TODO(FW-6417): this can be removed when foreground is required
4343
// Fallback to "base" variant when "foreground" variant is undefined
4444
@if ($variation == foreground and $value == null) {
4545
$variation: base;
@@ -143,7 +143,7 @@
143143

144144
$prefix: if($hue == "subtle", "-subtle", "");
145145

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
147147
// Only proceed if the color variants exist
148148
@if $color-variants {
149149
// Grab the individual color variants
@@ -165,7 +165,7 @@
165165
) !important;
166166
--ion-color#{$prefix}-shade: var(--ion-color-#{$color-name}#{$prefix}-shade, #{$shade}) !important;
167167
--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
169169
// required by all palettes for all themes:
170170
// --ion-color#{$prefix}-foreground: var(--ion-color-#{$color-name}#{$prefix}-foreground, #{$foreground}) !important;
171171
--ion-color#{$prefix}-foreground: var(
@@ -241,7 +241,7 @@
241241
--ion-color-#{$color-name}#{$prefix}-contrast-rgb: #{map.get($colors, contrast-rgb)};
242242
--ion-color-#{$color-name}#{$prefix}-shade: #{map.get($colors, shade)};
243243
--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
245245
// themes. It should not be added until we want foreground to be required for
246246
// ios and md because this will be a breaking change, requiring users to add
247247
// `--ion-color-{color}-foreground` in order to override the default colors

core/src/themes/native/test/colors/index.html

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
}
4747

4848
.ion-color {
49-
color: var(--ion-color-base);
49+
color: var(--ion-color-foreground);
5050
}
5151

5252
.ion-color-background {
@@ -66,17 +66,17 @@
6666

6767
.ion-color-background-08 {
6868
background: rgb(var(--ion-color-base-rgb), 0.08);
69-
color: var(--ion-color-base);
69+
color: var(--ion-color-foreground);
7070
}
7171

7272
.ion-color-background-12 {
7373
background: rgb(var(--ion-color-base-rgb), 0.12);
74-
color: var(--ion-color-base);
74+
color: var(--ion-color-foreground);
7575
}
7676

7777
.ion-color-background-16 {
7878
background: rgb(var(--ion-color-base-rgb), 0.16);
79-
color: var(--ion-color-base);
79+
color: var(--ion-color-foreground);
8080
}
8181
</style>
8282
</head>

0 commit comments

Comments
 (0)