Skip to content

Commit a83ae28

Browse files
gavinbarronMnickii
andauthored
fix: person card color (#2533)
removes fluent card and manually styles card background now uses --neutral-floating-layer as card background color Co-authored-by: Nickii Miaro <[email protected]>
1 parent 5bdaea6 commit a83ae28

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ $person-card-base-icons-left-spacing: var(
4646
--file-list-background-color: transparent;
4747
--file-list-box-shadow: none;
4848
--file-item-background-color: transparent;
49-
--card-height: auto;
5049

5150
.small {
5251
max-height: 100vh;

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
@import '../../components/mgt-messages/mgt-messages.theme';
2020
@import '../../components/mgt-file-list/mgt-file-list.theme';
2121

22-
$person-card-background-color: var(--person-card-background-color, transparent);
22+
$person-card-background-color: var(--person-card-background-color, var(--neutral-layer-floating));
2323
$person-card-expanded-background-color-hover: var(
2424
--person-card-expanded-background-color-hover,
2525
var(--neutral-fill-hover)

packages/mgt-components/src/components/mgt-person-card/mgt-person-card.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -564,15 +564,15 @@ export class MgtPersonCard extends MgtTemplatedComponent {
564564
? html`<div @keydown=${this.handleEndOfCard} aria-label=${this.strings.endOfCard} tabindex="0" id="end-of-container"></div>`
565565
: html``;
566566
return html`
567-
<fluent-card class="root" dir=${this.direction}>
567+
<div class="root" dir=${this.direction}>
568568
<div class=${classMap({ small: this._smallView })}>
569569
${navigationTemplate}
570570
${closeCardTemplate}
571571
<div class="person-details-container">${personDetailsTemplate}</div>
572572
<div class="expanded-details-container">${expandedDetailsTemplate}</div>
573573
${tabLocker}
574574
</div>
575-
</fluent-card>
575+
</div>
576576
`;
577577
}
578578

0 commit comments

Comments
 (0)