fix: code smells (#603)

* fix: code smells * chore: remove todo * fix: code smells
graasp · Jan 14, 2025 · 7298252 · 7298252
1 parent 689f89e
commit 7298252
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 16 deletions.
diff --git a/src/components/ui/Navigator/CurrentItemNavigation.tsx b/src/components/ui/Navigator/CurrentItemNavigation.tsx
@@ -1,8 +1,9 @@
+import { Stack } from '@mui/material';
+
 import { DiscriminatedItem, ItemType } from '@graasp/sdk';
 
 import { TypographyLink } from '../TypographyLink.js';
 import ItemMenu, { ItemMenuProps } from './ItemMenu.js';
-import CenterAlignWrapper from './common/CenterAlignWrapper.js';
 
 export type CurrentItemProps = {
   item: DiscriminatedItem;
@@ -24,7 +25,7 @@ export function CurrentItemNavigation({
   showArrow,
 }: Readonly<CurrentItemProps>): JSX.Element | null {
   return (
-    <CenterAlignWrapper>
+    <Stack alignItems="center">
       <TypographyLink
         id={buildBreadcrumbsItemLinkId?.(item.id)}
         key={item.id}
@@ -43,6 +44,6 @@ export function CurrentItemNavigation({
           renderArrow={showArrow}
         />
       )}
-    </CenterAlignWrapper>
+    </Stack>
   );
 }
diff --git a/src/components/ui/Navigator/common/CenterAlignWrapper.tsx b/src/components/ui/Navigator/common/CenterAlignWrapper.tsx
diff --git a/src/components/ui/Navigator/common/constants.ts b/src/components/ui/Navigator/common/constants.ts
diff --git a/src/modules/account/settings/publicProfile/EditPublicProfile.tsx b/src/modules/account/settings/publicProfile/EditPublicProfile.tsx
@@ -61,7 +61,6 @@ export function EditPublicProfile({
     mode: 'onChange',
     defaultValues: {
       bio: profile?.bio,
-      // TODO: sanitize links
       twitterID: profile?.twitterID,
       facebookID: profile?.facebookID,
       linkedinID: profile?.linkedinID,

diff --git a/src/modules/landing/home/UserStoryButton.tsx b/src/modules/landing/home/UserStoryButton.tsx
@@ -18,7 +18,10 @@ const StyledAnchor = styled('a')(({ theme }) => ({
   padding: theme.spacing(2, 4),
 }));
 
-export function UserStoryButton({ id, text }: { id: string; text: string }) {
+export function UserStoryButton({
+  id,
+  text,
+}: Readonly<{ id: string; text: string }>) {
   return (
     <StyledAnchor href={`#${id}`}>
       {text}

diff --git a/src/modules/landing/support/SmallTutorialCard.tsx b/src/modules/landing/support/SmallTutorialCard.tsx
@@ -8,11 +8,11 @@ export function SmallTutorialCard({
   title,
   description,
   link,
-}: {
+}: Readonly<{
   title: string;
   description: string;
   link: string;
-}) {
+}>) {
   const { t } = useTranslation(NS.Landing, { keyPrefix: 'SUPPORT.TUTORIALS' });
   return (
     <Card sx={{ width: '100%', height: '100%', p: 2, textAlign: 'center' }}>