Karthik Chikmagalur <karthikchikmaga...@gmail.com> writes: >> The PR do port the package to this new handler property: >> https://github.com/gaoDean/org-remoteimg/pull/4 >> >> However it left me with a few questions: >> - Who is supposed to take care of the scaling/width of the preview >> image, the handler or the caller of the handler? > > The handler is responsible for sizing the image. The reason for this is > that the preview does not have to be an image -- it can be any kind of > overlay decoration. > > If you decide to use an image file as the preview, you can call > `org-link-preview-file' inside your handler to handle the geometry for > you. This includes the size and alignment specified by #+attr_* > keywords, `org-image-max-width' and `org-image-align'. > > If you are using a preview image from image data, you'll have to copy > some of the code in `org-link-preview-file' to your handler if you want > to respect these user options.
Would it be possible to also handle image data in the function or refactor the org-link-preview-file function in a way that the geometry handling is done in a helper function which can be reused by other handlers. In the example I mentioned it should be possible to use the cached file from the url cache but that might be not so easy in other cases. >> - When calling the link preview function for a description with prefix >> argument 1 I noticed that it doesn't get the contents of the >> description. >> Why is that? In this instance the link looked like this: >> >> [[https://stable.melpa.org/#/rpm-spec-mode][file:https://stable.melpa.org/packages/rpm-spec-mode-badge.svg]] > > I don't follow. What do you mean by "it doesn't get the contents"? Did > the preview work as expected when you used a prefix arg of 1? The link element passed towards the handler didn't contain the description e.g. in this case file:https://stable.melpa.org/packages/rpm-spec-mode-badge.svg.