Revision: 16433
          http://sourceforge.net/p/skim-app/code/16433
Author:   hofman
Date:     2026-09-07 16:17:15 +0000 (Mon, 07 Sep 2026)
Log Message:
-----------
do set frame from prefs first also when autosaving succeeds, apparently 
defaults may (have been) overwritten when failing

Modified Paths:
--------------
    trunk/NSWindowController_SKExtensions.m

Modified: trunk/NSWindowController_SKExtensions.m
===================================================================
--- trunk/NSWindowController_SKExtensions.m     2026-09-07 16:01:01 UTC (rev 
16432)
+++ trunk/NSWindowController_SKExtensions.m     2026-09-07 16:17:15 UTC (rev 
16433)
@@ -53,13 +53,12 @@
     NSPointPointer pointPtr = (NSPointPointer)NSMapGet(nextWindowLocations, 
(__bridge void *)name);
     NSPoint point;
     
+    [window setFrameUsingName:name];
     [self setShouldCascadeWindows:NO];
-    if ([window setFrameAutosaveName:name]) {
+    if ([window setFrameAutosaveName:name] || pointPtr == NULL)
         point = SKTopLeftPoint([window frame]);
-    } else {
-        [window setFrameUsingName:name];
-        point = pointPtr ? *pointPtr : SKTopLeftPoint([window frame]);
-    }
+    else
+        point = *pointPtr;
     point = [window cascadeTopLeftFromPoint:point];
     NSMapInsert(nextWindowLocations, (__bridge void *)name, &point);
 }

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
Skim-app-commit mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-commit

Reply via email to