pelita icon indicating copy to clipboard operation
pelita copied to clipboard

Bot.say with emojis

Open refactoriel opened this issue 5 years ago • 2 comments

Bot.say method should accept emojis.

Currently it crashes the game.

refactoriel avatar Sep 07 '19 18:09 refactoriel

I’d be open for an implementation (hack) that makes this possible (https://stackoverflow.com/questions/52228940/how-can-i-enable-support-for-emoji-in-tkinter-applications) but I’m not sure about the others. (Also, I am not super confident it will work in all cases without further problems :sadface:.)

Debilski avatar Sep 17 '19 11:09 Debilski

I'd go the other way and make bot.say as strict as the TEAM_NAME. Only a selected sub-set of ASCII should be allowed. Simply to avoid edge-case hunters/students to find nice ways to crash the game. TK is not up to a proper/gracefully-failing/full Unicode support, so we should just acknowledge that and move on.

On Tue 17 Sep, 04:13 -0700, Rike-Benjamin Schuppner [email protected] wrote:

I’d be open for an implementation (hack) that makes this possible (https://stackoverflow.com/questions/52228940/how-can-i-enable-support-for-emoji-in-tkinter-applications) but I’m not sure about the others. (Also, I am not super confident it will work in all cases without further problems :sadface:.)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub¹, or mute the thread².☘

––––

¹ https://github.com/ASPP/pelita/issues/691?email_source=notifications&email_token=AACUYCYWKGR6WJO62I6OPALQKC3VZA5CNFSM4IUQ5B62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD64FVMY#issuecomment-532175539 ² https://github.com/notifications/unsubscribe-auth/AACUYCYN45QLUQOIOOCK6RDQKC3VZANCNFSM4IUQ5B6Q

otizonaizit avatar Sep 17 '19 14:09 otizonaizit