Revision: 29141 http://sourceforge.net/p/bibdesk/svn/29141 Author: hofman Date: 2025-04-19 16:32:39 +0000 (Sat, 19 Apr 2025) Log Message: ----------- revert previous commit, I had it wrong
Modified Paths: -------------- trunk/bibdesk_vendorsrc/amaxwell/FileView/FVFileView.m Modified: trunk/bibdesk_vendorsrc/amaxwell/FileView/FVFileView.m =================================================================== --- trunk/bibdesk_vendorsrc/amaxwell/FileView/FVFileView.m 2025-04-19 16:27:41 UTC (rev 29140) +++ trunk/bibdesk_vendorsrc/amaxwell/FileView/FVFileView.m 2025-04-19 16:32:39 UTC (rev 29141) @@ -165,9 +165,11 @@ @interface FVDropHighlightView : FVDisplayView { NSUInteger _dropIndex; FVDropOperation _dropOperation; + BOOL _dragHasURLs; } @property (nonatomic) NSUInteger dropIndex; @property (nonatomic) FVDropOperation dropOperation; +@property (nonatomic) BOOL dragHasURLs; @end #pragma mark - @@ -2834,10 +2836,7 @@ // We won't reset the drop location info when we propose NSDragOperationNone, because the delegate may want to override our decision, we will reset it at the end - // !!! this is quite expensive to call repeatedly in -draggingUpdated - BOOL dragHasURLs = FVPasteboardHasURL([sender draggingPasteboard]); - - if (dragHasURLs == NO) { + if ([_dropHighlightView dragHasURLs] == NO) { // We have to make sure the pasteboard really has a URL here, since most NSStrings aren't valid URLs, but the delegate may accept other types dragOp = NSDragOperationNone; } @@ -2868,10 +2867,6 @@ dragOp = NSDragOperationLink; } - if (_dropHighlightView == nil) { - _dropHighlightView = [[FVDropHighlightView alloc] initWithFrame:[_contentView bounds] delegate:self]; - [_contentView addSubview:_dropHighlightView]; - } [_dropHighlightView setDropIndex:dropIndex]; [_dropHighlightView setDropOperation:dropOp]; @@ -2892,6 +2887,12 @@ // this is called as soon as the mouse is moved to start a drag, or enters the window from outside - (NSDragOperation)draggingEntered:(id <NSDraggingInfo>)sender { + if (_dropHighlightView == nil) { + _dropHighlightView = [[FVDropHighlightView alloc] initWithFrame:[_contentView bounds] delegate:self]; + [_contentView addSubview:_dropHighlightView]; + } + // !!! this is quite expensive to call repeatedly in -draggingUpdated + [_dropHighlightView setDragHasURLs:FVPasteboardHasURL([sender draggingPasteboard])]; return [self draggingUpdated:sender]; } @@ -5112,6 +5113,7 @@ @synthesize dropIndex=_dropIndex; @synthesize dropOperation=_dropOperation; +@synthesize dragHasURLs=_dragHasURLs; - (id)initWithFrame:(NSRect)frameRect { self = [super initWithFrame:frameRect]; This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. _______________________________________________ Bibdesk-commit mailing list Bibdesk-commit@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bibdesk-commit