Skip to content

Commit 6594bcb

Browse files
beth-panxnmetulev
andauthored
Fix login / mgt-person custom css (#836)
* Fix login / mgt-person custom css * Address pr feedback Co-authored-by: Nikola Metulev <[email protected]>
1 parent 8b5cabe commit 6594bcb

File tree

3 files changed

+27
-6
lines changed

3 files changed

+27
-6
lines changed

packages/mgt-components/src/components/mgt-login/mgt-login.scss

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,16 +39,21 @@ mgt-login .login-button {
3939
height: var(--height);
4040
margin: var(--margin);
4141
padding: var(--padding);
42-
color: $button__color;
42+
color: set-var(button__color, $theme-default, $login);
4343
background-color: var(--button-background-color, transparent);
4444
border: none;
4545
cursor: pointer;
4646
transition: color 0.3s, background-color 0.3s;
4747

48+
mgt-person {
49+
--color-sub1: set-var(button__color, $theme-default, $login);
50+
}
51+
4852
&:hover {
4953
color: $button__color--hover;
5054
--color: #{$button__color--hover};
5155
background-color: $button__background-color--hover;
56+
--color-sub1: #{$button__color--hover};
5257
}
5358

5459
&:focus {
@@ -78,8 +83,11 @@ mgt-login .popup {
7883
background-color: $popup__background-color;
7984
min-width: 240px;
8085
z-index: 1;
81-
color: $popup__color;
82-
--color: #{$popup__color};
86+
color: set-var(popup__color, $theme-default, $login);
87+
88+
mgt-person {
89+
--color-sub1: set-var(popup__color, $theme-default, $login);
90+
}
8391
}
8492

8593
:host .popup-content,

packages/mgt-components/src/components/mgt-login/mgt-login.theme.scss

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,23 @@
88
@import '../../styles/shared-sass-variables.scss';
99

1010
$button__color--hover: var(--button-color--hover, #{$commblue_primary});
11-
$button__color: var(--button-color, set-var(color__sub1, $theme-default, $common));
1211
$button__background-color--hover: var(
1312
--button-background-color--hover,
1413
set-var(background-color--hover, $theme-default, $common)
1514
);
16-
$popup__color: var(--popup-color, set-var(color__sub1, $theme-default, $common));
1715
$popup__background-color: var(--popup-background-color, set-var(background-color__sub1, $theme-default, $common));
16+
17+
$login: (
18+
button__color: (
19+
_var: --button-color,
20+
dark: $text__color__sub1--dark,
21+
light: $text__color__sub1--light
22+
),
23+
popup__color: (
24+
_var: --popup-color,
25+
dark: $text__color__sub1--dark,
26+
light: $text__color__sub1--light
27+
)
28+
);
29+
30+
@include create-themes($login);

packages/mgt-components/src/components/mgt-person/mgt-person.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ $presence-color-oof: #b4009e;
5050

5151
:host .root {
5252
display: block;
53-
color: $color;
5453
}
5554

5655
:host svg,
@@ -64,6 +63,7 @@ mgt-person .person-root {
6463
position: relative;
6564
display: flex;
6665
align-items: center;
66+
color: $color;
6767

6868
.user-avatar {
6969
width: $avatar-size;

0 commit comments

Comments
 (0)