Skip to content

Commit

Permalink
fix(ui): fix icon of notification item
Browse files Browse the repository at this point in the history
  • Loading branch information
Vu Van Duc authored and Vu Van Duc committed Jul 15, 2024
1 parent 3d0d8ee commit bdcaf34
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 9 deletions.
Binary file removed src/ui/assets/images/multisign-icon.jpg
Binary file not shown.
17 changes: 10 additions & 7 deletions src/ui/pages/Notifications/NotificationItem.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
import { IonIcon, IonItem, IonLabel } from "@ionic/react";
import { t } from "i18next";
import { ellipsisHorizontal, fingerPrintOutline } from "ionicons/icons";
import {
ellipsisHorizontal,
fingerPrintOutline,
idCardOutline,
} from "ionicons/icons";
import { MouseEvent, useCallback, useEffect, useState } from "react";
import { Trans } from "react-i18next";
import { Agent } from "../../../core/agent/agent";
Expand All @@ -12,7 +16,6 @@ import { MultiSigIcpRequestDetails } from "../../../core/agent/services/identifi
import { useAppSelector } from "../../../store/hooks";
import { getConnectionsCache } from "../../../store/reducers/connectionsCache";
import KeriLogo from "../../assets/images/KeriGeneric.jpg";
import MultisignReferIcon from "../../assets/images/multisign-icon.jpg";
import { timeDifference } from "../../utils/formatters";
import { NotificationItemProps } from "./Notification.types";

Expand Down Expand Up @@ -82,11 +85,11 @@ const NotificationItem = ({
switch (item.a.r) {
case NotificationRoute.ExnIpexGrant:
case NotificationRoute.ExnIpexApply:
return fingerPrintOutline;
return idCardOutline;
case NotificationRoute.MultiSigIcp:
return MultisignReferIcon;
return fingerPrintOutline;
default:
return MultisignReferIcon;
return idCardOutline;
}
};

Expand All @@ -111,9 +114,9 @@ const NotificationItem = ({
className="notifications-tab-item-logo"
data-testid="notifications-tab-item-logo"
/>
<img
<IonIcon
src={referIcon(item)}
alt="refer-icon"
size="small"
className="notification-ref-icon"
/>
</div>
Expand Down
4 changes: 2 additions & 2 deletions src/ui/pages/Notifications/Notifications.scss
Original file line number Diff line number Diff line change
Expand Up @@ -100,8 +100,8 @@
position: absolute;
bottom: 0.115rem;
right: 0.7rem;
width: 1.25rem;
height: 1.25rem;
width: 0.85rem;
height: 0.85rem;
border-radius: 0.625rem;
padding: 0.25rem;
background-color: var(--ion-color-light-grey);
Expand Down

0 comments on commit bdcaf34

Please sign in to comment.