Clean up

gallery-so · Apr 19, 2024 · 22d1022 · 22d1022
1 parent ea7bf71
commit 22d1022
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 34 deletions.
diff --git a/apps/mobile/src/components/Button.tsx b/apps/mobile/src/components/Button.tsx
@@ -191,7 +191,6 @@ export function Button({
     buttonVariants[colorScheme][disabled ? 'disabled' : variant][active ? 'active' : 'inactive'];
 
   const sizeVariants: { [size in Size]: string } = {
-    // TODO: check regression
     xs: 'h-[24] px-4',
     sm: 'h-[36] px-4',
     md: 'h-[44] px-6',

diff --git a/apps/mobile/src/components/GalleryEditor/GalleryEditorRow.tsx b/apps/mobile/src/components/GalleryEditor/GalleryEditorRow.tsx
@@ -11,9 +11,7 @@ import { GalleryEditorRowFragment$key } from '~/generated/GalleryEditorRowFragme
 import { GalleryTouchableOpacity } from '../GalleryTouchableOpacity';
 import { GalleryEditorActiveActions } from './GalleryEditorActiveActions';
 import { GalleryEditorTokenPreview } from './GalleryEditorTokenPreview';
-
-const horizontalRowPadding = 16;
-const inBetweenColumnPadding = 0;
+import { horizontalRowPadding, inBetweenColumnPadding } from './utils';
 
 type Props = {
   sectionId: string;

diff --git a/apps/mobile/src/components/GalleryEditor/SortableRowList.tsx b/apps/mobile/src/components/GalleryEditor/SortableRowList.tsx
@@ -61,33 +61,31 @@ export function SortableRowList({ rows, sectionId, queryRef, onDragEnd }: Props)
   }, [rowOffsets]);
 
   return (
-    <>
-      <View
-        className="relative"
-        style={[
-          {
-            height: containerHeight,
-            top: 0,
-            left: 0,
-            right: 0,
-          },
-        ]}
-      >
-        {rows.map((row, index) => {
-          return (
-            <SortableRow
-              key={`${row.id}`}
-              index={index}
-              positions={positions}
-              animatedIndex={animatedIndex}
-              itemHeights={itemHeights}
-              onDragEnd={onDragEnd}
-            >
-              <GalleryEditorRow sectionId={sectionId} row={row} queryRef={query} />
-            </SortableRow>
-          );
-        })}
-      </View>
-    </>
+    <View
+      className="relative"
+      style={[
+        {
+          height: containerHeight,
+          top: 0,
+          left: 0,
+          right: 0,
+        },
+      ]}
+    >
+      {rows.map((row, index) => {
+        return (
+          <SortableRow
+            key={`${row.id}`}
+            index={index}
+            positions={positions}
+            animatedIndex={animatedIndex}
+            itemHeights={itemHeights}
+            onDragEnd={onDragEnd}
+          >
+            <GalleryEditorRow sectionId={sectionId} row={row} queryRef={query} />
+          </SortableRow>
+        );
+      })}
+    </View>
   );
 }
diff --git a/apps/mobile/src/components/GalleryEditor/utils.ts b/apps/mobile/src/components/GalleryEditor/utils.ts
@@ -7,8 +7,8 @@ type Offset = {
   height: number;
 };
 
-const horizontalRowPadding = 16;
-const inBetweenColumnPadding = 0;
+export const horizontalRowPadding = 16;
+export const inBetweenColumnPadding = 0;
 
 export function getRowHeight(row: StagedRow, screenDimensionsWidth: number) {
   const column = row.columns;