Change Log of Dired Preview

This document contains the release notes for each tagged commit on the project's main git repository: https://github.com/protesilaos/dired-preview.

The newest release is at the top. For further details, please consult the manual: https://protesilaos.com/emacs/dired-preview.

0.2.0 on 2024-05-07

This release brings many small improvements and fixes some bugs.

The preview shows up right away on new directories

In the past, entering a new directory would not trigger a preview: it would wait for some motion before doing so. This could make it seem that the mode was not enabled in the current buffer. Now, the preview shows up immediately.

Thanks to Nofint for the initial contribution in pull request 8: https://github.com/protesilaos/dired-preview/pull/8. The changes are small and do not require copyright assignment to the Free Software Foundation.

Directories are previewed as well

The contents are displayed in a Dired buffer, using the current settings for it. This means that colours are possible icons are shown too.

Large files are previewed in `hexl-mode`

As the documentation of this mode suggests:

A mode for editing binary files in hex dump format. This is not an ordinary major mode; it alters some aspects of the current mode’s behavior, but not all; also, you can exit Hexl mode and return to the previous mode using C-c C-c.

The C-c C-c key binding toggles the mode, in case you want to see the raw output.

Thanks to Karthik Chikmagalur for the contribution. This was done on the now defunct mailing list: https://lists.sr.ht/~protesilaos/general-issues/%3C871qeb56bw.fsf@gmail.com%3E.

Previews are displayed in a full-featured mode

Before, we would delay the execution of the mode hooks to speed things up. Though this came at the cost of (i) making the buffer less capable and (ii) forcing us to handle the execution of those hooks at a later stage when they would be needed, which proved to be error-prone in a number of scenaria.

Files without an extension are still previewed

This is a bug fix. Before, they would be considered as part of the files to be ignored, per dired-preview-ignored-extensions-regexp.

Previews are not added to what `recentf-mode` tracks

Thanks to Juergen Hoetzel for reporting this issue and for tweaking my suggested patch for it. This was done in pull request 12: https://github.com/protesilaos/dired-preview/pull/12. The change is small, so Juergen does not need to assign copyright to the Free Software Foundation.

A nil value for `split-width-threshold` still works

We no longer consider this an error and adapt the preview accordingly. Thanks to Juergen Hoetzel for the contribution. This was done in pull request 11 and does not require copyright assignment: https://github.com/protesilaos/dired-preview/pull/11.

For developers: previews are done with `cl-defmethod`

Each file type will thus have its own method on how to display the contents in a buffer. The idea is to make this easier to extend. The goal is to have methods that can preview PDFs and images without blocking Emacs. Other file types can be considered as well, though those two are the immediate priority.

0.1.0 on 2023-07-08

The dired-preview package was in a public testing phase from 2023-06-25 until today. In the meantime, lots of changes have been made in the interest of usability and robustness.

Global and buffer-local modes

The dired-preview-mode is a local minor mode, while dired-preview-global-mode is its global counterpart. Both only take effect in Dired buffers.

The idea for a global and a local mode is to empower the user to toggle the functionality on demand, such as when they are in a meeting and want to disable/enable previews in a given context.

During the development phase, I had made an error regarding the scope of what should be a local minor mode. Thanks to Christian Tietze for pointing it out: https://lists.sr.ht/~protesilaos/general-issues/%3Cm1zg4noej2.fsf%40christiantietze.de%3E.

Preview delay runs on an idle timer

Originally, previews would run on a timer that would block Emacs. Whereas we now arrange to only trigger a preview when Emacs is idle for a customisable amount of seconds. Refer to the user option dired-preview-delay.

Trigger a preview in the post-command phase

In the original design, previews were triggered by bespoke dired-preview commands that were remapped to n and p in Dired buffers. This had several downsides, namely, (i) the other motions would not pick up the trigger, (ii) we would have to remap all possible motions, and (iii) the code was needlessly complex.

Currently, we install a local hook in the post-command phase, which will trigger a preview if the previous command was a Dired motion. In future versions, we may expand the list of commands that we check for.

Thanks to Peter Prevos for reporting this in issue 1 on the GitHub mirror: https://github.com/protesilaos/dired-preview/issues/1.

Thanks to Christian Tietze and Ed Hamilton for discussing this topic with me on the mailing list: https://lists.sr.ht/~protesilaos/general-issues/%3Cm1zg4noej2.fsf%40christiantietze.de%3E. Commit ae93720 by Christian Tietze is based on this discussion, although the implementation details have since been redone.

During the development phase, I had made the mistake of checking the last-command, whereas I should be testing against the this-command. Thanks to Karthik Chikmagalur for pointing out my error: https://lists.sr.ht/~protesilaos/general-issues/%3C87sfab8ixn.fsf%40gmail.com%3E.

The placement of the preview window is customisable

We arrange to display previews in a side window. Due to the inherent complexity of the display-buffer function and its accoutrements, a user option is necessarily reserved for experienced users. To this end, we provide the dired-preview-display-action-alist-function. Refer to the dired-preview-display-action-alist-dwim function for the implementation details.

Thanks to Karthik Chikmagalur for making an initial suggestion about such a feature: https://lists.sr.ht/~protesilaos/general-issues/%3C87jzvp484n.fsf%40gmail.com%3E

Thanks to Bruno Boal for discussing the user option and concomitant function with me and for checking the relevant definitions. This was done via a private channel and the information is shared with permission.

Window placement and deletion is more robust

The idea of "preview" windows is that they are not ordinary windows that the user can interact with. As such, they are to be deleted when some non-preview mode of action is taken.

Testing for such cases was extensive and required lots of changes to the code base. Thanks to Bruno Boal for performing the tests with me, for brainstorming possible solutions, and for inspecting the implementation details. This was done via a private channel and the information is shared with permission.

We no longer try to preview irregular files

Before, dired-preview would attempt to produce a preview of named pipes and sockets. This was not intended and has since been addressed. Use 'file-regular-p' instead of 'file-exists-p'

Thanks to Karthik Chikmagalur for bringing this matter to my attention and for recommending the use of file-regular-p instead of file-exists-p: https://lists.sr.ht/~protesilaos/general-issues/%3C87pm5cnpaf.fsf%40gmail.com%3E.

Preview buffers are killed up to a cumulative size threshold

In the original design, we were killing preview buffers all at once. This was wasteful because all the work we were doing in the background to, for example, fetch a large file was discarded even though the user could theoretically request another preview of it.

The current approach is to keep around the newer buffers in order to speed up potential requests for another preview. Older buffers are discarded starting from the oldest. The clearance of older buffers is done until we reach a cumulative size of what is specified as the value of the variable dired-preview--buffers-threshold.

Note that the symbol includes double dashes, meaning that it is intended for "private" (internal) purposes. I am mentioning it here, because this seems like a good candidate for a future user option, subject to further refinements.

Thanks to Bruno Boal for suggesting this idea and checking its implementation with me. This was done via a private channel and the information is shared with permission.