Revision: 15380
          http://sourceforge.net/p/skim-app/code/15380
Author:   hofman
Date:     2025-06-05 22:28:30 +0000 (Thu, 05 Jun 2025)
Log Message:
-----------
no need to set shouldDisplay an dshouldPrint for notes added in bulk

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

Modified: trunk/SKMainWindowController.m
===================================================================
--- trunk/SKMainWindowController.m      2025-06-05 21:37:13 UTC (rev 15379)
+++ trunk/SKMainWindowController.m      2025-06-05 22:28:30 UTC (rev 15380)
@@ -2201,11 +2201,11 @@
     } else {
         [[undoManager prepareWithInvocationTarget:[notification object]] 
removeAnnotation:annotation];
         
-        if ([annotation isSkimNote]) {
-            [annotation setShouldDisplay:[pdfView hideNotes] == NO];
-            [annotation setShouldPrint:[pdfView hideNotes] == NO];
-        }
         if ([annotation isSkimNote] && mwcFlags.addOrRemoveNotesInBulk == 0) {
+            if ([annotation isSkimNote]) {
+                [annotation setShouldDisplay:[pdfView hideNotes] == NO];
+                [annotation setShouldPrint:[pdfView hideNotes] == NO];
+            }
             mwcFlags.updatingNoteSelection = 1;
             [self insertObject:annotation inNotesAtIndex:[notes count]];
             [rightSideController.noteArrayController rearrangeObjects]; // 
doesn't seem to be done automatically

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



_______________________________________________
Skim-app-commit mailing list
Skim-app-commit@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/skim-app-commit

Reply via email to