thread: optionally show per-message Subject in summary line#1714
thread: optionally show per-message Subject in summary line#1714matclab wants to merge 1 commit intopazz:masterfrom
Conversation
Adds a new config option `msg_summary_show_subject` that controls whether the Subject of each message is displayed on its summary line in thread mode. Valid values: * `never` (the default) keeps the current behaviour — no visible change for users who don't opt in. * `always` always shows the subject as a dedicated column inserted before the tag widgets, so it is truncated last when the line is short. * `different` shows the subject only when it differs from its parent subject after stripping common list/reply/forward prefixes. Recognised prefixes cover `Re`, `Fwd`, `Fw`, `Aw`, `Wg`, `Tr`, `Sv`, `Vs`, `Odp`, `Rv`, `Res` and `Rif` (case insensitive, recursive, with the `Re[2]:` counter form). The Subject header is captured at `Message` construction time (symmetrically with the existing `From` capture), so rendering the summary of a folded thread does not parse the underlying email file. A new theming attribute `thread.summary.subject` lets users style the subject column independently of the rest of the summary line. Fixes pazz#566
|
To be transparent, this PR was written with the help of an AI in the process of testing it for my job in order to define how we can try to not use it w.r.t to the market. I have reviewed it with my knowledge and understanding of alot code, but I'm sure my review was biased. I understand it may be a sufficient reason to reject it, and it is ok. |
|
Interesting, This seems to be similar to my work at #1589 I've just refreshed my branch so you can compare as we've been using this in our team for a long time. |
|
Oh dear, I haven't see this PR ! Thanks. |
|
I've just done a side by side comparison of this PR vs my PR at #1589. Left panel is this one, right panel is mine. 
|
|
2 participants
Add subject in thread view
Fix #566