Large emojis in post preview of bookmarks

UX
,
1

Steps to reproduce:

  1. Bookmark an official plugin or theme component (It works with some non-official ones too)
  2. Visit https://meta.discourse.org/my/activity/bookmarks

large emojis
large emojis1618×250 22.7 KB

I think small emojis would fit better.

small emojis
small emojis1619×173 17.9 KB

3 Likes
2

Just a bump on this as its still present now.

image
image1682×409 53.4 KB

2 Likes
3

@renato I was about to apply a CSS fix but those emoji’s actually get a class only-emoji they should not have.

I’ve found this from 5y ago

But with the recent changes with prosemirror, I think things may have changed?

Anyway, would prefer to properly fix it instead of applying a CSS bandaid.

1 Like
5

This isn’t to do with the editor, it’s to do with the post excerpts. For this list we use:

And because the OP of these theme topics have tables, those get stripped out, and we end up with this kind of thing:

:discourse2:
Summary
Topic Cards restyle the topic list to display as cards with a thumbnail


<img src="//localhost:4200/images/emoji/twitter/eyeglasses.png?v=14" title=":eyeglasses:" class="emoji only-emoji" alt=":eyeglasses:" loading="lazy" width="20" height="20">
Preview
<a href="https://discourse.theme-creator.io/theme/Discourse/topic-cards">Preview on Discourse Theme Creator</a>


<img src="//localhost:4200/images/emoji/twitter/hammer_and_wrench.png?v=14" title=":hammer_and_wrench:" class="emoji only-emoji" alt=":hammer_and_wrench:" loading="lazy" width="20" height="20">
Repository
<a href="https://github.com/discourse/discourse-topic-cards">https://github.com/discourse/discourse-topic-cards</a>


<img src="//localhost:4200/images/emoji/twitter/open_book.png?v=14" title=":open_book:" class="emoji only-emoji" alt=":open_book:" loading="lazy" width="20" height="20">
New to Discourse Themes?
<a href="https://meta.discourse.org/t/beginners-guide-to-using-discourse-themes/91966">Beginner’s guide to using Discourse Themes</a>


Wow topic

So technically the only-emoji is correct here, but practicallly it’s wrong. I think this could be fairly tricky to fix…maybe we need a no_big_emoji option for PrettyText.cook…but IDK for sure.

1 Like