From 9adad4a4ab5067fd6c74f028f7d1739eb2892699 Mon Sep 17 00:00:00 2001 From: Paul Date: Fri, 3 Sep 2021 11:21:51 +0100 Subject: [PATCH] Fix issues introduced by #162. Switch to more consistent naming scheme. --- external/lang | 2 +- src/components/common/ChannelIcon.tsx | 8 ++++---- src/components/common/user/UserIcon.tsx | 15 ++++++++------- src/styles/_variables.scss | 4 +++- 4 files changed, 16 insertions(+), 13 deletions(-) diff --git a/external/lang b/external/lang index cab80e5c..8676906e 160000 --- a/external/lang +++ b/external/lang @@ -1 +1 @@ -Subproject commit cab80e5c4840c91f53fbc43f97a98923ec3abc51 +Subproject commit 8676906ec594a42ee72dcc5ad227357bb300570d diff --git a/src/components/common/ChannelIcon.tsx b/src/components/common/ChannelIcon.tsx index 48a88400..a7828338 100644 --- a/src/components/common/ChannelIcon.tsx +++ b/src/components/common/ChannelIcon.tsx @@ -52,9 +52,9 @@ export default observer( } // The border radius of the channel icon, if it's a server-channel it should be square (undefined). - let borderRadius; - if (!isServerChannel) { - borderRadius = "--border-radius-half"; + let borderRadius: string | undefined = "--border-radius-channel-icon"; + if (isServerChannel) { + borderRadius = undefined; } return ( @@ -65,7 +65,7 @@ export default observer( height={size} loading="lazy" aria-hidden="true" - {...borderRadius} + borderRadius={borderRadius} src={iconURL ?? fallback} /> ); diff --git a/src/components/common/user/UserIcon.tsx b/src/components/common/user/UserIcon.tsx index 369cd404..884d8e1c 100644 --- a/src/components/common/user/UserIcon.tsx +++ b/src/components/common/user/UserIcon.tsx @@ -1,4 +1,5 @@ import { MicrophoneOff } from "@styled-icons/boxicons-regular"; +import { VolumeMute } from "@styled-icons/boxicons-solid"; import { observer } from "mobx-react-lite"; import { useParams } from "react-router-dom"; import { Presence } from "revolt-api/types/Users"; @@ -12,7 +13,6 @@ import { useClient } from "../../../context/revoltjs/RevoltClient"; import IconBase, { IconBaseProps } from "../IconBase"; import fallback from "../assets/user.png"; -import {VolumeMute} from "@styled-icons/boxicons-solid"; type VoiceStatus = "muted" | "deaf"; interface Props extends IconBaseProps { @@ -48,7 +48,7 @@ const VoiceIndicator = styled.div<{ status: VoiceStatus }>` } ${(props) => - (props.status === "muted" || props.status === "deaf") && + (props.status === "muted" || props.status === "deaf") && css` background: var(--error); `} @@ -104,7 +104,7 @@ export default observer( width={size} height={size} hover={hover} - borderRadius="--border-radius-user-profile-picture" + borderRadius="--border-radius-user-icon" aria-hidden="true" viewBox="0 0 32 32"> - {props.voice === "deaf" && ( + {(props.voice === "deaf" && ( - ) ||props.voice === "muted" && ( - - )} + )) || + (props.voice === "muted" && ( + + ))} )} diff --git a/src/styles/_variables.scss b/src/styles/_variables.scss index a4e91630..56bef282 100644 --- a/src/styles/_variables.scss +++ b/src/styles/_variables.scss @@ -23,7 +23,9 @@ --scrollbar-thickness-ff: thin; --border-radius: 6px; --border-radius-half: 50%; - --border-radius-user-profile-picture: var(--border-radius-half); + + --border-radius-user-icon: var(--border-radius-half); + --border-radius-channel-icon: var(--border-radius-half); --border-radius-server-icon: var(--border-radius-half); --input-border-width: 2px;