Skip to content

Conversation

@ReedyBear
Copy link

I've updated the emotes.json and emotes.png to have bear waving emote

I requested this change at #12 (comment)

Sorry about the messy PR. Haven't done a lot of these.

I used the same timing as meow, because I tested on my machine by replacing the meow images with bear images.
I previously tested this on my machine by replacing the kitty meows with the bear waves, and it worked. Have not tested this new setup where bears have their own spot and are added to the json file.
Add "bear" to emotes.json
@ReedyBear
Copy link
Author

I have tested & confirmed that it works on my machine.

@ReedyBear
Copy link
Author

In case BetterChat does not integrate my suggestions, for anyone who wants to replace meow emote with bear emote, see https://github.com/ReedyBear/tm-better-chat/tree/replace-meow-with-bear ... this will only change it on your screen. 🐻

@codecat
Copy link
Owner

codecat commented Dec 21, 2024

The PR itself looks good, but this makes it easy to accidentally type bear and have the emote shown. I don't believe this emote is very popular either (I've never seen it used before). I don't know if it makes sense to add this to the global list tbh.

@ReedyBear
Copy link
Author

Perhaps "bearhello" or "hibear" or "bearwave" would be better? I like hibear.

I totally respect that, I haven't seen people use it either. I will, however, present some points in favor of adding bear wave:

  • Bears are cute
  • There is only one other furry gif emote
  • There is a lot of spare space left on the emotes board, and no emotes have been added this year.
  • It could be given a name like BearTest to signify that it might be removed in a later patch
  • Maybe people would use it if it were available? Maybe? I would use it! lol
  • four frames of a bear wave

Again, I totally respect if you choose not to add it! I had to make my case though :D

And big thanks for making BetterChat. Also, v glad kitty meow is available :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants