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