Revision: 16348
http://sourceforge.net/p/skim-app/code/16348
Author: hofman
Date: 2026-06-02 17:04:17 +0000 (Tue, 02 Jun 2026)
Log Message:
-----------
no need to clear selection after adding markup note, is already done
Modified Paths:
--------------
trunk/SKPDFView.m
Modified: trunk/SKPDFView.m
===================================================================
--- trunk/SKPDFView.m 2026-06-02 16:57:14 UTC (rev 16347)
+++ trunk/SKPDFView.m 2026-06-02 17:04:17 UTC (rev 16348)
@@ -1730,12 +1730,10 @@
[self setCurrentAnnotation:nil];
BOOL mayMultiClick = [theEvent clickCount] < 3 && [NSApp
willDragMouse] == NO;
[super mouseDown:theEvent];
- if ([[self currentSelection] hasCharacters]) {
+ if ([[self currentSelection] hasCharacters])
[self
addAnnotationWithType:NOTE_TYPE_FROM_TEMP_TOOL_MODE(temporaryToolMode)];
- [self setCurrentSelection:nil];
- } else if (mayMultiClick) {
+ else if (mayMultiClick)
delayTempToolMode = YES;
- }
}
if (delayTempToolMode)
[self performSelectorOnce:@selector(resetTemporaryToolMode)
afterDelay:[NSEvent doubleClickInterval]];
@@ -1785,10 +1783,8 @@
} else if ([self doDragSelectedTextWithEvent:theEvent] == NO) {
[self setCurrentAnnotation:nil];
[super mouseDown:theEvent];
- if (toolMode == SKToolModeNote && [self canAddNotes] &&
IS_MARKUP(annotationMode) && [[self currentSelection] hasCharacters]) {
+ if (toolMode == SKToolModeNote && [self canAddNotes] &&
IS_MARKUP(annotationMode) && [[self currentSelection] hasCharacters])
[self addAnnotationWithType:annotationMode];
- [self setCurrentSelection:nil];
- }
}
}
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
