diff --git a/src/components/ExternalPill/index.tsx b/src/components/ExternalPill/index.tsx index 63ce8671..0d49542e 100644 --- a/src/components/ExternalPill/index.tsx +++ b/src/components/ExternalPill/index.tsx @@ -45,6 +45,7 @@ export function ExternalPill({ {modifiers.map((modifier, index) => ( - <> + {index < modifiers.length - 1 && ( + )} - + ))} {keys.length > 0 && +} {keys.map((key, index) => ( - <> + {index < keys.length - 1 && ( + )} - + ))} ); @@ -96,13 +97,14 @@ export function KeyHint({
{titleText}
{dismissable && ( -
-
+ )}
diff --git a/src/components/LanguageIndicator/index.tsx b/src/components/LanguageIndicator/index.tsx index 54f3eb50..12a3456f 100644 --- a/src/components/LanguageIndicator/index.tsx +++ b/src/components/LanguageIndicator/index.tsx @@ -54,7 +54,7 @@ export function LanguageIndicator({ indicatorOnly = false, }: LanguageIndicatorProps) { return ( -
+
diff --git a/src/components/SegmentedButton/index.tsx b/src/components/SegmentedButton/index.tsx index eff528aa..11912c12 100644 --- a/src/components/SegmentedButton/index.tsx +++ b/src/components/SegmentedButton/index.tsx @@ -35,7 +35,7 @@ const SegmentedButtonBase = ({ children, className }: SegmentedButtonProps) => { }, ); }), - [children], + [children, hoveredId], ); return ( diff --git a/src/components/Table/context/tableProvider.tsx b/src/components/Table/context/tableProvider.tsx index 516cc6dc..a29c9a27 100644 --- a/src/components/Table/context/tableProvider.tsx +++ b/src/components/Table/context/tableProvider.tsx @@ -28,7 +28,7 @@ export function TableProvider({ return newSet; }); }, - [expandedRowKeys, setExpandedRowKeys], + [setExpandedRowKeys], ); return ( diff --git a/src/components/Tabs/index.tsx b/src/components/Tabs/index.tsx index e78aae96..18b24bab 100644 --- a/src/components/Tabs/index.tsx +++ b/src/components/Tabs/index.tsx @@ -48,11 +48,11 @@ export function Tabs({ className, }: TabsProps) { const [activeTab, setActiveTab] = useState( - selectedTab || children[0].props.id, + selectedTab ?? children[0].props.id, ); useEffect(() => { - setActiveTab(selectedTab || children[0].props.id); + setActiveTab(selectedTab ?? children[0].props.id); }, [selectedTab]); const validChildren = React.Children.toArray(children).filter( diff --git a/src/components/UserAvatar/index.tsx b/src/components/UserAvatar/index.tsx index 10bc42b9..46adfb81 100644 --- a/src/components/UserAvatar/index.tsx +++ b/src/components/UserAvatar/index.tsx @@ -29,7 +29,7 @@ function getFallbackColor(name: string | undefined) { const firstLetter = name[0].toLowerCase(); const index = firstLetter.charCodeAt(0) - "a".charCodeAt(0); - return fallbackColors[index % fallbackColors.length]; + return fallbackColors[Math.abs(index) % fallbackColors.length]; } export function UserAvatar({ diff --git a/src/components/WorkspaceSelector/SearchBox.tsx b/src/components/WorkspaceSelector/SearchBox.tsx index cb9b48c0..643e1958 100644 --- a/src/components/WorkspaceSelector/SearchBox.tsx +++ b/src/components/WorkspaceSelector/SearchBox.tsx @@ -29,6 +29,8 @@ export function SearchBox({ {search && (