Skip to content
This repository has been archived by the owner on Dec 11, 2021. It is now read-only.

[Bug report] "deleted" tag overlap with message content #66

Open
zAlweNy26 opened this issue Jul 16, 2021 · 4 comments
Open

[Bug report] "deleted" tag overlap with message content #66

zAlweNy26 opened this issue Jul 16, 2021 · 4 comments
Labels
bug Something isn't working

Comments

@zAlweNy26
Copy link
Contributor

zAlweNy26 commented Jul 16, 2021

Describe the bug

Overlaid text when both edited and deleted

Steps to reproduce

  1. Ask someone to edit a message
  2. Ask them to delete it
  3. Watch the result

Expected behaviour

It should be spaced so that the text does not overlap, just like this :
Telegram_KXkWGE2E3c

Actual behaviour

The text of the message overlay the "eliminated edited" string like this :
Telegram_KXkWGE2E6c
"eliminato" means eliminated
"modificato" means edited
Example in italian, sorry.

Configuration

  • Operating system: Windows 10
  • Version of Telegram: 2.8.8 x64
  • Version of the plugin: 0.4.0

I suggest that when an edited message is deleted, the string "edited" is replaced with "eliminated" or just make it like the expected behaviour.

@zAlweNy26 zAlweNy26 changed the title [Bug report] Modified then eliminated [Bug report] Message edited then eliminated Jul 16, 2021
@SpriteOvO SpriteOvO added the bug Something isn't working label Jul 16, 2021
@SpriteOvO
Copy link
Owner

Thanks for reporting!

Displaying edited is the expected behavior, the solution to this bug is to change some UI value related to the width, I will try to fix it when I have time.

@SpriteOvO SpriteOvO changed the title [Bug report] Message edited then eliminated [Bug report] "deleted" tag overlap with message content Jul 16, 2021
@zAlweNy26
Copy link
Contributor Author

zAlweNy26 commented Jul 16, 2021

Is this the line of code that changes the width?

pMainView->SetWidth(pMainView->GetWidth() + _MarkData.Width);

@SpriteOvO
Copy link
Owner

Is this the line of code that changes the width?

pMainView->SetWidth(pMainView->GetWidth() + _MarkData.Width);

No, it's not anywhere in this repository currently, to fix this bug would require reverse engineering to get a new width location and then writing code to modify it.

@zAlweNy26
Copy link
Contributor Author

Oh ok, my bad, sorry !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants