Skip to content
This repository has been archived by the owner on Aug 14, 2022. It is now read-only.

Add a Skim.app opener #29

Closed
wants to merge 1 commit into from
Closed

Add a Skim.app opener #29

wants to merge 1 commit into from

Conversation

urdh
Copy link
Contributor

@urdh urdh commented Sep 10, 2014

Only adds the actual opener, with no code to make it accessible to the user. This would probably be done using some kind of option, but I'm not sure what the best implementation of PDF viewer selection options would be.

This is kind of in preparation of introducing SyncTeX capabilities, since Preview.app doesn't support this(?). Skim.app does, through a utility included in its application contents, but using that utility requires that the caller knows the full path to Skim (standard is /Applications/Skim.app) which may be something to have in mind when designing a PDF viewer option.

Briefly SyncTeX with Skim would be calling /Applications/Skim.app/Contents/SharedSupport/displayline -r [-g] <line> <pdf-file> <tex-file> for forward syncing as the Sublime package does, and doing atom <tex-file>:<line> in the other direction (assuming atom/atom#1743 gets fixed), so it's not that difficult assuming the path to Skim.app is known and appropriate hooks are available in Atom itself.

Only adds the actual opener, with no code to make it accessible to the user.
This would probably be done using some kind of option, but I'm not sure what
the best implementation of PDF viewer selection options would be.

This is kind of in preparation of introducing SyncTeX capabilities, since
Preview.app doesn't support this (?). Skim.app does, through a utility
included in its application contents, but using that utility requires that
the caller knows the full path to Skim (standard is `/Applications/Skim.app`)
which may be something to have in ming when designing a PDF viewer option.
@thomasjo
Copy link
Owner

First of all, sorry about not responding to this earlier, I completely forgot about it! Secondly, thanks for contributing!

I started on a Skim.app opener a little while back, but never got around to pushing my branch to make it visible that it was something being worked on. I have since both pushed the branch, and subsequently merged the opener into master.

Haven't cut a new version yet though, and I'd love to hear your thoughts on the current implementation of SkimOpener. There's no forward and backward sync support in place yet, but the AppleScript approach I'm using to launch Skim.app, does ensure the displayed PDF is refreshed on every successful build at least.

I'm going to close this PR, but I'd be super happy if you wanted to take a stab at forward syncing in a new PR. Not exactly sure where or how we should implement such a feature, but either one of us can open an issue and we can discuss further there.

@thomasjo thomasjo closed this Sep 26, 2014
@urdh urdh deleted the skim-opener branch September 29, 2014 21:02
@urdh
Copy link
Contributor Author

urdh commented Sep 29, 2014

I'm looking at some of the SyncTex stuff in urdh:skim-synctex – see #31.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants