Skip to content

Commit b80ff6f

Browse files
Merge pull request #623 from microsoft/psl-bugv3-26397
fix: Add Deletion icon in team selector
2 parents ab34cd0 + 88c3096 commit b80ff6f

2 files changed

Lines changed: 12 additions & 12 deletions

File tree

src/frontend/src/components/common/TeamSelector.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import {
2929
} from '@fluentui/react-components';
3030
import {
3131
ChevronUpDown16Regular,
32-
MoreHorizontal20Regular,
32+
DeleteRegular,
3333
Search20Regular,
3434
Dismiss20Regular,
3535
CheckmarkCircle20Filled,
@@ -475,21 +475,21 @@ const TeamSelector: React.FC<TeamSelectorProps> = ({
475475
mountNode={document.querySelector('[role="dialog"]') || undefined}
476476
>
477477
<Button
478-
icon={<MoreHorizontal20Regular />}
478+
icon={<DeleteRegular />}
479479
appearance="subtle"
480480
size="small"
481481
disabled={true}
482-
className={`${styles.moreButton} ${styles.moreButtonDisabled || ''}`}
482+
className={`${styles.deleteButton} ${styles.deleteButtonDisabled || ''}`}
483483
onClick={(e: React.MouseEvent) => e.stopPropagation()}
484484
/>
485485
</Tooltip>
486486
) : (
487487
<Button
488-
icon={<MoreHorizontal20Regular />}
488+
icon={<DeleteRegular />}
489489
appearance="subtle"
490490
size="small"
491491
onClick={(e: React.MouseEvent) => handleDeleteTeam(team, e)}
492-
className={styles.moreButton}
492+
className={styles.deleteButton}
493493
/>
494494
)}
495495
</div>

src/frontend/src/styles/TeamSelector.module.css

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -391,8 +391,8 @@
391391
min-height: auto !important;
392392
}
393393

394-
/* More Button Column */
395-
.moreButton {
394+
/* Delete Button Column */
395+
.deleteButton {
396396
color: var(--colorNeutralForeground2) !important;
397397
background-color: transparent !important;
398398
min-width: 32px !important;
@@ -403,18 +403,18 @@
403403
margin-top: 2px;
404404
}
405405

406-
.moreButton:hover {
407-
background-color: var(--colorNeutralBackground3) !important;
408-
color: var(--colorNeutralForeground1) !important;
406+
.deleteButton:hover {
407+
background-color: var(--colorPaletteRedBackground2) !important;
408+
color: var(--colorPaletteRedForeground1) !important;
409409
}
410-
.moreButtonDisabled {
410+
.deleteButtonDisabled {
411411
color: var(--colorNeutralForeground4) !important;
412412
background-color: transparent !important;
413413
cursor: not-allowed !important;
414414
opacity: 0.6 !important;
415415
}
416416

417-
.moreButtonDisabled:hover {
417+
.deleteButtonDisabled:hover {
418418
background-color: transparent !important;
419419
color: var(--colorNeutralForeground4) !important;
420420
opacity: 0.6 !important;

0 commit comments

Comments
 (0)