-
Notifications
You must be signed in to change notification settings - Fork 18
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
Some minor fixes #29
base: master
Are you sure you want to change the base?
Some minor fixes #29
Conversation
Thanks a lot for taking this initiative! Please see comments above. |
Some people might be running this on old emacs versions without As for the repeating |
Thanks for the fixes! Will merge this after some more local testing. |
Hm, I got an error
in Emacs 28.2. Then I ran |
Fix #27 by ensuring the point is placed at the relevant bookmark entry when org-mode functions are called, using save-window-excursion and save-mark-and-excursion to keep this from affecting user experience.
Major note: As with the original code, the buffer needs to be saved manually.
Minor note: Sometimes the wget process doesn't finish by the time org-mode notices the ARCHIVED_AT link, so it initially appears dead. Waiting until the process finishes and [saving and] refreshing the buffer fixes this.
Minor note: This doesn't suppress the wget buffer, it just keeps it from appearing on-screen.