--- In [email protected], sgp <[EMAIL PROTECTED]> wrote:

Hi sg,

Thanks for the feedback :)
> 
> > 
> > - implemented pdf search feature for use with Adobe Acrobat Reader.
> Very nice! Some suggestions involving this code snippet:
> 
> do (acroread, ?x"x++pdffile++?x"x)
> Wait.for(activewindow("=AcroRd32"))
> 
> a. add a timeout to wait.for() otherwise the script will wait
> forever if Acrobat doesn't start
> Wait.for(10000,activewindow(...))

OK, the hope is the configured exe will run and be the right thing,
but I guess somebody could make it run something else.
> 
> b. change the caption to include Acrobat.exe, for people who use
> the full product instead of just the Reader
> Wait.for(10000,activewindow("=AcroRd32,=Acrobat"))

Do you know it to work with full Acrobat?

I have Acrobat version 7 and it doesn't work with that (different
class names, etc.). I vastly prefer browsing and searching pdfs with
Reader 8.1 and use my full version only for creating and editing pdfs.
Version 7 has some searching quirks like needing to expand a tree to
see any (single pdf) hits. Sometimes I think I should get the trial
version of 8 just to see if they improved the Word add-in. But if my
equipment is underpowered for version 7 with big docs with lots of
cross referencing, I suspect it would be worse with 8. Anyway unless
you've tested with full Acrobat, I'll stick with "=AcroRd32".

> 
> The above change works very well for me. Error trapping for when
> wait.for expires without finding Acrobat is to be added to the
> script.

Will definitely add a timeout and error trap.

> 
> > implemented a new value 2 for research mode, which allows copying
> > to the clipboard but does not restore original content. Shows as
> > gray-checked in options. Thank you, that was for me I guess. It
> > works fine here.

Seemed logical, I've even like it myself with value 2. :)

Regards,
Sheri


Reply via email to