Add From: to MessageView list

[fasterit]

I found it inconsistent that the From: is shown on the Inbox view but not on the messages view.
Hope this helps others that have a diverse set of senders, too.
/DLange

[axllent]

Thank you. Originally I intentionally skipped this to save space and to resemble the mobile view of the "inbox view", however I realise now that I used to To instead of the From.... an oversight on my behalf.

Adding the From here does not actually add that much space, however makes it very difficult to see the difference between the subject line and the From line. I'm wondering is this is maybe not a better (and easier to read) alternative to your proposal?

What do you think?

[fasterit]

I actually like seeing "To:" and "From:".
Visually I like your proposal but may be you can add the "To:" line again below the sender?

[axllent]

Do you mean like this?

[fasterit]

Yes, I like that. It seems the most useful way of presenting enough data for an easy overview of the message view.

[axllent]

OK cool, I'm happy with that. Please leave this with me as I've already made the changes locally which is quite different to what you have, and I also include a couple of minor changes to the mobile view of the inbox view. I'll push through the update shortly (which will become available in the docker edge build if you're interested after I push the change). Please note that this won't be released for another week or two as I am about to have shoulder surgery, so it will take a bit longer than usual before I release a new version (I just released a version a couple of days ago) 👍

Thanks for the PR though, I do appreciate it even if I'm not merging it in this time!

[fasterit]

No worries, we have the "issue" solved with the PR, so take all your time. We'll converge back to upstream once you push your change.
Good luck with the shoulder surgery!
/DLange

[axllent]

Thank you!

[axllent]

This has now been released in v1.21.7.