Skip to content
New issue

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

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

Already on GitHub? # to your account

Offloaded chat bubble logic to a chatbubble class #14

Merged
merged 1 commit into from
Apr 19, 2022
Merged

Conversation

Avi12910
Copy link
Collaborator

Added numerous style improvements

Processes gif messages

Added numerous style improvements

Processes gif messages
@netlify
Copy link

netlify bot commented Apr 19, 2022

Deploy Preview for competent-knuth-a385f3 ready!

Name Link
🔨 Latest commit 9d7d536
🔍 Latest deploy log https://app.netlify.com/sites/competent-knuth-a385f3/deploys/625e2009b070c70008feadaa
😎 Deploy Preview https://deploy-preview-14--competent-knuth-a385f3.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@al-chang
Copy link
Contributor

Looks good, not sure how easy or hard this would be, but what is the viability of making it so the chat bubble is centered over the player's head? I imagine it might require some math based on the length of the message.

@al-chang al-chang merged commit c76f54a into main Apr 19, 2022
@al-chang al-chang deleted the chat-bubble branch April 19, 2022 17:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants