search

izuzak / atom-pdf-view

Support for viewing PDF files in Atom.
https://atom.io/packages/pdf-view
MIT License
106 stars 30 forks source link

Add option to change SyncTeX click behaviour #114

Closed nsaje closed 8 years ago

nsaje commented 8 years ago

The action can now be specified in the config. It can be either Disabled, Click or Double click.

I also renamed the function to reverseSync(), since it makes more sense combined with #111 .

closes #110

nsaje commented 8 years ago

Good catch @izuzak ! What do you think of the approach of calling updatePdf() when that option changes? We could handle listeners separately, but this is simpler if it's good enough - since configuration changes are quite rare.

izuzak commented 8 years ago

What do you think of the approach of calling updatePdf() when that option changes? We could handle listeners separately, but this is simpler if it's good enough - since configuration changes are quite rare.

Yeah, sounds like that should work. I'd prefer the other approach which wouldn't require re-updating the whole document, but let's use this for now and see if someone complains. :grin:

Thanks again for this! :cake: