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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

void #5290

Closed
wants to merge 1 commit into from
Closed

void #5290

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 21, 2023

No description provided.

@ghost ghost self-requested a review as a code owner March 21, 2023 17:52
@CLAassistant
Copy link

CLAassistant commented Mar 21, 2023

CLA assistant check
All committers have signed the CLA.

@philknows
Copy link
Member

Hey @abebeos , thanks for your contribution. We value your opinion and will leave this PR open for more commenting from other users and maintainers. We generally won't merge something in that doesn't have a majority consensus on whether the problem being solved here is in fact, a problem. And that this PR rectifies this problem.

@ghost ghost mentioned this pull request Mar 21, 2023
@nflaig
Copy link
Member

nflaig commented Mar 21, 2023

Would be in favor of keeping the emojis as they convey additional information 鈩癸笍

(Just to clarify: those emojis are nicely blockable via e.g. AdBlockPlus - I do not see them anymore in my browser, the problem is solved for me personally.)

it's great that you found a solution to get rid of these abominations 馃槃

@wemeetagain
Copy link
Member

I like the emojis 馃ぁ

馃憥 from me

@q9f q9f closed this Mar 25, 2023
@ghost ghost changed the title remove low-quality icons void Mar 29, 2023
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.

None yet

5 participants