all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Sacha Chua <sacha@sachachua.com>
To: emacs-devel@gnu.org
Subject: [PATCH] image-dired-show-all-from-dir: reuse dired's default directory
Date: Thu, 02 Jan 2025 15:00:29 -0500	[thread overview]
Message-ID: <87cyh5ugte.fsf@sachachua.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 502 bytes --]

Hello, everyone!

With a tiny tweak, image-dired-show-all-from-dir can handle a bunch of
things that dired can handle already (view arbitrary list of files,
handle wildcards). This is how I've been using it to quickly look at the
most recent screenshots in a couple of directories:
https://sachachua.com/blog/2025/01/using-image-dired-to-browse-the-latest-screenshots-from-multiple-directories/

What do you think? I believe I have copyright
assignment papers on file, although this is a tiny change.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-image-dired-show-all-from-dir-Reuse-dired-s-default-.patch --]
[-- Type: text/x-diff, Size: 1811 bytes --]

From afc2b649cd2b26765f4a7fd4c6f3eac51c4018f5 Mon Sep 17 00:00:00 2001
From: Sacha Chua <sacha@sachachua.com>
Date: Thu, 2 Jan 2025 14:52:28 -0500
Subject: [PATCH] image-dired-show-all-from-dir: Reuse dired's
 default-directory

* lisp/image/image-dired.el (image-dired-show-all-from-dir): Delegate
figuring out default-directory to dired. This handles the case where
image-dired is called with a cons cell that has a directory in the car
and a list of files in the cdr, which dired can handle, and therefore
allows image-dired-show-all-from-dir to show an arbitrary list of
images. It also means that image-dired-show-all-from-dir can handle
wildcards.
---
 lisp/image/image-dired.el | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lisp/image/image-dired.el b/lisp/image/image-dired.el
index 83745e88f09..06fbca6a3f7 100644
--- a/lisp/image/image-dired.el
+++ b/lisp/image/image-dired.el
@@ -630,7 +630,8 @@ image-dired-show-all-from-dir
   (interactive "DShow thumbnails for directory: ")
   (dired dir)
   (dired-mark-files-regexp (image-dired--file-name-regexp))
-  (let ((files (dired-get-marked-files nil nil nil t)))
+  (let ((files (dired-get-marked-files nil nil nil t))
+        (dired-default-directory default-directory))
     (cond ((and (null (cdr files)))
            (message "No image files in directory"))
           ((or (not image-dired-show-all-from-dir-max-files)
@@ -644,7 +645,7 @@ image-dired-show-all-from-dir
            (let ((inhibit-message t))
              (dired-unmark-all-marks))
            (pop-to-buffer image-dired-thumbnail-buffer)
-           (setq default-directory dir)
+           (setq default-directory dired-default-directory)
            (image-dired--update-header-line))
           (t (message "Image-Dired canceled")))))
 
-- 
2.43.0


[-- Attachment #3: Type: text/plain, Size: 7 bytes --]


Sacha

             reply	other threads:[~2025-01-02 20:00 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-02 20:00 Sacha Chua [this message]
2025-01-02 20:22 ` [PATCH] image-dired-show-all-from-dir: reuse dired's default directory Stefan Kangas
2025-01-02 22:17   ` Sacha Chua
2025-01-03  0:23     ` Stefan Kangas
2025-01-07  7:21 ` Stefan Kangas

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=87cyh5ugte.fsf@sachachua.com \
    --to=sacha@sachachua.com \
    --cc=emacs-devel@gnu.org \
    /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.