Re: [PATCH] Emacs: Indent first header line only when indentation is turned on

2020-08-10 Thread David Edmondson
On Sunday, 2020-08-09 at 22:01:44 +03, Teemu Likonen wrote: > Previously in message-show mode message's first header line (From > header) was always indented, even if user had turned thread > indentation off with "<" (notmuch-show-toggle-thread-indentation) > command. > > This change modifies

[PATCH v2] Emacs: Indent first header line only when indentation is turned on

2020-08-10 Thread Teemu Likonen
Previously in message-show mode message's first header line (From header) was always indented, even if user had turned thread indentation off with "<" (notmuch-show-toggle-thread-indentation) command. This change modifies notmuch-show-insert-headerline function so that it doesn't indent the first

Re: [PATCH v2] Emacs: Indent first header line only when indentation is turned on

2020-08-10 Thread David Edmondson
On Monday, 2020-08-10 at 17:37:57 +03, Teemu Likonen wrote: > Previously in message-show mode message's first header line (From > header) was always indented, even if user had turned thread > indentation off with "<" (notmuch-show-toggle-thread-indentation) > command. > > This change modifies

Sanitize bidi control chars

2020-08-10 Thread Teemu Likonen
* 2020-08-10 19:45:11+03, Teemu Likonen wrote: > If we wanted to clean message headers from possible unpaired overrides > we should clean all these: > > U+202A LEFT-TO-RIGHT EMBEDDING (push) > U+202B RIGHT-TO-LEFT EMBEDDING (push) > U+202C POP DIRECTIONAL FORMATTING (pop) > U+202D

Re: [PATCH v5] Emacs: Ensure left-to-right display for message headers

2020-08-10 Thread Teemu Likonen
* 2020-08-09 23:12:28+03, utf wrote: > How about this =D > From: contains U+202E (LEFT-TO-RIGHT OVERRIDE) (in > =?utf-8?Q?T=E2=80=AEomi?=) Indeed message's header fields can contain such override characters. The override mode should be terminated with U+202C POP DIRECTIONAL FORMATTING within

Re: [PATCH v5] Emacs: Ensure left-to-right display for message headers

2020-08-10 Thread David Edmondson
On Sunday, 2020-08-09 at 23:12:28 +03, =?utf-8?Q?T=E2=80=AEomi?= Ollila wrote: > How about this =D Your point being that the code sanitising the displayed header could do better? > (Sorry how the headers might look...) > > From: contains U+202E (LEFT-TO-RIGHT OVERRIDE) (in