element-hq / element-hq/element-android

`m.location` body is neither a location description or a user-friendly content description

Open
#7,422 0 comments 0 reactions 0 assignees View on GitHub
A-Location O-Frequent S-Minor T-Defect
Dominant language
Kotlin
Stars
3.7k
Forks
917
PR merge metrics
No merged PRs in 30d

Description

### Steps to reproduce

1. Send your location to a room by selecting a random point on a map

### Outcome

#### What did you expect?

Element should send a `m.location` event, whose [`body` is "A description of the location e.g. ‘Big Ben, London, UK’, or some kind of content description for accessibility e.g. ’location attachment’."](https://spec.matrix.org/v1.4/client-server-api/#mlocation)

#### What happened instead?

Element copied the `geo_uri` in the `body`:

```json
{
"content": {
"body": "geo:,;u=",
"geo_uri": "geo:,;u=",
"msgtype": "m.location",
"org.matrix.msc1767.text": "geo:,;u=",
"org.matrix.msc3488.asset": {
"type": "m.self"
},
"org.matrix.msc3488.location": {
"description": "geo:,;u=",
"uri": "geo:,;u="
},
"org.matrix.msc3488.ts": 1666292534921
},
"origin_server_ts": 1666292535068,
"sender": "@:matrix.org",
"type": "m.room.message",
"unsigned": {
"age": 283
},
"event_id": "",
"room_id": ""
}
```

### Your phone model

_No response_

### Operating system version

_No response_

### Application version and app store

_No response_

### Homeserver

_No response_

### Will you send logs?

No

### Are you willing to provide a PR?

No

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.