Edited bubble broken

[VolkerJunginger]

Steps to reproduce

Create a text message. Edit it. You'll see that the "Edited" hint & the text overlaps.

While testing it last week (cw50) worked already in an older build.

Outcome

What did you expect?

What happened instead?

Your phone model

No response

Operating system version

No response

Application version and app store

No response

Homeserver

No response

Will you send logs?

Yes

Are you willing to provide a PR?

Yes

[bmarty]

@VolkerJunginger I cannot repro, but maybe not searching in the right direction. Can you attach a screenshot to the issue please?

[VolkerJunginger]

[VolkerJunginger]

see 2nd and third message.

[jmartinesp]

The way how this works was slightly changed when we replaced the rendering of text timeline messages and added mentions, but maybe it was enough to break this.

[bmarty]

(maybe due to the German word for edited which is longer, but I think we measure the translated word)

[jmartinesp]

Technically, the best solution would be to edit the RTE library to be able to have an onTextLayout callback and use something like this, but in the meanwhile I'll add a fix for the current 'add NBSP chars' solution.

[VolkerJunginger]

bug is still there w. nightly from 21.12.

[jmartinesp]

It seems to work fine for me in German in the last nightly:

Screenshot

Can you share an example of a case where it doesn't work?

[manuroe]

@VolkerJunginger can you check it again?

[VolkerJunginger]

It is still broken.

[jmartinesp]

Not for me:

There may be some corner case that causes the issue, but we haven't seen it anywhere else, so I don't think that's the case.

Are you sure you're using the latest version?

Oh, I just reproduced it. It has to be edited in the EXA app and the previous text must be close to the timestamp. When this happens, for some reason the bubble doesn't resize.