fix: introduce UI toggle to allow/disallow updating#18497
Conversation
b0670ac to
20944dd
Compare
|
Why are we not always allowing editing? 🤔 There is no switch to allow editing for text, so this extra switch feels a bit disruptive. We should indicate that a file has been edited always in the message, though. If we do want a switch I would suggest we keep it enabled by default and a bit more hidden, what do you think? |
We had reports before, especially when sharing screenshots and images it's totally unexpected that others can replace/edit them.
Not technically possible
As per team week discussion we are most likely going to add a compression toggle anyway, so it would just be 2 for now. |
Is it "everyone in the room" or just moderators? What would be the use case for others being able to edit the image? If it is technically not possible then maybe we just don't allow editing for images for the sake of transparency? Taking a quick look at Teams, Slack, Google Workspace Chat and Discord don't allow others to edit even text messages in the first place. This switch being so present is asking the user to make another decision before the simple and very commonly used action of sending an image. |
Everyone got write permissions before, if file uploaded directly to Talk
Organisations working with visual materials / marketing maybe?
That sounds like another way of working around current behaviour (everything is read-only, if you need editing - share from Files app). Also video/audio could fall into that category. Or the other way around, somehow receive list of office-provided filetype (+ .txt .md files), and allow edit them by default - but I agree now that is also subtle and non-obvoius behaviour
We can default back to legacy switches (toggle on the left side, smaller) or checkboxes |
92ff5f9 to
39240aa
Compare
DorraJaouad
left a comment
There was a problem hiding this comment.
Tested, yet I am not sure if showing this toggle for images /videos only shares is good
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
aba58be to
6576763
Compare
|
/backport to stable34 |
☑️ Resolves
AI (if applicable)
🖌️ UI Checklist
🖼️ Screenshots / Screencasts
🚧 Tasks
🏁 Checklist
Signed-off-by: Maksim Sukharev antreesy.web@gmail.com