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

Forward search in SyncTeX not working #107

Closed edwinksl closed 8 years ago

edwinksl commented 8 years ago

I was trying out the latex and pdf-view packages and noticed that forward search (from editor to PDF) in SyncTex does not work, although reverse search (from PDF to editor) does work wonderfully.

izuzak commented 8 years ago

Just to make sure I understand -- can you clarify what "forward search (from editor to PDF)" and "reverse search (from PDF to editor)" means? The SyncTeX integration does one thing: when you click on something in the PDF, it brings you to the relevant line in the source file. There's no search involved. Did you perhaps expect that when you click a line in the source file -- the PDF would scroll to show the relevant part for that line?

edwinksl commented 8 years ago

Yeah, I expected that clicking a line in the source file would bring me to the relevant part in the PDF.

P.S.: "Forward search" and "reverse/inverse search" do sound confusing, but unfortunately that's how they are commonly called; see https://www.gnu.org/software/auctex/manual/auctex/I_002fO-Correlation.html for example.

izuzak commented 8 years ago

Yeah, I expected that clicking a line in the source file would bring me to the relevant part in the PDF.

That would be a neat feature, but it's currently expected that it doesn't happen. Added the enhancement label on this issue. :v: Thanks for confirming

dpo commented 8 years ago

:+1:

hmarthinsen commented 8 years ago

That would be very useful.

schlichtanders commented 8 years ago

+1 it would be awesome to have such a fully interactive latex-preview within atom

izuzak commented 8 years ago

I'm going to reopen this. #111 added one half of support for this. @nsaje will probably followup with a PR to atom-latex soon to get the other part in as well.

@nsaje if you don't have time for that -- can you add a comment here so that someone else might tackle it? :pray: Also, please let us know when that support lands in atom-latex so that we can close this. :v:

nsaje commented 8 years ago

The PR has landed in atom-latex so the full functionality is now available to users that have both packages installed :+1:

izuzak commented 8 years ago

:sparkling_heart: @nsaje Thanks so much for your work -- I'm closing this now. :sparkler: