Skip to content

✨ Add top 10 emotes section#57

Open
tadhgboyle wants to merge 4 commits intoAndroz2091:masterfrom
tadhgboyle:master
Open

✨ Add top 10 emotes section#57
tadhgboyle wants to merge 4 commits intoAndroz2091:masterfrom
tadhgboyle:master

Conversation

@tadhgboyle
Copy link
Copy Markdown

How it looks:
Screen Shot 2021-12-03 at 7 10 55 PM

If someone spammed :pepeLa: 600 times in 1 message, their stats will be thrown off for this. Now it only would count that as one
Comment thread src/app/demo.js Outdated
@tadhgboyle
Copy link
Copy Markdown
Author

@Androz2091 ?

@louis1706
Copy link
Copy Markdown

How i can use it ? i am curious about this information

Copy link
Copy Markdown
Owner

@Androz2091 Androz2091 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution. Could you reset the changes to yarn.lock and reduce the number of shown emotes from 10 to 5 so the UI still looks nice on desktops without a huge gap?

@louis1706
Copy link
Copy Markdown

louis1706 commented Jul 8, 2024

Thank you for your contribution. Could you reset the changes to yarn.lock and reduce the number of shown emotes from 10 to 5 so the UI still looks nice on desktops without a huge gap?

can't you modify it for him ? it's would be faster ?

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.

4 participants