From: Mark Walters <markwalters1009@gmail.com>
To: Sam Halliday <sam.halliday@gmail.com>
Cc: notmuch@notmuchmail.org
Subject: Re: use font-face instead of explicit colouring
In-Reply-To:
 <CALR_T9CPMKq+9pC9HnKpOiOVnK8DrC33qVF1B8QSctYnSJQ63w@mail.gmail.com>
References: <87mvqdmer2.fsf@gmail.com> <87k2lhmecn.fsf@gmail.com>
 <87h9glb367.fsf@qmul.ac.uk>
 <CALR_T9CPMKq+9pC9HnKpOiOVnK8DrC33qVF1B8QSctYnSJQ63w@mail.gmail.com>
User-Agent: Notmuch/0.18.1+485~gca076ce (http://notmuchmail.org) Emacs/24.4.1
 (x86_64-pc-linux-gnu)
Date: Sat, 05 Mar 2016 11:26:54 +0000
Message-ID: <87egbpb1xd.fsf@qmul.ac.uk>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list


Hi

On Sat, 05 Mar 2016, Sam Halliday <sam.halliday@gmail.com> wrote:
> That's a shame. Having light/dark alternatives are OK as a workaround but
> it's a shame this can't just all be done with faces. I don't see why there
> needs to be a string to face lookup map, why not just put the string in the
> face name?

If you did that how would you let a user customize it (I mean as a
defcustom, rather than setq in .emacs)? There is also a possible
question of dealing with tags with weird characters (eg spaces or
worse), so some form of quoting would be needed.

It's not something I have interest in implementing, but I would be happy
to review patches doing this. Dme might also have some comments -- he
had a complete rewrite of  the code dealing with the search format.

Best wishes

Mark





> On 5 Mar 2016 11:00 a.m., "Mark Walters" <markwalters1009@gmail.com> wrote:
>
>>
>> Hi
>>
>> On Sat, 05 Mar 2016, Sam Halliday <sam.halliday@gmail.com> wrote:
>> > I should say that I am aware of the workaround, e.g.
>> >
>> >     (setq
>> >       notmuch-search-line-faces '(("unread" :weight bold)
>> >                                   ("flagged" :inherit
>> 'font-lock-string-face)))
>> >
>> > but this must be applied per-user. If you removed this defcustom and had
>> > unread/flagged/other faces, then themes (such as darcula) would be able
>> > to provide sensible defaults out of the box.
>>
>> I am not sure I see how this would work without cluttering up the
>> customize options significantly. The notmuch-search-line-faces allows
>> selecting the face based on any tag (so for example I use different
>> colours for emails tagged "work", and tagged "waiting"). We could have
>> additional notmuch-search-line-unread-face which could then be  used in
>> the above -- but now the user has two places to customize the same thing.
>>
>> How do you see this working?
>>
>> (We probably could make the default depend on light or dark theme to make
>> the default more usable.)
>>
>> Best wishes
>>
>> Mark
>>
>> >
>> > Sam Halliday <sam.halliday@gmail.com> writes:
>> >
>> >> [ text/plain ]
>> >> Hi,
>> >>
>> >> I use a dark theme and many of the colour choices of notmuch collide
>> >> with my theme. However, notmuch is explicitly adding information such as
>> >> "blue foreground" instead of using faces that I can customise.
>> >>
>> >> Could you please consider using faces instead of explicit colours?
>> >>
>> >> [ text/plain ]
>> >>
>> >> --
>> >> Best regards,
>> >> Sam
>> >> [ signature.asc: application/pgp-signature ]
>> >
>> > --
>> > Best regards,
>> > Sam
>> > _______________________________________________
>> > notmuch mailing list
>> > notmuch@notmuchmail.org
>> > https://notmuchmail.org/mailman/listinfo/notmuch
>>