Release v1.2: Fix unknown schematic persistence & streamline mobile preview UX

Author: mnchetra

README.md

@@ -36,6 +36,19 @@ A Mindustry Java mod that introduces a customizable, floating **Quick Access HUD
 
 ---
 
+## 🆕 What's New in v1.2
+
+### 🐛 Bug Fixes:
+- **Unknown/Unnamed Schematic Persistence**: Fixed an issue where schematics with default `"unknown"` names, missing tags, or in-game copied selections were lost when closing and re-opening the game. Schematics without library files are now serialized cleanly using Base64.
+- **Duplicate Collision Fix for Unnamed Schematics**: Improved schematic slot comparison (`isSameSchematic`) to perform deep tile structure checks, preventing unnamed or copied schematics of the same size from falsely colliding as duplicates.
+
+### ✨ New Features & UX Enhancements:
+- **Instant 1-Step Mobile Preview**: Long-pressing a slot on Mobile (or right-clicking on PC) now opens the **Schematic Preview Dialog** directly in 1 single step, displaying a full visual layout, tile counts, and item build costs.
+- **All-in-One Preview & Slot Management**: Added a `🔄 Replace` button directly inside the Schematic Preview Dialog alongside `⚡ Select & Use`, `🎨 Icon`, and `❌ Clear`.
+- **Improved Mobile Picker UX**: Enlarged info button touch targets (34px) and added long-press gesture support on schematic library cards to instantly preview any schematic before assigning it.
+
+---
+
 ## 🛠️ Building & Installation
 
 ### Building for Desktop Testing

mod.hjson

@@ -3,7 +3,7 @@ name: "quick-schema"
 author: "Seven"
 main: "quickschema.QuickSchemaMod"
 description: "Quick Access Favorites Bar for Mindustry Schematics! Works seamlessly on both PC and Mobile. If there an issue leave it on github."
-version: "1.1"
+version: "1.2"
 minGameVersion: 158
 java: true
 hidden: true

src/quickschema/QuickSlotManager.java

@@ -38,7 +38,7 @@ public static void load() {
                         String b64 = data.substring(4);
                         s = Vars.schematics.readBase64(b64);
                     }
