Use flex: 0.5 for every action + use consistent variable naming

observation · May 6, 2024 · a3713a6 · a3713a6
1 parent a57afda
commit a3713a6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/components/Lightbox.tsx b/src/components/Lightbox.tsx
@@ -86,15 +86,15 @@ type Props = {
 }
 
 const Lightbox = ({ index, onClose, onDelete, photos, title, description, content, style }: Props) => {
-  const initialIndex = index ?? 0
+  const initialImageIndex = index ?? 0
   const [currentImageIndex, setCurrentImageIndex] = useState<number>()
 
-  const onPressDelete = onDelete ? () => onDelete(currentImageIndex ?? initialIndex) : undefined
+  const onPressDelete = onDelete ? () => onDelete(currentImageIndex ?? initialImageIndex) : undefined
 
   return (
     <ImageView
       images={photos.map((photo) => ({ uri: photo }))}
-      imageIndex={initialIndex}
+      imageIndex={initialImageIndex}
       visible={index !== undefined}
       swipeToCloseEnabled={false}
       onImageIndexChange={setCurrentImageIndex}
@@ -153,7 +153,7 @@ const styles = StyleSheet.create({
     marginHorizontal: theme.margin.common,
   },
   buttonContainer: {
-    width: '50%',
+    flex: 0.5,
     alignItems: 'center',
   },
 })
diff --git a/src/components/__tests__/__snapshots__/Lightbox.test.tsx.snap b/src/components/__tests__/__snapshots__/Lightbox.test.tsx.snap
@@ -1967,7 +1967,7 @@ exports[`Lightbox Rendering With a delete button 1`] = `
               style={
                 {
                   "alignItems": "center",
-                  "width": "50%",
+                  "flex": 0.5,
                 }
               }
             >