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

Lag suffix, can be applied to an emote to save space #116

Merged
merged 6 commits into from
Jul 7, 2019
Merged

Lag suffix, can be applied to an emote to save space #116

merged 6 commits into from
Jul 7, 2019

Conversation

WhiteNervosa
Copy link
Contributor

@WhiteNervosa WhiteNervosa commented Jul 5, 2019

The lag suffix can be changed to just be an emote animation on one frame to save space

@jbpratt jbpratt requested review from jbpratt, kenany and xDashh July 5, 2019 11:31
Copy link
Member

@jbpratt jbpratt left a comment

Choose a reason for hiding this comment

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

Commented on the shortening of links. Rest LGTM

assets/chat/js/formatters.js Outdated Show resolved Hide resolved
@WhiteNervosa WhiteNervosa changed the title Visually shorten links in chat; lag suffix to make any emote lag Visually shorten links in chat; lag suffix to make any emote lag; Frozen Suffix Added Jul 5, 2019
@WhiteNervosa WhiteNervosa changed the title Visually shorten links in chat; lag suffix to make any emote lag; Frozen Suffix Added Visually shorten links in chat; lag suffix to make any emote lag; Frozen/Wide Suffix Jul 5, 2019
@WhiteNervosa
Copy link
Contributor Author

WhiteNervosa commented Jul 6, 2019

I just now understood what JBPratt meant #82 , I'll split all of these into separate branches so its easier for you guys later today. I'm learning Git, my apologies.

@WhiteNervosa WhiteNervosa changed the title Visually shorten links in chat; lag suffix to make any emote lag; Frozen/Wide Suffix Lag suffix, can be applied to an emote to save space Jul 6, 2019
@xDashh
Copy link
Member

xDashh commented Jul 7, 2019

I personally generally dislike taking the main meme of an existing emote and generifying it (as e.g. with dank), but I can also see some good meme potential out of this, so let's try I guess.

@jbpratt78 your changes were addressed? Also, is the timing the same as currently on LAG emote?

@jbpratt
Copy link
Member

jbpratt commented Jul 7, 2019

@xDashh yep, looks good. I agree on the concern but am down to try it out.

@xDashh
Copy link
Member

xDashh commented Jul 7, 2019

@jbpratt78 you should be able to approve and merge (if it makes sense to merge this before others, I'm not sure)

@jbpratt jbpratt merged commit 37569eb into MemeLabs:master Jul 7, 2019
@jbpratt
Copy link
Member

jbpratt commented Jul 7, 2019

Merged, thank you!

SmCTwelve pushed a commit to SmCTwelve/chat-gui that referenced this pull request Jul 8, 2019
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