Wow, it's so slick and polished now! Thank you, Sheri, there's a lot that I can 
learn from your script.

Sheri wrote:

> - added optional ini file for default options.
Nice.

> 
> - 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(...))

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"))

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.

> 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.

Reply via email to