snapexcel 1.2.5 → 1.2.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "snapexcel",
-  "version": "1.2.5",
+  "version": "1.2.7",
   "description": "Desktop Screenshot Logger for UI Testing - Capture any window including Android emulators",
   "main": "src/main.js",
   "bin": {

package/publish.sh

@@ -0,0 +1,5 @@
+#!/bin/bash
+
+# SnapExcel - Publish to npm script
+
+git add . && git commit -m "fix: responsive layout, prevent focus stealing, UI improvements" && npm version patch && npm publish

package/src/main.js

@@ -74,14 +74,15 @@ ipcMain.handle('get-always-on-top', () => {
 // Capture entire screen at native resolution (optimized for 250 screenshot capacity)
 ipcMain.handle('capture-screen', async () => {
   try {
-    // Hide the main window before capturing so it doesn't appear in screenshot
-    if (mainWindow) {
+    // Only hide window if it's visible and not minimized (to avoid pulling user back)
+    const shouldHideWindow = mainWindow && mainWindow.isVisible() && !mainWindow.isMinimized();
+
+    if (shouldHideWindow) {
       mainWindow.hide();
+      // Small delay to ensure window is fully hidden
+      await new Promise(resolve => setTimeout(resolve, 150));
     }
 
-    // Small delay to ensure window is fully hidden
-    await new Promise(resolve => setTimeout(resolve, 150));
-
     // Get actual screen size for full resolution capture
     const primaryDisplay = screen.getPrimaryDisplay();
     const { width, height } = primaryDisplay.size;
@@ -96,9 +97,9 @@ ipcMain.handle('capture-screen', async () => {
       thumbnailSize: { width: captureWidth, height: captureHeight }
     });
 
-    // Show the main window again after capture
-    if (mainWindow) {
-      mainWindow.show();
+    // Only show window again if we hid it (don't disturb user on other screens)
+    if (shouldHideWindow && mainWindow) {
+      mainWindow.showInactive(); // Show without stealing focus
       if (isAlwaysOnTop) {
         mainWindow.setAlwaysOnTop(true);
       }
@@ -113,8 +114,8 @@ ipcMain.handle('capture-screen', async () => {
     return { success: false, error: 'No screen found' };
   } catch (error) {
     // Make sure to show window even if there's an error
-    if (mainWindow) {
-      mainWindow.show();
+    if (mainWindow && !mainWindow.isMinimized()) {
+      mainWindow.showInactive();
     }
     return { success: false, error: error.message };
   }
@@ -194,16 +195,20 @@ ipcMain.handle('start-region-capture', async () => {
         regionWindow.webContents.send('set-background', fullScreenshot, display.size);
       });
 
-      // Function to restore main window properly
+      // Function to restore main window without stealing focus
       const restoreMainWindow = () => {
-        if (mainWindow && mainWindowBounds) {
-          // Use setTimeout to let the region window fully close first
+        if (mainWindow) {
+          // Just restore the window state without bringing it to foreground
+          // This prevents pulling user to SnapExcel window when they're on different screen
           setTimeout(() => {
-            mainWindow.restore();
-            mainWindow.setSize(mainWindowBounds.width, mainWindowBounds.height);
-            mainWindow.setPosition(mainWindowBounds.x, mainWindowBounds.y);
-            // Don't focus - let user continue with their app
-            mainWindow.blur();
+            if (mainWindow.isMinimized()) {
+              // Use showInactive to restore without stealing focus
+              mainWindow.showInactive();
+            }
+            // Don't change size/position - keep window where it was
+            if (isAlwaysOnTop) {
+              mainWindow.setAlwaysOnTop(true);
+            }
           }, 100);
         }
       };

package/src/renderer.js

@@ -733,6 +733,9 @@ async function captureScreenshot() {
 
       state.sheets[state.currentSheetIndex].screenshots.push(screenshot);
       saveState();
+
+      // Small delay to let window fully restore before updating UI
+      await new Promise(resolve => setTimeout(resolve, 100));
       updateUI();
       lastCaptureTime = Date.now();
 
@@ -798,6 +801,10 @@ async function captureRegion() {
 
       state.sheets[state.currentSheetIndex].screenshots.push(screenshot);
       saveState();
+
+      // Small delay to let window fully restore before updating UI
+      await new Promise(resolve => setTimeout(resolve, 100));
+
       // Don't call updateUI() which might resize the window - just update the grid
       updateScreenshotsGrid(state.sheets[state.currentSheetIndex].screenshots);
       elements.screenshotCount.textContent = state.sheets[state.currentSheetIndex].screenshots.length;

package/src/styles.css

@@ -47,8 +47,7 @@ body {
 .app-container {
   display: flex;
   flex-direction: column;
-  min-height: 100vh;
-  max-height: 100vh;
+  height: 100vh;
   overflow: hidden;
 }
 
@@ -159,7 +158,9 @@ body {
   display: flex;
   flex-direction: column;
   overflow-y: auto;
+  overflow-x: hidden;
   padding: 16px;
+  flex: 1;
 }
 
 .view.hidden {
@@ -534,6 +535,9 @@ body {
   border-radius: 5px;
   border: 0.5px solid rgba(4, 30, 66, 0.04);
   padding: 5px;
+  flex: 1;
+  min-height: 100px;
+  overflow: hidden;
 }
 
 .section-header {
@@ -559,14 +563,16 @@ body {
 
 .screenshots-grid {
   display: grid;
-  grid-template-columns: repeat(4, 1fr);
+  grid-template-columns: repeat(auto-fill, minmax(120px, 1fr));
+  grid-auto-rows: min-content;
+  align-content: start;
   gap: 8px;
   overflow-y: auto;
   padding: 5px;
   background: rgba(255, 255, 255, 0.5);
   border-radius: 3px;
+  flex: 1;
   min-height: 70px;
-  max-height: 200px;
 }
 
 /* Ultra thin scrollbar - both vertical and horizontal */