Skip to content

[Story] Public content display policy #2787

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

Open
6 of 8 tasks
manuroe opened this issue Mar 14, 2025 · 5 comments
Open
6 of 8 tasks

[Story] Public content display policy #2787

manuroe opened this issue Mar 14, 2025 · 5 comments

Comments

@manuroe
Copy link
Member

manuroe commented Mar 14, 2025

Description

  • As a homeserver admin
  • I want to set the public media content display policy for my users
  • So that they can safely their matrix app even in the context of an open federation

Scope

The new server policy setting will have impact on:

  • Display of previews and thumbnails of media in timelines of public rooms
  • Display of invite avatars

Acceptance criteria

Dependencies

Out of scope

  • Nothing

Questions

  • Should we have separated settings, one for invite, one for images in the timeline? --> Yes
  • Is scoping the impact on public rooms good enough?
  • Invite: Should we display the avatars on tap? -> No
  • What should be the server setting value media_previews for m.org? Private may be safe enough --> it is going to be 'on'
  • Do we need work in the SDK or Ruma? --> Yes

Subtasks

Android

iOS

Web

Other

@Velin92
Copy link
Member

Velin92 commented Mar 18, 2025

when we say we want to hide avatars in invites we mean:

  • the avatar of the inviter in the room preview screen
  • and also the avatar of the room to which you have been invited (if the room is not a DM) in the room preview screen
  • the avatar of the invite cell in the room list screen
  • the avatar of the inviter in the invite cell in the room list screen
  • the avatar of the invite in the push notification

Also when we say hide the avatar, we are okay just not showing the actual image through the URL, but we are okay in displaying the placeholder initials right?
Or we want to use something like a removable blur? (this ofc won't work for the push notification, so I guess in that case we may need to just use the initials)

@manuroe @amshakal

@amshakal
Copy link

That's exactly right. Yes to all the scenarios you mentioned.

And yes, let's show initials instead of blurring.

@Velin92
Copy link
Member

Velin92 commented Mar 18, 2025

That's exactly right. Yes to all the scenarios you mentioned.

And yes, let's show initials instead of blurring.

Initials everywhere or just for the notification?

@Velin92
Copy link
Member

Velin92 commented Apr 28, 2025

@manuroe @mxandreas
Had a chat with @poljar about this, we would also need to add these two new values to Ruma to be able to support them, since they would need a default value when the hs does not provide them, and a specific enum type for each of them. So is not as easy as just adding them in the FFI layer, on the SDK side.

@Velin92
Copy link
Member

Velin92 commented May 12, 2025

We also need this: matrix-org/matrix-rust-sdk#4994

This PR would be very useful because it allows to onserve the account data value in real time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants