nextcloud / spreed

🗨️ Nextcloud Talk – chat, video & audio calls for Nextcloud
https://nextcloud.com/talk
GNU Affero General Public License v3.0
1.63k stars 437 forks source link

Add context to forwarded messages #8470

Open pmarini-nc opened 1 year ago

pmarini-nc commented 1 year ago

How to use GitHub


Is your feature request related to a problem? Please describe. When forwarding a message from a conversation A to a conversation B:

Describe the solution you'd like The forwarded message should show some context about its origin, for example "Forwarded from Conversation A" in conversation B and "Forwarded to Conversation B" in conversation A.

Instead of text maybe we can use icons..

Describe alternatives you've considered

Additional context

nickvergessen commented 1 year ago

This is the current privacy-aware design decision. The other participants might not be allowed to know that ConvA or ConvB exists.

Also you can still copy around manually without that information being shown, so it doesn't give a guarantee either.

pmarini-nc commented 1 year ago

That's true I didn't think about it.

Still it would be great to have an indication that the message has been forwarded. For example ">>" for forwarded in, "<<" for forwarded out.

nickvergessen commented 1 year ago

Still it would be great to have an indication that the message has been forwarded. For example ">>" for forwarded in, "<<" for forwarded out.

I don't feel like exposing user to this. Especially since it was not done in the past it would be weird to start it now.

pmarini-nc commented 1 year ago

Fair enough. Then should we close this with rejected status? Thanks!

marcoambrosini commented 6 months ago

I think that the current implementation can be confusing, because for many use cases that type of context is needed. I don't think there's a privacy problem if we only add the label "Forwarded" on the message in the recipient's chat. I don't think there's the need for a forwarded out label.

Screenshot 2024-05-07 at 15 47 48
marcoambrosini commented 6 months ago

adding @nextcloud/designers for more input

nimishavijay commented 6 months ago

Here's how others do it:

image image

image image

I would definitely not show the sender's name for privacy reasons. IMO, other than for preventing misinformation I don't see a strong need for a "Forwarded" label.

marcoambrosini commented 6 months ago

When I use this feature and forward a message to someone I always find myself having to provide this missing context in the following message in the form of "This message above has been posted in the X chat" or "he message that I just sent was sent to me by Y" Clearly indicating that the message has been forwarded allows us to shorten that form a bit by providing some context.

nickvergessen commented 6 months ago

You'd still need to provide the original author in that case.

marcoambrosini commented 6 months ago

Yes, that'd still be on you

marcoambrosini commented 6 months ago

Further references

Teams: Quote component and author

Screenshot 2024-05-09 at 10 21 49

Slack: Also quote component and author + info about original conversation

Screenshot 2024-05-09 at 10 24 45

I think we could also use the quote style to further differentiate it from other messages