UX: adds emoji in the table-title of admin-report-table (PR #12687)

@jjaffeux

GitHub

Can you just try to convert emoji code from title with emojiUnescape before displaying it ?

Also we just need the emoji, no need for text + emoji

Can you just try to convert emoji code from title with emojiUnescape before displaying it ? I think converting emoji from the title might act weird when coincidentally title is an emoji name. Then we will display the emoji and hide the title. thoughts?

No it’s ok to convert. we have the helper for it, just use it. If we have :laughing: in a text, it’s expected to be converted

Sure, that’s right :sweat_smile:

I wonder if you can’t actually convert it directly from discourse-reactions code

I wonder if you can’t actually convert it directly from discourse-reactions code

From discourse reactions, we don’t need to convert, as we are taking titles that are entered using emoji-picker

Also, In this our first column is a date, and if we take the title as emoji, it converts date to an emoji. Is it okay?

Screenshot 2021-04-16 at 5 29 28 PM

@jjaffeux Added emoji’s img tag from backend & done minimal changes here. Do have a look

A commit that appears in this pull request is being discussed here.

A commit that appears in this pull request is being discussed here.

A commit that appears in this pull request is being discussed here.

A commit that appears in this pull request is being discussed here.