Skip to content

Commit 7da8739

Browse files
committed
fix(sass): made variables for button-default-border and button-default-active-border
made variables for button-default-border and button-default-active-bordermade variables #5576
1 parent c7e61e4 commit 7da8739

File tree

3 files changed

+16
-11
lines changed

3 files changed

+16
-11
lines changed

Diff for: scss/_button.scss

+9-9
Original file line numberDiff line numberDiff line change
@@ -72,55 +72,55 @@
7272
}
7373

7474
&.button-light {
75-
@include button-style($button-light-bg, $button-light-border, $button-light-active-bg, $button-light-active-border, $button-light-text);
75+
@include button-style($button-light-bg, $button-default-border, $button-light-active-bg, $button-default-active-border, $button-light-text);
7676
@include button-clear($button-light-border);
7777
@include button-outline($button-light-border);
7878
}
7979

8080
&.button-stable {
81-
@include button-style($button-stable-bg, $button-stable-border, $button-stable-active-bg, $button-stable-active-border, $button-stable-text);
81+
@include button-style($button-stable-bg, $button-default-border, $button-stable-active-bg, $button-default-active-border, $button-stable-text);
8282
@include button-clear($button-stable-border);
8383
@include button-outline($button-stable-border);
8484
}
8585

8686
&.button-positive {
87-
@include button-style($button-positive-bg, $button-positive-border, $button-positive-active-bg, $button-positive-active-border, $button-positive-text);
87+
@include button-style($button-positive-bg, $button-default-border, $button-positive-active-bg, $button-default-active-border, $button-positive-text);
8888
@include button-clear($button-positive-bg);
8989
@include button-outline($button-positive-bg);
9090
}
9191

9292
&.button-calm {
93-
@include button-style($button-calm-bg, $button-calm-border, $button-calm-active-bg, $button-calm-active-border, $button-calm-text);
93+
@include button-style($button-calm-bg, $button-default-border, $button-calm-active-bg, $button-default-active-border, $button-calm-text);
9494
@include button-clear($button-calm-bg);
9595
@include button-outline($button-calm-bg);
9696
}
9797

9898
&.button-assertive {
99-
@include button-style($button-assertive-bg, $button-assertive-border, $button-assertive-active-bg, $button-assertive-active-border, $button-assertive-text);
99+
@include button-style($button-assertive-bg, $button-default-border, $button-assertive-active-bg, $button-default-active-border, $button-assertive-text);
100100
@include button-clear($button-assertive-bg);
101101
@include button-outline($button-assertive-bg);
102102
}
103103

104104
&.button-balanced {
105-
@include button-style($button-balanced-bg, $button-balanced-border, $button-balanced-active-bg, $button-balanced-active-border, $button-balanced-text);
105+
@include button-style($button-balanced-bg, $button-default-border, $button-balanced-active-bg, $button-default-active-border, $button-balanced-text);
106106
@include button-clear($button-balanced-bg);
107107
@include button-outline($button-balanced-bg);
108108
}
109109

110110
&.button-energized {
111-
@include button-style($button-energized-bg, $button-energized-border, $button-energized-active-bg, $button-energized-active-border, $button-energized-text);
111+
@include button-style($button-energized-bg, $button-default-border, $button-energized-active-bg, $button-default-active-border, $button-energized-text);
112112
@include button-clear($button-energized-bg);
113113
@include button-outline($button-energized-bg);
114114
}
115115

116116
&.button-royal {
117-
@include button-style($button-royal-bg, $button-royal-border, $button-royal-active-bg, $button-royal-active-border, $button-royal-text);
117+
@include button-style($button-royal-bg, $button-default-border, $button-royal-active-bg, $button-default-active-border, $button-royal-text);
118118
@include button-clear($button-royal-bg);
119119
@include button-outline($button-royal-bg);
120120
}
121121

122122
&.button-dark {
123-
@include button-style($button-dark-bg, $button-dark-border, $button-dark-active-bg, $button-dark-active-border, $button-dark-text);
123+
@include button-style($button-dark-bg, $button-default-border, $button-dark-active-bg, $button-default-active-border, $button-dark-text);
124124
@include button-clear($button-dark-bg);
125125
@include button-outline($button-dark-bg);
126126
}

Diff for: scss/_mixins.scss

+4-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
// --------------------------------------------------
44

55
@mixin button-style($bg-color, $border-color, $active-bg-color, $active-border-color, $color) {
6-
border-color: transparent;//$border-color;
6+
border-color: $border-color;
77
background-color: $bg-color;
88
color: $color;
99

@@ -14,7 +14,9 @@
1414
}
1515
&.active,
1616
&.activated {
17-
//border-color: transparent;//$active-border-color;
17+
@if $active-border-color != ""{
18+
border-color: $active-border-color;
19+
}
1820
background-color: $active-bg-color;
1921
//box-shadow: inset 0 1px 4px rgba(0,0,0,0.1);
2022
}

Diff for: scss/_variables.scss

+3
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,9 @@ $button-bar-button-height: 32px !default;
9393
$button-bar-button-padding: 8px !default;
9494
$button-bar-button-icon-size: 20px !default;
9595

96+
$button-default-border: transparent;
97+
$button-default-active-border:null;
98+
9699
$button-light-bg: $light !default;
97100
$button-light-text: #444 !default;
98101
$button-light-border: #ddd !default;

0 commit comments

Comments
 (0)