Update tab_gallery_sorter.py

tab_gallery_sorter.py

@@ -13,11 +13,11 @@ def render(quality, profile_name):
     profiles = SorterEngine.load_profiles()
     p_data = profiles.get(profile_name, {})
     
+    # Path Inputs
     c1, c2 = st.columns(2)
     path_s = c1.text_input("Source Folder", value=p_data.get("tab5_source", "/storage"), key="t5_s")
     path_o = c2.text_input("Output Folder", value=p_data.get("tab5_out", "/storage"), key="t5_o")
     
-    # Save Path Button
     if path_s != p_data.get("tab5_source") or path_o != p_data.get("tab5_out"):
         if st.button("💾 Save Settings"):
             SorterEngine.save_tab_paths(profile_name, t5_s=path_s, t5_o=path_o)
@@ -26,7 +26,7 @@ def render(quality, profile_name):
     # 3. Sidebar: Categories
     cats = SorterEngine.get_categories()
     if not cats:
-        st.warning("No categories found. Please add some in the Sidebar or Tab 4.")
+        st.warning("No categories found. Please add some in the Sidebar.")
         return
     selected_cat = st.sidebar.radio("Active Tag", cats, key="t5_cat_radio")
 
@@ -35,7 +35,6 @@ def render(quality, profile_name):
         st.info("Enter a valid source path.")
         return
 
-    # Fetch all images, but we will only display a slice of them
     all_images = SorterEngine.get_images(path_s, recursive=True)
     staged = SorterEngine.get_staged_data()
     
@@ -43,39 +42,44 @@ def render(quality, profile_name):
         st.info("No images found in this folder.")
         return
 
+    # --- VIEW SETTINGS ---
+    # New Slider for Grid Size
+    with st.expander("👀 View Settings", expanded=False):
+        page_size = st.slider("Images per Page", min_value=12, max_value=100, value=24, step=4)
+
     # --- PAGINATION LOGIC ---
-    PAGE_SIZE = 24
     total_items = len(all_images)
-    total_pages = math.ceil(total_items / PAGE_SIZE)
+    total_pages = math.ceil(total_items / page_size)
     
-    # Ensure page number is valid
+    # Bounds Check
-    if st.session_state.t5_page >= total_pages: st.session_state.t5_page = total_pages - 1
+    if st.session_state.t5_page >= total_pages: st.session_state.t5_page = max(0, total_pages - 1)
-    if st.session_state.t5_page < 0: st.session_state.t5_page = 0
     
-    start_idx = st.session_state.t5_page * PAGE_SIZE
+    start_idx = st.session_state.t5_page * page_size
-    end_idx = start_idx + PAGE_SIZE
+    end_idx = start_idx + page_size
     current_batch = all_images[start_idx:end_idx]
 
-    # --- PAGINATION CONTROLS (TOP) ---
+    # --- HELPER: Pagination Controls ---
-    cp1, cp2, cp3 = st.columns([1, 2, 1])
+    def show_pagination_controls(key_suffix):
-    with cp1:
+        cp1, cp2, cp3 = st.columns([1, 2, 1])
-        if st.button("⬅️ Previous", disabled=(st.session_state.t5_page == 0), key="prev_top"):
+        with cp1:
-            st.session_state.t5_page -= 1
+            if st.button("⬅️ Previous", disabled=(st.session_state.t5_page == 0), key=f"prev_{key_suffix}"):
-            st.rerun()
+                st.session_state.t5_page -= 1
-    with cp2:
+                st.rerun()
-        st.markdown(f"<h4 style='text-align: center;'>Page {st.session_state.t5_page + 1} of {total_pages}</h4>", unsafe_allow_html=True)
+        with cp2:
-    with cp3:
+            st.markdown(f"<h5 style='text-align: center;'>Page {st.session_state.t5_page + 1} of {total_pages}</h5>", unsafe_allow_html=True)
-        if st.button("Next ➡️", disabled=(st.session_state.t5_page >= total_pages - 1), key="next_top"):
+        with cp3:
-            st.session_state.t5_page += 1
+            if st.button("Next ➡️", disabled=(st.session_state.t5_page >= total_pages - 1), key=f"next_{key_suffix}"):
-            st.rerun()
+                st.session_state.t5_page += 1
+                st.rerun()
 
+    # 1. TOP CONTROLS
+    show_pagination_controls("top")
     st.divider()
 
     # --- GRID DISPLAY ---
     cols = st.columns(4)
     for idx, img_path in enumerate(current_batch):
-        # Create a unique key based on the actual file path hash + page index
+        # Unique key combines page number and index to avoid conflicts
-        # This prevents buttons from confusing each other
         unique_key = f"{st.session_state.t5_page}_{idx}"
         
         with cols[idx % 4]:
@@ -104,10 +108,8 @@ def render(quality, profile_name):
                 if not is_staged:
                     if st.button("Tag", key=f"tag_{unique_key}", use_container_width=True):
                         ext = os.path.splitext(img_path)[1]
-                        # Determine suffix
                         count = len([v for v in staged.values() if v['cat'] == selected_cat]) + 1
                         new_name = f"{selected_cat}_{count:03d}{ext}"
-                        
                         SorterEngine.stage_image(img_path, selected_cat, new_name)
                         st.rerun()
                 else:
@@ -116,6 +118,11 @@ def render(quality, profile_name):
                         st.rerun()
 
     st.divider()
+    
+    # 2. BOTTOM CONTROLS
+    show_pagination_controls("bottom")
+    
+    st.divider()
 
     # --- APPLY SECTION ---
     c_act1, c_act2 = st.columns([3, 1])
@@ -125,5 +132,5 @@ def render(quality, profile_name):
         with st.spinner("Moving files..."):
             SorterEngine.commit_staging(path_o, cleanup, source_root=path_s)
         st.success("Batch Complete!")
-        st.session_state.t5_page = 0 # Reset to first page after moving
+        st.session_state.t5_page = 0 
         st.rerun()