Skip to content

Commit a5d430d

Browse files
Fix display issues of close button in contextual light and dark modes (#41126)
1 parent 0cbfe13 commit a5d430d

File tree

3 files changed

+17
-7
lines changed

3 files changed

+17
-7
lines changed

scss/_close.scss

+9-6
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
--#{$prefix}btn-close-focus-shadow: #{$btn-close-focus-shadow};
1313
--#{$prefix}btn-close-focus-opacity: #{$btn-close-focus-opacity};
1414
--#{$prefix}btn-close-disabled-opacity: #{$btn-close-disabled-opacity};
15-
--#{$prefix}btn-close-white-filter: #{$btn-close-white-filter};
1615
// scss-docs-end close-css-vars
1716

1817
box-sizing: content-box;
@@ -21,6 +20,7 @@
2120
padding: $btn-close-padding-y $btn-close-padding-x;
2221
color: var(--#{$prefix}btn-close-color);
2322
background: transparent var(--#{$prefix}btn-close-bg) center / $btn-close-width auto no-repeat; // include transparent for button elements
23+
filter: var(--#{$prefix}btn-close-filter);
2424
border: 0; // for button elements
2525
@include border-radius();
2626
opacity: var(--#{$prefix}btn-close-opacity);
@@ -47,17 +47,20 @@
4747
}
4848

4949
@mixin btn-close-white() {
50-
filter: var(--#{$prefix}btn-close-white-filter);
50+
--#{$prefix}btn-close-filter: #{$btn-close-filter-dark};
5151
}
5252

5353
.btn-close-white {
5454
@include btn-close-white();
5555
}
5656

57+
:root,
58+
[data-bs-theme="light"] {
59+
--#{$prefix}btn-close-filter: #{$btn-close-filter};
60+
}
61+
5762
@if $enable-dark-mode {
58-
@include color-mode(dark) {
59-
.btn-close {
60-
@include btn-close-white();
61-
}
63+
@include color-mode(dark, true) {
64+
@include btn-close-white();
6265
}
6366
}

scss/_variables-dark.scss

+6
Original file line numberDiff line numberDiff line change
@@ -94,3 +94,9 @@ $accordion-button-active-icon-dark: url("data:image/svg+xml,<svg xmlns='http://
9494
$carousel-indicator-active-bg-dark: $carousel-dark-indicator-active-bg !default;
9595
$carousel-caption-color-dark: $carousel-dark-caption-color !default;
9696
$carousel-control-icon-filter-dark: $carousel-dark-control-icon-filter !default;
97+
98+
//
99+
// Close button
100+
//
101+
102+
$btn-close-filter-dark: $btn-close-white-filter !default;

scss/_variables.scss

+2-1
Original file line numberDiff line numberDiff line change
@@ -1713,7 +1713,8 @@ $btn-close-opacity: .5 !default;
17131713
$btn-close-hover-opacity: .75 !default;
17141714
$btn-close-focus-opacity: 1 !default;
17151715
$btn-close-disabled-opacity: .25 !default;
1716-
$btn-close-white-filter: invert(1) grayscale(100%) brightness(200%) !default;
1716+
$btn-close-filter: null !default;
1717+
$btn-close-white-filter: invert(1) grayscale(100%) brightness(200%) !default; // Deprecated in v5.3.4
17171718
// scss-docs-end close-variables
17181719

17191720

0 commit comments

Comments
 (0)