Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove 0.05em padding from emoji #1617

Closed
wants to merge 1 commit into from

Conversation

WebFreak001
Copy link
Contributor

seems completely random and breaks chaining multiple custom emoji together in a row, as well as not being aligned to pixels.

before:

emoji chain

after:

emoji chain

seems completely random and breaks chaining multiple custom emoji together in a row
Copy link

github-actions bot commented Jan 5, 2024

Preview: https://1617--pr-cinny.netlify.app
⚠️ Exercise caution. Use test accounts. ⚠️

@ajbura
Copy link
Member

ajbura commented Jan 30, 2024

The padding between emoji's are on purpose to create good distinction between emoji's. for example two smile should not have their cheeks touching each other or a car and bus emoji put together doesn't look like they are in collision.

@WebFreak001
Copy link
Contributor Author

ok how about me making this a setting then? I personally strongly prefer having emoji touching since you can make creative emoji combinations (and in bridged discord rooms this is also how the other side sees them)

@ajbura
Copy link
Member

ajbura commented Jan 30, 2024

Please create an issue to discuss it.

@WebFreak001
Copy link
Contributor Author

#1634

@ajbura
Copy link
Member

ajbura commented Jan 10, 2025

closing as per #1634 (comment)

@ajbura ajbura closed this Jan 10, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jan 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants