Re: Keen on a dired-hide-details-mode patch?

[Alvaro Ramirez <alvaro@xenodium.com>]

> Alvaro -
>
> Yes, `dired-maybe-insert-subdir` is the command that inserts a 
> subdir provided
> it is issued when the point is over a directory. For reference, 
> I invoke it
> from Casual Dired from here
> https://github.com/kickingvegas/casual-dired/blob/4be72b52f91700cdb529a185b8f6f21bd0a86542/lisp/casual-dired.el#L76
>
> Ideally your hiding code would show the relative pathname of a 
> subdir when
> hiding is turned on, otherwise the fullpath.

I think https://0x0.st/s/1UqYa811zfdRYdMdDWdE-Q/XO2D.gif shows 
that? or do we mean something else?

Without patching your core dired.el, try the following and see if 
that does what you expect:

(use-package dired
  :hook ((dired-mode . dired-hide-details-mode)
         (dired-after-readin 
         . ar/hide-dired-details-include-all-subdir-paths))
  :config
  (defun ar/hide-dired-details-include-all-subdir-paths ()
    (save-excursion
      (goto-char (point-min))
      (while (re-search-forward dired-subdir-regexp nil t)
        (let* ((match-bounds (cons (match-beginning 1) (match-end 
        1)))
               (path (file-name-directory (buffer-substring (car 
               match-bounds) (cdr match-bounds))))
               (path-start (car match-bounds))
               (path-end (+ (car match-bounds) (length path)))
               (inhibit-read-only t))
          (put-text-property path-start path-end 'invisible 
          'dired-hide-details-information)))))) 

> Note that my current understanding of subdir behavior with 
> default Dired is
> that it only supports subdirectories within the current parent 
> directory. I
> vaguely recall there are other 3rd party packages that try to 
> let you insert
> out-of-hierarchy subdirs. I don’t think it’s something you need 
> to support, but
> this patch could potentially break such packages if pushed into 
> core.

I use one of them (dired-subtree). No breakage so far.