unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
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



  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

  List information: https://www.gnu.org/software/emacs/

* 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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).