Swiftgram / Telegram-iOS

Supercharged Telegram fork for iOS from original creator of Nicegram
https://apps.apple.com/app/apple-store/id6471879502?pt=126511626&ct=gh&mt=8
94 stars 6 forks source link

Fix: Sending custom emojis adds underlying emojis to recently used #2

Closed levochkaa closed 2 weeks ago

levochkaa commented 2 weeks ago

Copy of https://github.com/TelegramMessenger/Telegram-iOS/pull/1328

Description: When sending Custom Emojis, their underlaying plain emojis get added to Recently Used. As far as I can tell from git blame, the bug is in the code for 9 months, before that plain emojis weren't added to recently used at all.

Bug in action:

https://github.com/TelegramMessenger/Telegram-iOS/assets/62802017/dafc1fe4-fa50-43de-b698-ac08d215e744

Fixed:

https://github.com/TelegramMessenger/Telegram-iOS/assets/62802017/b8be8f40-65f6-4e94-b266-8a5b172af0a0

Steps to reproduce:

Expected: Only Custom Emojis get added to the Recently Used group Reality: Both the custom emoji and plain underlaying emoji get added

Kylmakalle commented 2 weeks ago

Added here. Thank you!! https://github.com/Swiftgram/Telegram-iOS/commit/0d439a5479395d2a47d80b61047447e5878ab7ab