Revision: 15127
          http://sourceforge.net/p/skim-app/code/15127
Author:   hofman
Date:     2025-04-13 14:14:56 +0000 (Sun, 13 Apr 2025)
Log Message:
-----------
Validate previous/next/first/last toolbar item subitmes

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

Modified: trunk/SKMainToolbarController.m
===================================================================
--- trunk/SKMainToolbarController.m     2025-04-12 21:33:46 UTC (rev 15126)
+++ trunk/SKMainToolbarController.m     2025-04-13 14:14:56 UTC (rev 15127)
@@ -280,11 +280,15 @@
             [item setMenuFormRepresentation:menuItem];
             
             NSToolbarItem *item1 = [[NSToolbarItem alloc] 
initWithItemIdentifier:@""];
+            [item1 setEnabled:[mainController.pdfView canGoToFirstPage]];
             NSToolbarItem *item2 = [[NSToolbarItem alloc] 
initWithItemIdentifier:@""];
             [item2 setLabels:NSLocalizedString(@"Previous", @"Toolbar item 
label")];
+            [item2 setEnabled:[mainController.pdfView canGoToPreviousPage]];
             NSToolbarItem *item3 = [[NSToolbarItem alloc] 
initWithItemIdentifier:@""];
             [item3 setLabels:NSLocalizedString(@"Next", @"Toolbar item 
label")];
+            [item3 setEnabled:[mainController.pdfView canGoToNextPage]];
             NSToolbarItem *item4 = [[NSToolbarItem alloc] 
initWithItemIdentifier:@""];
+            [item4 setEnabled:[mainController.pdfView canGoToLastPage]];
             [(NSToolbarItemGroup *)item setSubitems:@[item1, item2, item3, 
item4]];
             
         } else if ([identifier 
isEqualToString:SKDocumentToolbarBackForwardItemIdentifier]) {
@@ -1386,6 +1390,11 @@
     NSArray *subitems = [(NSToolbarItemGroup *)[toolbarItems 
objectForKey:SKDocumentToolbarPreviousNextItemIdentifier] subitems];
     [[subitems objectAtIndex:0] setEnabled:[mainController.pdfView 
canGoToPreviousPage]];
     [[subitems objectAtIndex:1] setEnabled:[mainController.pdfView 
canGoToNextPage]];
+    subitems = [(NSToolbarItemGroup *)[toolbarItems 
objectForKey:SKDocumentToolbarPreviousNextFirstLastItemIdentifier] subitems];
+    [[subitems objectAtIndex:0] setEnabled:[mainController.pdfView 
canGoToFirstPage]];
+    [[subitems objectAtIndex:1] setEnabled:[mainController.pdfView 
canGoToPreviousPage]];
+    [[subitems objectAtIndex:2] setEnabled:[mainController.pdfView 
canGoToNextPage]];
+    [[subitems objectAtIndex:3] setEnabled:[mainController.pdfView 
canGoToLastPage]];
 }
 
 - (void)handleScaleChangedNotification:(NSNotification *)notification {

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