-                    if (s != null && !contains(s)) {
+                    if (s != null) {
                         slots.add(s);
                         slotIcons.add(iconKey);
                     }
@@ -55,18 +55,26 @@ public static void save() {
             if (i < slots.size) {
                 Schematic s = slots.get(i);
                 String iconKey = i < slotIcons.size ? slotIcons.get(i) : "";
-                if (s.file != null) {
-                    Core.settings.put("quickschema_slot_" + i, "file:" + s.file.name());
-                } else if (s.name() != null && !s.name().isEmpty()) {
-                    Core.settings.put("quickschema_slot_" + i, "name:" + s.name());
-                } else {
-                    try {
-                        String b64 = Vars.schematics.writeBase64(s);
-                        Core.settings.put("quickschema_slot_" + i, "b64:" + b64);
-                    } catch (Throwable t) {
-                        Core.settings.put("quickschema_slot_" + i, "");
+                String saveData = "";
+                if (s != null) {
+                    if (s.file != null && Vars.schematics.all().find(sc -> sc.file != null && sc.file.name().equals(s.file.name())) != null) {
+                        saveData = "file:" + s.file.name();
+                    } else if (s.name() != null && !s.name().isEmpty() 
+                            && !s.name().equalsIgnoreCase("unknown") 
+                            && !s.name().startsWith("Copied ")
+                            && Vars.schematics.all().find(sc -> sc.name().equals(s.name())) != null) {
+                        saveData = "name:" + s.name();
+                    } else {
+                        try {
+                            String b64 = Vars.schematics.writeBase64(s);
+                            saveData = "b64:" + b64;
+                        } catch (Throwable t) {
+                            Log.err("Failed to write base64 for QuickSchema slot " + i, t);
+                            saveData = "";
+                        }
                     }
                 }
+                Core.settings.put("quickschema_slot_" + i, saveData);
                 Core.settings.put("quickschema_slot_icon_" + i, iconKey);
             } else {
                 Core.settings.put("quickschema_slot_" + i, "");
@@ -154,14 +162,46 @@ public static boolean addSchematic(Schematic schematic) {
         return true;
     }
 
+    public static boolean isSameSchematic(Schematic a, Schematic b) {
+        if (a == b) return true;
+        if (a == null || b == null) return false;
+
+        if (a.file != null && b.file != null) {
+            return a.file.name().equals(b.file.name());
+        }
+
+        String nameA = a.name();
+        String nameB = b.name();
+        boolean validNameA = nameA != null && !nameA.isEmpty() && !nameA.equalsIgnoreCase("unknown") && !nameA.startsWith("Copied ");
+        boolean validNameB = nameB != null && !nameB.isEmpty() && !nameB.equalsIgnoreCase("unknown") && !nameB.startsWith("Copied ");
+
+        if (validNameA && validNameB) {
+            return nameA.equals(nameB) && a.width == b.width && a.height == b.height;
+        }
+
+        if (a.width != b.width || a.height != b.height) return false;
+        if (a.tiles == null || b.tiles == null) return a.tiles == b.tiles;
+        if (a.tiles.size != b.tiles.size) return false;
+
+        for (int i = 0; i < a.tiles.size; i++) {
+            Schematic.Stile t1 = a.tiles.get(i);
+            Schematic.Stile t2 = b.tiles.get(i);
+            if (t1.block != t2.block || t1.x != t2.x || t1.y != t2.y || t1.rotation != t2.rotation) {
+                return false;
+            }
+            if (t1.config != null ? !t1.config.equals(t2.config) : t2.config != null) {
+                return false;
+            }
+        }
+
+        return true;
+    }
+
     public static int findSlot(Schematic schematic) {
         if (schematic == null) return -1;
         for (int i = 0; i < slots.size; i++) {
             Schematic s = slots.get(i);
-            if (s == null) continue;
-            if (s == schematic) return i;
-            if (s.file != null && schematic.file != null && s.file.name().equals(schematic.file.name())) return i;
-            if (s.name() != null && s.name().equals(schematic.name()) && s.width == schematic.width && s.height == schematic.height) return i;
+            if (isSameSchematic(s, schematic)) return i;
         }
         return -1;
     }

src/quickschema/ui/QuickBarUI.java

@@ -116,14 +116,13 @@ private void setup() {
                          .color(arc.graphics.Color.lightGray).fontScale(0.85f);
                     }));
 
-                    // Mouse Right Click (PC)
-                    btn.clicked(KeyCode.mouseRight, () -> showSlotMenu(index, s));
+                    // Mouse Right Click (PC) & Touch Long Press (Mobile) directly opens visual preview & slot controls
+                    btn.clicked(KeyCode.mouseRight, () -> new SchematicPreviewDialog(s, index, this::setup).show());
 
-                    // Touch Long Press / Hold Gesture (Mobile)
                     btn.addListener(new ElementGestureListener() {
                         @Override
                         public boolean longPress(Element element, float x, float y) {
-                            showSlotMenu(index, s);
+                            new SchematicPreviewDialog(s, index, QuickBarUI.this::setup).show();
                             return true;
                         }
                     });

src/quickschema/ui/SchematicPickerDialog.java

@@ -106,7 +106,7 @@ private void rebuildList(Table listTable) {
 
             header.button(Icon.info, Styles.clearNonei, () -> {
                 new SchematicPreviewDialog(s, -1, onSelect).show();
-            }).size(26f).padRight(4f);
+            }).size(34f).padRight(4f);
 
             String displayName = s.name();
             if (displayName.length() > 12) {
@@ -151,15 +151,23 @@ private void rebuildList(Table listTable) {
                 .color(Color.lightGray).fontScale(0.75f).center();
             card.add(footer).pad(2f).row();
 
-            // Click card button (proper dynamic sizing to avoid card overlapping)
-            listTable.button(btn -> btn.add(card).growX(), Styles.cleart, () -> {
+            // Click card to assign, long-press card to preview directly on Mobile
+            var cardBtn = listTable.button(btn -> btn.add(card).growX(), Styles.cleart, () -> {
                 boolean success = QuickSlotManager.setSlot(targetSlot, s);
                 if (success) {
                     Vars.ui.showInfoToast("Assigned [" + s.name() + "] to Quick Slot " + (targetSlot + 1), 2f);
                     hide();
                     if (onSelect != null) onSelect.run();
                 }
-            }).pad(3f).growX();
+            }).pad(3f).growX().get();
+
+            cardBtn.addListener(new arc.scene.event.ElementGestureListener() {
+                @Override
+                public boolean longPress(arc.scene.Element element, float x, float y) {
+                    new SchematicPreviewDialog(s, -1, onSelect).show();
+                    return true;
+                }
+            });
 
             colCount++;
             if (colCount % cols == 0) {

src/quickschema/ui/SchematicPreviewDialog.java

@@ -134,6 +134,11 @@ private void setup() {
                 }).show();
             }).size(110f, 50f).pad(4f);
 
+            buttons.button("🔄 Replace", Icon.refresh, Styles.cleart, () -> {
+                hide();
+                new SchematicPickerDialog(slotIndex, onUpdate).show();
+            }).size(110f, 50f).pad(4f);
+
             buttons.button("❌ Clear", Icon.cancel, Styles.cleart, () -> {
                 QuickSlotManager.clearSlot(slotIndex);
                 if (onUpdate != null) onUpdate.run();