From 7122d44b134a39e91e8210648738ff4125ed9114 Mon Sep 17 00:00:00 2001
From: ousugo <dkzyxh@gmail.com>
Date: Wed, 12 Mar 2025 14:26:52 +0800
Subject: [PATCH] refactor(PaintingsList): Reposition new painting button

Move the NewPaintingButton component to be rendered before the DragableList and remove unnecessary margin-top styling
---
 src/renderer/src/pages/paintings/PaintingsList.tsx | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/renderer/src/pages/paintings/PaintingsList.tsx b/src/renderer/src/pages/paintings/PaintingsList.tsx
index 44dc42aa..7ebf67d9 100644
--- a/src/renderer/src/pages/paintings/PaintingsList.tsx
+++ b/src/renderer/src/pages/paintings/PaintingsList.tsx
@@ -31,6 +31,11 @@ const PaintingsList: FC<PaintingsListProps> = ({
 
   return (
     <Container style={{ paddingBottom: dragging ? 80 : 10 }}>
+      {!dragging && (
+        <NewPaintingButton onClick={onNewPainting}>
+          <PlusOutlined />
+        </NewPaintingButton>
+      )}
       <DragableList
         list={paintings}
         onUpdate={updatePaintings}
@@ -55,11 +60,6 @@ const PaintingsList: FC<PaintingsListProps> = ({
           </CanvasWrapper>
         )}
       </DragableList>
-      {!dragging && (
-        <NewPaintingButton onClick={onNewPainting}>
-          <PlusOutlined />
-        </NewPaintingButton>
-      )}
     </Container>
   )
 }
@@ -134,7 +134,6 @@ const NewPaintingButton = styled.div`
   width: 80px;
   height: 80px;
   min-height: 80px;
-  margin-top: -10px;
   background-color: var(--color-background-soft);
   cursor: pointer;
   transition: background-color 0.2s ease;