bug: Neomutt with home-manager; valid options on sort

[bbaserdem]

home-manager/modules/programs/neomutt.nix

Line 369 in 2f23fa3

# allow users to choose any option from sortOptions, or any option prefixed with "reverse-"

The list of valid options for the config option programs.neomutt.sort does not include the sorting options prefixed with last-. Can be seen as one of the available options in the documentation for muttrc. (https://neomutt.org/man/neomuttrc)

You may optionally use the "reverse-" prefix to specify reverse sorting order, or the "last-" prefix to sort threads based on the corresponding attribute of the last descendant rather than the thread root. If both prefixes are in use, "reverse-" must come before "last-". The "last-" prefix has no effect on a flat view.

It wouldn't be much of an issue by setting this through programs.neomutt.settings.sort, but then the folder-hook generated by the accounts.email.accounts.<name> entry sets the sort option referring to programs.neomutt.sort; overriding the sorting option. Right now I use a workaround with setting this again in the folder hook account-.email.accounts.<name>.neomutt.extraConfig = "set sort=${config.programs.neomutt.settings.sort}";

(I could send a pull request as this should be a very quick fix that edits a single line; namely the line 371 should be changed to the following)

          (sortOptions
            ++ (map (option: "reverse-" + option) sortOptions)
            ++ (map (option: "last-" + option) sortOptions)
            ++ (map (option: "reverse-last-" + option) sortOptions)
          );

Or something more elegant than this

[teto]

could send a pull request as this should be a very quick fix that edits a single line

Please do