Re: [O] [Patch] org-display-inline-images: Add support for remote images

2014-11-26 Thread Kit-Yan Choi
ionality ON/OFF, default off maybe. Thanks, Kit On Tue, Nov 25, 2014 at 12:04 PM, Michael Albinus wrote: > Kit-Yan Choi writes: > > > Michael, > > Hi Kit, > > > Thanks for the suggestion. Indeed `file-local-copy' would have made > > the code cleaner. Yet `fi

Re: [O] [Patch] org-display-inline-images: Add support for remote images

2014-11-25 Thread Kit-Yan Choi
nnecessary file transfer, which takes time (think of it like "rsync -a"). Therefore I wanted to preserve the remote path under a temporary directory. Best, Kit On Tue, Nov 25, 2014 at 10:39 AM, Michael Albinus wrote: > Kit-Yan Choi writes: > > > Ah my apologies. I forgot I

Re: [O] [Patch] org-display-inline-images: Add support for remote images

2014-11-25 Thread Kit-Yan Choi
dth width))) (when image (let* ((link ;; If inline image is the description On Tue, Nov 25, 2014 at 10:15 AM, Kit-Yan Choi wrote: > Thank you for your comments! They are very helpful. > > > Thanks for your patch. However, I wonder if we really want this. Remote > >

Re: [O] [Patch] org-display-inline-images: Add support for remote images

2014-11-25 Thread Kit-Yan Choi
name file + (make-directory tramp-tmpdir t) + (if (file-newer-than-file-p file newname) +(copy-file file newname t t)) + newname) +file) + (and width 'imagemagick) + nil + :width width))) (when image (let* ((link ;; If inline image is the description On Tue, Nov

[O] [Patch] org-display-inline-images: Add support for remote images

2014-11-21 Thread Kit-Yan Choi
Hi, I would like to submit a patch to support displaying remote images inline in Org-mode. Attached is the formatted patch (or github branch here .) I have tested the code with "make test". FSF document is sig