Skip to content

Commit e197e3c

Browse files
author
msingh45
authored
Merge pull request #127 from ASU/uds-345
fix(bootstrap4-theme): Centered exit icon in close button
2 parents 71b36ae + 619c3e4 commit e197e3c

File tree

5 files changed

+25
-1
lines changed

5 files changed

+25
-1
lines changed

packages/bootstrap4-theme/src/scss/design-tokens/_variables.scss

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11

22
// Do not edit directly
3-
// Generated on Fri, 29 Jan 2021 16:27:51 GMT
3+
// Generated on Tue, 02 Mar 2021 19:26:48 GMT
44

55
$uds-asset-font-icon-name: "FontAwesome";
66
$uds-asset-font-icon-ttf: "assets/fontawesome/webfonts/fa-regular-400.ttf";
@@ -177,6 +177,8 @@ $uds-component-button-badge-height: 1.375rem; // Magic number: 22px not a multip
177177
$uds-component-button-close-height: 2rem;
178178
$uds-component-button-close-opacity: 50%;
179179
$uds-component-button-close-width: 2rem;
180+
$uds-component-button-close-padding-top: 0;
181+
$uds-component-button-close-padding-bottom: 0.125rem;
180182
$uds-component-button-close-disabled-opacity: 100%;
181183
$uds-component-button-close-white-background-color: #ffffff;
182184
$uds-component-button-close-gray-background-color: #bfbfbf;

packages/bootstrap4-theme/src/scss/extends/_alerts.scss

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,13 @@
1818
}
1919
.alert-close {
2020
flex: 1;
21+
22+
// Edits close button content (i.e. "X" inside bubble)
23+
.close
24+
{
25+
padding-top: $btn-close-padding-top;
26+
padding-bottom: $btn-close-padding-bottom;
27+
}
2128
}
2229
}
2330

packages/bootstrap4-theme/src/scss/extends/_banners.scss

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,13 @@
3636
flex: 2;
3737
margin-top: -$uds-size-spacing-2;
3838
margin-right: -$uds-size-spacing-2;
39+
40+
// Edits close button content (i.e. "X" inside bubble)
41+
.close
42+
{
43+
padding-top: $btn-close-padding-top;
44+
padding-bottom: $btn-close-padding-bottom;
45+
}
3946
}
4047
}
4148

packages/bootstrap4-theme/src/scss/variables/_buttons.scss

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@ $btn-padding-x:$uds-component-button-padding-x;
99
$btn-disabled-opacity:$uds-component-button-disabled-opacity;
1010
$btn-line-height:$uds-component-button-line-height;
1111
$btn-transition: $uds-component-button-transition;
12+
$btn-close-padding-top: $uds-component-button-close-padding-top;
13+
$btn-close-padding-bottom: $uds-component-button-close-padding-bottom;
1214

1315
// Bootstrap variable overrides for badges
1416
$badge-border-radius: $uds-component-button-border-radius;

packages/design-tokens/components/button.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,12 @@
9898
"width": {
9999
"value": "2"
100100
},
101+
"padding-top": {
102+
"value": "0"
103+
},
104+
"padding-bottom": {
105+
"value": "0.125rem"
106+
},
101107
"disabled": {
102108
"opacity": {
103109
"value": "100%"

0 commit comments

Comments
 (0)