Skip to content

Commit 91827b1

Browse files
committed
refactor: replace HelpIcon w/WidgetsIcon
Based on user feedback, we believe the `WidgetsIcon` will cause less confusion.
1 parent 02c0100 commit 91827b1

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

site/src/components/Resources/ResourceAvatar.tsx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import Avatar from "@material-ui/core/Avatar"
22
import { makeStyles } from "@material-ui/core/styles"
33
import FolderIcon from "@material-ui/icons/FolderOutlined"
4-
import HelpIcon from "@material-ui/icons/HelpOutlined"
4+
import WidgetsIcon from "@material-ui/icons/WidgetsOutlined"
55
import ImageIcon from "@material-ui/icons/ImageOutlined"
66
import MemoryIcon from "@material-ui/icons/MemoryOutlined"
77
import React from "react"
@@ -22,15 +22,13 @@ const iconByResource: Record<string, typeof MemoryIcon> = {
2222
google_compute_instance: AdjustedMemoryIcon,
2323
aws_instance: AdjustedMemoryIcon,
2424
kubernetes_deployment: AdjustedMemoryIcon,
25-
null_resource: HelpIcon,
25+
null_resource: WidgetsIcon,
2626
}
2727

2828
export type ResourceAvatarProps = { type: string }
2929

3030
export const ResourceAvatar: React.FC<ResourceAvatarProps> = ({ type }) => {
31-
// this resource can return undefined
32-
// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
33-
const IconComponent = iconByResource[type] ?? HelpIcon
31+
const IconComponent = iconByResource[type] ?? WidgetsIcon
3432
const styles = useStyles()
3533

3634
return (

0 commit comments

Comments
 (0)