If I'm reading the code aright, looks like this is handled in  
mouseUp:, and mouseUp: doesn't seem to be called at all.

In a quick test project with a PDFView, mouseUp: works fine until a  
document is loaded, so one of the views inside PDFView is snarfing  
events and fouling things up.  Likewise, mouseDragged: is never  
called.  I tried using my hack for NSSplitView, but that made no  
difference in the test project.

Oh, I also noticed this

                 // due to an Appkit bug, endUndoGrouping registers an  
extra change count, which is not reverted when the group is undone
                 [[[[self window] windowController] document]  
updateChangeCount:NSChangeUndone];

which may be fixed on Leopard if my vague recollection is correct.   
Maybe the change count should be checked first?

-- 
adam

On Oct 30, 2007, at 12:37 PM, Christiaan Hofman wrote:

> Adam, could you check if PDFView uses an event loop for selection on
> Leopard? Or otherwise if this could be fixed by calling [super
> mouseDown:] before adding the highlight? (just some random thoughts).
>
> Christiaan
>
> On 30 Oct 2007, at 8:07 PM, Marian Dörk wrote:
>
>> Hello,
>>
>> it seems as if highlighting with the Highlight tool is broken on
>> Leopard. Highlighting though works when text is selected with the  
>> Text
>> Tool and then marked up via the New Highlight command from the Notes
>> menu.
>>
>> Are other users experiencing the same? I just added a bug tracker
>> entry, too.
>>
>> Thanks
>> Marian
>>
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems?  Stop.
> Now Search log events and configuration files using AJAX and a  
> browser.
> Download your FREE copy of Splunk now >> http://get.splunk.com/
> _______________________________________________
> skim-app-develop mailing list
> skim-app-develop@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/skim-app-develop


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
skim-app-develop mailing list
skim-app-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/skim-app-develop

Reply via email to