[Skim-app-commit] SF.net SVN: skim-app:[15605] trunk/SKMainWindowController.m

Revision: 15605
          http://sourceforge.net/p/skim-app/code/15605
Author:   hofman
Date:     2025-07-04 16:25:53 +0000 (Fri, 04 Jul 2025)
Log Message:
-----------
static set for note keeys that don't affect display

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

Modified: trunk/SKMainWindowController.m
===================================================================
--- trunk/SKMainWindowController.m	2025-07-04 15:24:39 UTC (rev 15604)
+++ trunk/SKMainWindowController.m	2025-07-04 16:25:53 UTC (rev 15605)
@@ -2766,8 +2766,12 @@
             if ([oldNoteProperties objectForKey:keyPath] == nil)
                 [oldNoteProperties setObject:oldValue forKey:keyPath];
             
+            static NSSet *invisibleKeys = nil;
+            if (invisibleKeys == nil)
+                invisibleKeys = [[NSSet alloc] initWithObjects:SKNPDFAnnotationModificationDateKey, SKNPDFAnnotationUserNameKey, SKNPDFAnnotationTextKey, nil];
+            
             // Update the UI, we should always do that unless the value did not really change or we're just changing the mod date or user name
-            if ([keyPath isEqualToString:SKNPDFAnnotationModificationDateKey] == NO && [keyPath isEqualToString:SKNPDFAnnotationUserNameKey] == NO && [keyPath isEqualToString:SKNPDFAnnotationTextKey] == NO && ([note isText] || [keyPath isEqualToString:SKNPDFAnnotationStringKey] == NO)) {
+            if ([invisibleKeys containsObject:keyPath] == NO && ([note isText] || [keyPath isEqualToString:SKNPDFAnnotationStringKey] == NO)) {
                 PDFPage *page = [note page];
                 
                 [self updateThumbnailAtPageIndex:[note pageIndex]];

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