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

"@room" tags do not render anymore #26719

Closed
mcclure opened this issue Dec 8, 2023 · 14 comments · Fixed by matrix-org/matrix-react-sdk#12318
Closed

"@room" tags do not render anymore #26719

mcclure opened this issue Dec 8, 2023 · 14 comments · Fixed by matrix-org/matrix-react-sdk#12318

Comments

@mcclure
Copy link

mcclure commented Dec 8, 2023

Steps to reproduce

The "@‍room" tag, messaging everyone in the room, no longer shows up as a "tag" for me in Browser Element. It is just the word "@‍room". I believe this is a bug because it is a regression (it used to work, as recently as 2 months ago) and because in other clients the same "@‍room" instances show up as a tag.

Outcome

What did you expect?

I expect that if someone has room-notify permissions "@‍room" will appear in a special way, such as in an oval, to show it is a mention. I expect that if someone does NOT have these permissions, it will show up as just the text "@‍room". Thus, to me "@‍room" rendering special or not is an important signal because it shows whether or not the notify-all worked¹.

So for example this is what a friend of mine sees in Element for Linux (Flatpak, Element version: 1.11.50, Olm 3.2.15). This is my "expected behavior".

image

When I look in the Element Android client I see essentially the same.

What happened instead?

This is what I see in browser:

image

This screenshot was taken on the same account as the Android version which worked. Reloading the page did not change the appearance.


¹ It was a little inconvenient, I moderate a bimonthly meeting where I need to announce the beginning by typing "@‍room", it didn't seem to do anything, so while trying to ask others if it was working I typed "@‍room" several more times and now I think I was spamming people with messages.

Note: In the interest of not spamming the Github user named "Room" I have inserted zero-width spaces into all instances of "@‍room" in this message.

Operating system

Windows 10 with Chrome 119.0.6045.159 AND Linux Ubuntu 23.10 with Firefox 119.0 Snap

Browser information

URL for webapp

https://app.element.io/

Application version

Element version: 1.11.51 Crypto version: Olm 3.2.15

Homeserver

:matrix.org . I am uncertain how to tell what server a particular "room" is hosted on.

Will you send logs?

Yes

@mcclure
Copy link
Author

mcclure commented Dec 8, 2023

I submitted logs

@t3chguy
Copy link
Member

t3chguy commented Dec 11, 2023

I cannot see any logs from you.
Please provide a screenshot showing the permissions for the room, and the event JSON for the event with the @room text in your original screenshot

@t3chguy t3chguy added the X-Needs-Info This issue is blocked awaiting information from the reporter label Dec 11, 2023
@jjj333-p
Copy link

jjj333-p commented Dec 11, 2023

I can provide said info in a different room, as seen on the current app.element.io

example in chat from my end:
image

as you can see I am administrator, pl 100, but also screenshot of pl settings:
image

and the json event:

{
  "type": "m.room.message",
  "sender": "@jjj333_p_1325:envs.net",
  "content": {
    "msgtype": "m.text",
    "body": "@room test",
    "m.mentions": {}
  },
  "origin_server_ts": 1702320679027,
  "unsigned": {
    "age": 628,
    "transaction_id": "m1702320678526.59"
  },
  "event_id": "$_NmaqDTT8fmm_3DRNYQu5wyVh1M6BPvsWacOp4uDFgM",
  "room_id": "!odwJFwanVTgIblSUtg:matrix.org"
}

looking at the json it is strange that it doesnt have room in the mentions field.

@t3chguy
Copy link
Member

t3chguy commented Dec 12, 2023

image

I cannot reproduce it

@jjj333-p
Copy link

jjj333-p commented Dec 12, 2023 via email

@mcclure
Copy link
Author

mcclure commented Dec 13, 2023

I cannot see any logs from you. Please provide a screenshot showing the permissions for the room, and the event JSON for the event with the @room text in your original screenshot

Here is the json for one such event:

{
  "room_id": "[[redacted]]:matrix.org",
  "type": "m.room.message",
  "content": {
    "msgtype": "m.text",
    "body": "@room ",
    "m.mentions": {
      "room": true
    }
  }
}

image

image

@t3chguy
Copy link
Member

t3chguy commented Dec 15, 2023

Mine was on the latest app.element.io is that what you’re using?

I was using Element Nightly

@bmarty
Copy link
Member

bmarty commented Dec 20, 2023

I can repro on https://develop.element.io after a F5:

  • in a DM
  • in a room where I am an admin

It's OK in the composer:
image

Source of those Events:

{
  "content": {
    "body": "@room sent from ea",
    "msgtype": "m.text"
  },
  "origin_server_ts": 1703063474104,
  "sender": "@benoit.marty:matrix.org",
  "type": "m.room.message",
  "unsigned": {
    "age": 420586
  },
  "event_id": "$XXX",
  "room_id": "!XXX:matrix.org"
}
{
  "content": {
    "body": "@room sent from Web",
    "m.mentions": {
      "room": true
    },
    "msgtype": "m.text"
  },
  "origin_server_ts": 1703063486340,
  "sender": "@benoit.marty:matrix.org",
  "type": "m.room.message",
  "unsigned": {
    "age": 408399,
    "transaction_id": "m1703063486241.669"
  },
  "event_id": "$XXX",
  "room_id": "!XXX:matrix.org"
}

So the regression is probably related to the new m.mentions field.

@devosonder
Copy link

devosonder commented Dec 25, 2023

This problem still persists. Join a room at https://develop.element.io/ or https://app.element.io/ and send a mention with @room. You will see that it does not work. @t3chguy

Microsoft Edge
Version 120.0.2210.91 (Official build) (64-bit)

OS: Windows 11

image

@t3chguy
Copy link
Member

t3chguy commented Dec 29, 2023

@devosonder when you join a room you won't have sufficient permissions to send those. They require you to be a moderator or above by default.

@devosonder
Copy link

@devosonder when you join a room you won't have sufficient permissions to send those. They require you to be a moderator or above by default.

my role is manager.

@yajo
Copy link

yajo commented Jan 12, 2024

Also happening here. Everybody can ping everybody:

imagen

However I cannot ping them:
imagen

I'm admin on the room BTW.

@anarute
Copy link

anarute commented Feb 28, 2024

We can also reproduce this on element web latest version and by testing past versions it seems this started on 1.11.44

@t3chguy t3chguy self-assigned this Mar 6, 2024
@t3chguy t3chguy removed the X-Needs-Info This issue is blocked awaiting information from the reporter label Mar 6, 2024
@t3chguy
Copy link
Member

t3chguy commented Mar 6, 2024

Regressed by matrix-org/matrix-react-sdk#11602

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

Successfully merging a pull request may close this issue.

7 participants