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
next 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
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=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 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).