Revision: 15704
          http://sourceforge.net/p/skim-app/code/15704
Author:   hofman
Date:     2025-10-24 18:21:11 +0000 (Fri, 24 Oct 2025)
Log Message:
-----------
add some comments

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

Modified: trunk/SKBasePDFView.m
===================================================================
--- trunk/SKBasePDFView.m       2025-10-24 18:07:24 UTC (rev 15703)
+++ trunk/SKBasePDFView.m       2025-10-24 18:21:11 UTC (rev 15704)
@@ -390,6 +390,7 @@
     [super goToDestination:destination];
 }
 
+// this is bound to PageDown in PDFView
 - (void)scrollPageUp:(id)sender {
     NSScrollView *scrollView;
     NSClipView *clipView = nil;
@@ -419,10 +420,12 @@
         [clipView scrollToPoint:bounds.origin];
         [scrollView reflectScrolledClipView:clipView];
     } else if (page && [[self currentPage] pageIndex] > [page pageIndex]) {
+        // Apple scrolls to the bottom of the next page rather than the top
         [self verticallyScrollToTop];
     }
 }
 
+// this is bound to PageUp in PDFView
 - (void)scrollPageDown:(id)sender {
     NSScrollView *scrollView;
     NSClipView *clipView = nil;
@@ -451,20 +454,25 @@
         [clipView scrollToPoint:bounds.origin];
         [scrollView reflectScrolledClipView:clipView];
     } else if (page && [[self currentPage] pageIndex] < [page pageIndex]) {
+        // Apple scrolls to the top of the next page rather than the bottom
         [self verticallyScrollToBottom];
     }
 }
 
+// this is bound to ArrowDown in PDFView
 - (void)scrollLineUp:(id)sender {
     PDFPage *page = ([self displayMode] & kPDFDisplaySinglePageContinuous) ? 
nil : [self currentPage];
     [super scrollLineUp:sender];
+    // Apple scrolls to the bottom of the next page rather than the top
     if (page && [[self currentPage] pageIndex] > [page pageIndex])
         [self verticallyScrollToTop];
 }
 
+// this is bound to ArrowUp in PDFView
 - (void)scrollLineDown:(id)sender {
     PDFPage *page = ([self displayMode] & kPDFDisplaySinglePageContinuous) ? 
nil : [self currentPage];
     [super scrollLineDown:sender];
+    // Apple scrolls to the top of the next page rather than the bottom
     if (page && [[self currentPage] pageIndex] < [page pageIndex])
         [self verticallyScrollToBottom];
 }

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