From: joakim@verona.se
To: Vitalie Spinu <spinuvit@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: imagemagic in image-mode and image-dired-thumbnail-mode?
Date: Sun, 14 Jul 2013 10:32:50 +0200 [thread overview]
Message-ID: <m3li591qz1.fsf@exodia.verona.se> (raw)
In-Reply-To: <87a9lp8uxk.fsf@gmail.com> (Vitalie Spinu's message of "Sun, 14 Jul 2013 09:25:27 +0200")
Vitalie Spinu <spinuvit@gmail.com> writes:
> Hello,
>
> For a new process-output-image-manipulation project I was looking for a
> generic image mode. Surprisingly, it looks like none of the existing
> modes is flexible/customizable/feature rich enough to be used as a
> prototype for such a project.
>
> Particularly, an essential feature for image scaling, imagemagic, is
> still not supported by basic image modes. Image-dired-thumbnail-mode has
> no idea of imagemagic at all. Image-mode cannot open imagemagic types
> as image-toggle-display-image hardcodes image type:
>
> ╭──────── #616 ─ lisp/image-mode.el ──
> │ (type (image-type file-or-data nil data-p))
> ╰──────── #616 ─
>
> Is there a hidden reason why image scaling is not yet implemented?
>
> ;; Not yet implemented.
> ;; (defvar image-transform-minor-mode-map
> ;; (let ((map (make-sparse-keymap)))
> ;; ;; (define-key map [(control ?+)] 'image-scale-in)
> ;; ;; (define-key map [(control ?-)] 'image-scale-out)
> ;; ;; (define-key map [(control ?=)] 'image-scale-none)
> ;; ;; (define-key map "c f h" 'image-scale-fit-height)
> ;; ;; (define-key map "c ]" 'image-rotate-right)
> ;; map)
> ;; "Minor mode keymap `image-transform-mode'.")
>
I worked on scaling support for image mode when I worked on the original ImageMagik patch.
However, I didnt have time to finish the support. The above keymap was
probably written by someone else, have a look in the log.
At any rate, as I recall, there are no reasons not to implement scaling
support in image mode, Someone(TM) just needs to invest some energy. It
shouldnt even be very hard.
>
> I naturally wonder if there should be a global user custom variable
> like:
>
> (defcustom image-view-type (and (image-type-available-p 'imagemagick)
> 'imagemagick))
>
> with all image modes taking it into account?
>
> On a related note, wouldn't image-mode be better off if implemented on
> top of some image-thumbnail-mode? Or maybe better, shouldn't image-mode
> support multiple images per buffer, and then other modes like
> image-dired-thumbnail-mode to derive from it?
>
> Vitalie
>
>
--
Joakim Verona
next prev parent reply other threads:[~2013-07-14 8:32 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-14 7:25 imagemagic in image-mode and image-dired-thumbnail-mode? Vitalie Spinu
2013-07-14 8:32 ` joakim [this message]
2013-07-14 11:48 ` Vitalie Spinu
2013-07-14 12:40 ` joakim
2013-07-14 13:01 ` Vitalie Spinu
2013-07-14 12:42 ` Lars Magne Ingebrigtsen
2013-07-14 18:21 ` Glenn Morris
2013-07-14 19:50 ` Lars Magne Ingebrigtsen
2013-07-14 20:06 ` Eli Zaretskii
2013-07-14 20:11 ` Lars Magne Ingebrigtsen
2013-07-14 22:00 ` Vitalie Spinu
2013-07-15 4:38 ` Eli Zaretskii
2013-07-15 4:15 ` Stephen J. Turnbull
2013-07-15 4:46 ` Eli Zaretskii
2013-07-15 5:45 ` Stephen J. Turnbull
2013-07-15 10:39 ` Óscar Fuentes
2013-08-02 15:32 ` Steinar Bang
2013-07-15 15:50 ` Eli Zaretskii
2013-07-14 18:33 ` Glenn Morris
2013-07-14 19:17 ` joakim
2013-07-15 10:51 ` Vitalie Spinu
2013-07-16 15:57 ` Glenn Morris
2013-07-16 21:26 ` Stefan Monnier
2013-07-17 7:29 ` Vitalie Spinu
2013-07-17 15:51 ` Vitalie Spinu
2013-07-18 8:47 ` Lars Magne Ingebrigtsen
2013-07-18 22:27 ` Vitalie Spinu
2013-07-19 9:22 ` Stefan Monnier
2013-07-20 7:25 ` Vitalie Spinu
2013-07-22 20:17 ` Vitalie Spinu
2013-07-22 20:31 ` Lars Magne Ingebrigtsen
2013-07-23 8:31 ` Vitalie Spinu
2013-07-18 23:22 ` image-transform.el and image-mode.el rewrite Vitalie Spinu
2013-07-19 11:52 ` Wolfgang Jenkner
2013-07-19 12:21 ` Wolfgang Jenkner
2013-07-20 7:18 ` Vitalie Spinu
2013-07-22 20:37 ` Glenn Morris
2013-07-22 21:05 ` Vitalie Spinu
2013-10-08 18:08 ` Glenn Morris
2013-10-08 23:43 ` Vitalie Spinu
2013-10-09 0:02 ` Michael Heerdegen
2014-12-15 9:33 ` Vitalie Spinu
2014-12-18 14:17 ` Michael Heerdegen
2014-12-18 21:32 ` Vitalie Spinu
2014-12-18 15:15 ` Stefan Monnier
2014-12-18 23:23 ` Vitalie Spinu
2014-12-19 4:19 ` Stefan Monnier
2014-12-19 4:46 ` Vitalie Spinu
2014-12-19 8:56 ` Eli Zaretskii
2014-12-19 17:50 ` Stefan Monnier
2014-12-19 19:37 ` Eli Zaretskii
2014-12-19 21:31 ` Stefan Monnier
2014-12-19 21:49 ` Eli Zaretskii
2014-12-19 10:24 ` Vitalie Spinu
2014-12-19 17:51 ` Stefan Monnier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3li591qz1.fsf@exodia.verona.se \
--to=joakim@verona.se \
--cc=emacs-devel@gnu.org \
--cc=spinuvit@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.