[Skim-app-commit] SF.net SVN: skim-app:[16644] trunk/SKReadingBar.m

[hofman--- via Skim-app-commit]

Revision: 16644
          http://sourceforge.net/p/skim-app/code/16644
Author:   hofman
Date:     2026-10-10 21:52:11 +0000 (Sat, 10 Oct 2026)
Log Message:
-----------
method to pass deleggate method

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

Modified: trunk/SKReadingBar.m
===================================================================
--- trunk/SKReadingBar.m        2026-10-10 17:00:32 UTC (rev 16643)
+++ trunk/SKReadingBar.m        2026-10-10 21:52:11 UTC (rev 16644)
@@ -126,6 +126,11 @@
     return rect;
 }
 
+- (void)didChangeBounds:(NSRect)oldBounds onPage:(PDFPage *)oldPage 
scroll:(BOOL)shouldScroll {
+    if (delegate)
+        [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:shouldScroll];
+}
+
 #pragma mark Accessors
 
 - (void)setNumberOfLines:(NSUInteger)number {
@@ -136,8 +141,8 @@
         numberOfLines = number;
         [self setCurrentBounds:[self currentBoundsFromLineRects:nil]];
         [[NSUserDefaults standardUserDefaults] setInteger:numberOfLines 
forKey:SKReadingBarNumberOfLinesKey];
-        if (delegate && NSEqualRects(oldBounds, currentBounds) == NO)
-            [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:NO];
+        if (NSEqualRects(oldBounds, currentBounds) == NO)
+            [self didChangeBounds:oldBounds onPage:oldPage scroll:NO];
     }
 }
 
@@ -198,8 +203,8 @@
     } else if ([self goToNextPageAtTop:YES]) {
         didMove = YES;
     }
-    if (didMove && delegate)
-        [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:YES];
+    if (didMove)
+        [self didChangeBounds:oldBounds onPage:oldPage scroll:YES];
     return didMove;
 }
 
@@ -216,8 +221,8 @@
     } else if ([self goToPreviousPageAtTop:NO]) {
         didMove = YES;
     }
-    if (didMove && delegate)
-        [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:YES];
+    if (didMove)
+        [self didChangeBounds:oldBounds onPage:oldPage scroll:YES];
     return didMove;
 }
 
@@ -225,8 +230,8 @@
     PDFPage *oldPage = currentLine != -1 ? page : nil;
     NSRect oldBounds = currentBounds;
     BOOL didMove = [self goToNextPageAtTop:YES];
-    if (didMove && delegate)
-        [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:YES];
+    if (didMove)
+        [self didChangeBounds:oldBounds onPage:oldPage scroll:YES];
     return didMove;
 }
 
@@ -234,8 +239,8 @@
     PDFPage *oldPage = currentLine != -1 ? page : nil;
     NSRect oldBounds = currentBounds;
     BOOL didMove = [self goToPreviousPageAtTop:YES];
-    if (didMove && delegate)
-        [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:YES];
+    if (didMove)
+        [self didChangeBounds:oldBounds onPage:oldPage scroll:YES];
     return didMove;
 }
 
@@ -253,8 +258,8 @@
     } else {
         [self goToNextPageAtTop:YES] || [self goToPreviousPageAtTop:NO];
     }
-    if (delegate && (page != oldPage || NSEqualRects(oldBounds, currentBounds) 
== NO))
-        [delegate readingBarDidChangeBounds:oldBounds onPage:oldPage 
toBounds:currentBounds onPage:page scroll:shouldScroll];
+    if (page != oldPage || NSEqualRects(oldBounds, currentBounds) == NO)
+        [self didChangeBounds:oldBounds onPage:oldPage scroll:shouldScroll];
 }
 
 - (void)goToLine:(NSInteger)line onPage:(PDFPage *)aPage {

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