Folding all messages does not apply after

[guibor]

I would expect (add-hook 'mu4e-headers-found-hook 'mu4e-headers-fold-all) to automatically fold threads after any query (for some reason setting the default mode to 'folded doesn't apply.

For some reason threads are all unfolded, even though running (mu4e-headers-fold-all) immediately folds all threads when run explicitly.

[guibor]

I tried (advice-add 'mu4e-headers-mark-threads :after 'mu4e-headers-fold-all) and got the appropriate behavior, but it seems hacky. I would also perhaps expect setting the default mode to 'folded to apply after any search (or perhaps have a different configuration for this.

Otherwise, the package has gone through a lot of improvements over the last few weeks and is great!

[guibor]

Actually the previous solution presented problems.

This worked:

(setq mu4e-headers-found-hook '(mu4e-headers-mark-threads mu4e-headers-fold-all))

essentially ensuring that hooks are executed in the right order. It looks like mu4e-headers-mark-threads unfolds all threads. I'm sure there's a more elegant solution (either through the package, or just as a way to order hooks appropriately)

[rougier]

Thanks for the fix. Not sure about the difference between the add-hook and setq versions. Does that mean add-hook prepend the hook ?

Also, could you make a PR to include your change (with possibly an adaptation whether default mode is folded/unfolded)?