Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The UX doesn't make it clear that room history changes only affect events written _after_ the settings change was made. #5247

Open
alexgleason opened this issue Oct 11, 2017 · 10 comments
Labels
A-Room-Settings A-Timeline P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@alexgleason
Copy link

I invited someone to a room. The room history was set to "members only (since they were invited)." It didn't work. The person joined the room and was able to scroll up and see the history.

This was incredibly awkward and extremely bad! Please fix this!

@turt2live
Copy link
Member

I can confirm this.

My view:
image

@temp:t2l.io's view
image

@lukebarnard1
Copy link
Contributor

Ok, so @erikjohnston has explained that you can only change the history visibility for messages sent after the history visibility change. Riot needs to spell this out much more clearly I think.

@lampholder
Copy link
Member

I post this only to highlight how insufficient a pop up is for communicating this very important message, since I think what we have here is (yet more) evidence that popups are essentially "invisible".

Anyway, when you make this change Riot pops up a warning explaining exactly what's going to happen:
image

@lampholder lampholder added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround P1 ui/ux labels Oct 12, 2017
@turt2live
Copy link
Member

To be honest, I didn't even read the dialog in my testing. I just assumed it wasn't a problem, and was surprised when the bug could be reproduced.

Some nice bold/red text explaining it would probably catch the eye enough to reconsider skimming the dialog.

@marykatefain
Copy link

I don't think that popup exists at all on mobile. I just tested it myself because I swear I never saw that before... I use riot almost exclusively on mobile.

@alexgleason
Copy link
Author

I changed the history visibility on Android which did not display this popup message.

For what it's worth, I've seen the message on desktop and even then misunderstood what it meant.

@lampholder
Copy link
Member

Yeah - popups are awful, esp at conveying important information like this.

So let's leave this bug scoped to doing something better in Riot web - I'll make another bug for android.

@lampholder
Copy link
Member

@lampholder lampholder changed the title Room history settings don't work The UX doesn't make it clear that room history changes only affect events written _after_ the settings change was made. Sep 12, 2018
@ara4n
Copy link
Member

ara4n commented Sep 12, 2018

The root cause of this is https://github.com/matrix-org/matrix-doc/issues/533

@hex-m
Copy link

hex-m commented Sep 23, 2020

To be honest, I didn't even read the dialog in my testing. I just assumed it wasn't a problem, and was surprised when the bug could be reproduced.

Some nice bold/red text explaining it would probably catch the eye enough to reconsider skimming the dialog.

In the latest version (Element 1.7.7 on Desktop) there isn't even a pop-up anymore. This is a serious privacy/security issue!

Screenshot from 2020-09-23 15-56-04

Screenshot from 2020-09-23 16-02-07

Making the explanation (or the word future) red/bold would be a good first step.

An even better solution would be to change the invite workflow and show a preview of what an invited user would see when they join.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Room-Settings A-Timeline P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

9 participants