Hi Daniil, I like this a lot and would like to take the patch after you have taken Ivans feedback, and maybe more feedback if you get any.
I would like it even more if following the link would rehighlight the selected text. Is there any way to achieve this? I have another question, this would maybe be for the original author, Anthony Lander? I see that there are functions with prefixes that are not "org-". This is dangerous because there might be packages around that use that name space. I also think it violates coding rules in Emacs. I think this should be changed - unless you know of a convention that all functions dealing with applescripts are supposed to have an "as-" prefix. I can fix this - but I wanted your feedback first. Thanks. - Carsten On 12 mei 2013, at 20:29, Daniil Frumin <difru...@gmail.com> wrote: > Hi, all! > > I use org-mac-link-grabber.el > <http://orgmode.org/worg/org-contrib/org-mac-link-grabber.html> almost every > day. However, it lacks support for an app that I'd like to use together with > org-mode. > > Skim.app is a light and fast PDF reader for Mac OS X with a note-taking > ability. I wrote a little patch for org-mac-link-grabber.el to support > grabbing links to documents. > > What it does: > > * Grabs not just the link to file, but a page > > * Inserts the selected text as a description, if present. Otherwise > inserts "<filename>, p. <page #>" > > * The shortcut is set to [S] > > * Defines a new "skim" link type > > It would be interesting to also add some support for importing notes from > Skim to org. > > Since Skim.app is not present in clean OS X installs, by default support for > grabbing links from it is disabled. You can enable it by customizing group > `org-mac-link-grabber'. > > So, maybe it's possible to get this patch into the tree? It's my first time > hacking on org (or even any major elisp extension), so it's probably that > I've messed up somewhere with a commit format or whatnot. > > Cheers. > > -- Daniil Frumin > <0001-Adding-Skim.app-support-to-org-mac-link-grabber.el.patch>
