question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

custom emoji overrides default emojis in emoji picker

See original GitHub issue

Note by @timabbott: We support overriding default emoji with a custom image, but also want to make sure users know this is happening when they do it.

I think we probably do want to allow overriding the image for a standard emoji if one wants to. Perhaps we want to (A) warn that when one is doing this in the UI and require confirmation and (B) perhaps require that the uploading user be an organization administrator to do so? (From https://github.com/zulip/zulip/issues/16913#issuecomment-747817924).


Original report below

Server Version: 3.3

Expected Behavior

When users upload custom emojis and choose names for them, if the name was already taken by a default emoji, then it should not save

Actual Behavior

One user uploaded a custom emoji and named it cry, which happens to also be a default emoji code :cry: 😢 https://emojipedia.org/crying-face/

Then in the emoji picker the default one is suppressed and not shown Screen Shot 2020-12-17 at 5 32 27 PM

message value: :cry: 😢 Actual rendered msg: Screen Shot 2020-12-17 at 5 30 31 PM

Solution

Would it be possible to block custom emojis using default emoji names? Would that be too expensive to check?

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Reactions:1
  • Comments:7 (7 by maintainers)

github_iconTop GitHub Comments

1reaction
timabbottcommented, Dec 18, 2020

There’s a legitimate use case for this with an organization that wants to use a specific image. E.g. a company named “dragon” might want to use their logo for :dragon: rather than the standard image.

I think we probably do want to allow overriding the image for a standard emoji if one wants to. Perhaps we want to (A) warn that when one is doing this in the UI and require confirmation and (B) require that the uploading user be an organization administrator to do so?

0reactions
timabbottcommented, Apr 26, 2021

This was closed by #16913 via adding the confirmation dialog. I’m not sure there’s anything else we definitely want to do here, but if so, we should open as a new issue. (E.g. if we think overriding default emoji should be limited to moderators and above as separate from the existing setting controlling who can upload custom emoji, we could try to design something there; I’m not sure it’s worth doing, though).

Read more comments on GitHub >

github_iconTop Results From Across the Web

Emoji Picker | Stream Chat - React SDK Docs
The default EmojiPicker renders individual emoji objects using the NimbleEmoji component from emoji-mart. Our custom set of emojis will need to follow the ......
Read more >
Configure custom emoji - admins - Discourse Meta
Customize emojis​​ You can upload emojis to override the default too. Go to: Discourse Admin - Customize - Emoji . You can: Enter...
Read more >
Manage one-click emoji reactions for your workspace ... - Slack
Choose default emoji for one-click reactions​​ Select Settings & administration from the menu, then click Customize [workspace name]. Click the Emoji tab at...
Read more >
emoji-picker-preact - npm
Customize Styling​​ You can override the style of emoji-picker-react with pickerStyle props.
Read more >
Customizable Slack-like emoji picker for React
By default the Emoji component will not render anything so that the emojis' don't take space in the picker when not available. When...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found