Skip to content

feat: mailbox icon in thread view - #13486

Open
madbob wants to merge 1 commit into
nextcloud:mainfrom
madbob:issue_9535
Open

feat: mailbox icon in thread view#13486
madbob wants to merge 1 commit into
nextcloud:mainfrom
madbob:issue_9535

Conversation

@madbob

@madbob madbob commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

A proposal for #9535: in the thread view, I've added the icon of the parent mailbox for each message (and a tooltip with the full name of the mailbox itself).

Screenshot From 2026-08-13 19-37-10

Evaluating this feature, I've been concerned of adding the full name of the mailbox as it would eventually squeezes even more the envelope's header (especially on mobile). An icon is mostly decorative, slightly informative, and the tooltip provides the full information in unobtrusive way (in particular for non-special mailboxes not having their own recognizable icon).

Here I've also fixed a minor alignment issue with the sender line and message's preview line, and I've removed the mailboxId parameter for ThreadEnvelope (I've not really understood his utility, as the same information is carried by envelope and indeed the mailboxId binded to the local route in Thread impacts in the functionalities enabled for every child envelope).

Fixes #9535

@ChristophWurst

Copy link
Copy Markdown
Member

Thanks for looking into this!

Would appreciate input from https://nextcloud-vue-components.netlify.app/#/Components/NcChip.

Maybe a NcChip? https://nextcloud-vue-components.netlify.app/#/Components/NcChip

@madbob

madbob commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator Author

Added a wrapping NcChip.

I would still avoid to include the textual name of the mailbox, both to save screen space and to avoid misalignment with different length names.

Screenshot From 2026-08-14 12-06-10

Signed-off-by: Roberto Guido <info@madbob.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show mailbox in thread view

2 participants