Re: [PATCH v2] emacs: logically group def{custom,face}s

Subject: Re: [PATCH v2] emacs: logically group def{custom,face}s

Date: Fri, 13 Jan 2012 16:15:59 +0000

To: David Bremner, Pieter Praet

Cc: Notmuch Mail

From: David Edmondson

On Thu, 12 Jan 2012 23:31:34 -0400, David Bremner <david@tethera.net> wrote:
> On Thu, 12 Jan 2012 18:12:16 +0100, Pieter Praet <pieter@praet.org> wrote:
> > To allow for expansion whilst keeping everything tidy and organized,
> > move all defcustom/defface variables to the following subgroups,
> > defined in notmuch-lib.el:
> > 
> > - Hello
> > - Search
> > - Show
> > - Send
> > - Crypto
> > - Hooks
> > - Appearance
> > - External Commands
> 
> I didn't investigate too closely, but I noticed when I customize-group
> emacs, each subgroup has some explanatory text beside it. Did you omit
> that on purpose?

That should be a requirement, I think.

• Previous message (by thread): Re: [PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups

Thread:

• Pieter Praet—[PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Daniel Schoepe—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::feature, signed, unread]
    • Jameson Graef Rollins—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, signed, unread]
      • Pieter Praet—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
        • David Bremner—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
          • Pieter Praet—[PATCH v2] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
            • David Bremner—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, unread]
              • David Edmondson—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, signed, unread]
                • Pieter Praet—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, unread]
              • Pieter Praet—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, unread]
                • Pieter Praet—[PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
                  • Austin Clements—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::review, unread]
                    • Jameson Graef Rollins—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
                    • Pieter Praet—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
                      • Pieter Praet—[PATCH v4] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
                        • David Edmondson—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, signed, unread]
                        • Austin Clements—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, notmuch::review, unread]
                          • Pieter Praet—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                            • Austin Clements—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                              • Pieter Praet—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                                • Pieter Praet—[PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
                                  • Austin Clements—Re: [PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups [inbox, notmuch::review, unread]
                                  • David Bremner—Re: [PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups [inbox, unread]
                        • David Bremner—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                          • Pieter Praet—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
            • Jameson Graef Rollins—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, notmuch::review, signed, unread]