Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run viewer asynchronously using vim.system #873

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

moreka
Copy link
Contributor

@moreka moreka commented Jan 28, 2025

Previously, when the viewer opened, the editor was blocked until the user closes the viewer. Now, the viewer opens asynchronously (using vim.system rather than the old vim.fn.system); in this way, the user can view the PDF while continuing editing the org file in the editor.

Previously, when the viewer opened, the editor was blocked until the
user closes the viewer. Now, the viewer opens asynchronously (using
`vim.system` rather than the old `vim.fn.system`); in this way, the user
can view the PDF while continuing editing the org file in the editor.
@moreka
Copy link
Contributor Author

moreka commented Jan 28, 2025

I understand also that the next step might be to add support for detecting PDFs that are already open in a PDF viewer and not opening them after export (and just updating them). But I guess this is going to be difficult to maintain and is not the main functionality of this plugin as one might expect.

Copy link
Member

@kristijanhusak kristijanhusak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kristijanhusak kristijanhusak merged commit f9a74a4 into nvim-orgmode:master Jan 28, 2025
6 checks passed
@moreka moreka deleted the run-viewer-async branch January 28, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants