On Apr 6, 2008, at 9:02 AM, Christiaan Hofman wrote:
>
> On 6 Apr 2008, at 5:42 PM, Adam R. Maxwell wrote:
>
>>
>> On Apr 6, 2008, at 7:50 AM, Christiaan Hofman wrote:
>>>
>>> On 6 Apr 2008, at 3:47 PM, Ivan Werning wrote:
>>>
>>>> On Apr 6, 2008, at 7:18 AM, Christiaan Hofman wrote:
>>>>>
>>>> The odd thing is that all these other programs like Yep,  
>>>> EagleFiler,
>>>> Papers, Yojimbo, seem to work with the zoom pinching even though
>>>> they
>>>> are not nearly as sophisticated as Skim in terms of PDF viewing/
>>>> editing. So my guess is that
>>>>
>>>
>>> Apparently Apple *does* implement this in PDFKit on Intel machines  
>>> (I
>>> can only see that on PPC it is not implemented). I explained above
>>> why
>>> it doesn't work in snapshots. The reason it does not work in the  
>>> main
>>> PDF view is that we override the magnification method for something
>>> else (the magnification tool). Apple should have warned us  
>>> developers
>>> about this, or otherwise use a method name we cannot so easily
>>> overwrite. So I consider this an Apple bug, and filed a bug report
>>> with them.
>>
>> Sounds right; class-dump says that PDFView implements
>>
>> - (void)magnifyWithEvent:(id)fp8;
>> - (void)swipeWithEvent:(id)fp8;
>>
>> which is certainly unfortunate.  It shows no difference between the
>> ppc7400 and i386 implementations, though, so it's odd that you don't
>> see it.  Too bad my MBP is a couple months too old to try this  
>> gesture
>> stuff.
>>
>> -- 
>> adam
>>
>
> Yes, I also see it now. So they must have added it somewhere in
> between, because the classdump I made earlier (in 10.5.1 IIRC) does
> not have it.

On second glance, it looks like magnifyWithEvent: is actually in  
NSResponder, so it's not really PDFView's fault.  There are some other  
gesture-related methods there as well.

-- 
adam


-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Register now and save $200. Hurry, offer ends at 11:59 p.m., 
Monday, April 7! Use priority code J8TLD2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Skim-app-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-users

Reply via email to