Skip to content

Feat/540-emoji-codes#2776

Merged
holmesworcester merged 40 commits intodevelopfrom
feat/540-emoji-codes
Apr 15, 2025
Merged

Feat/540-emoji-codes#2776
holmesworcester merged 40 commits intodevelopfrom
feat/540-emoji-codes

Conversation

@holmesworcester
Copy link
Copy Markdown
Collaborator

@holmesworcester holmesworcester commented Mar 21, 2025

Adds emojicodes (:heart:), emoticons (<3), tab completion, a dropdown, and actually bright red hearts, for better vibes. (Our hearts had been very 1980s ASCII goth.)

Pull Request Checklist

  • I have linked this PR to a related GitHub issue.
  • I have added a description of the change (and Github issue number, if any) to the root CHANGELOG.md.

…cause it was easier to start fresh--but who took a first pass on this and got something working! <3)
@holmesworcester
Copy link
Copy Markdown
Collaborator Author

@islathehut this is ready for review for reals now.

Comment thread CHANGELOG.md Outdated
Co-authored-by: Philip Durbin <philipdurbin@gmail.com>
Copy link
Copy Markdown
Collaborator

@adrastaea adrastaea left a comment

Choose a reason for hiding this comment

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

Approving despite my stylistic preference note. Will leave it up to you whether you agree and want to change that behavior before merging or make an issue to address later.

holmesworcester and others added 3 commits April 15, 2025 02:27
…nelInput/ChannelInput.tsx

Co-authored-by: Taea <88346289+adrastaea@users.noreply.github.com>
…ut leaves a note for posterity so they don't get added back when someone asks for them
@holmesworcester
Copy link
Copy Markdown
Collaborator Author

@adrastaea noting this here, too: f91cc9d

it's another funny issue with parsing emojis, this time with our linter!

@adrastaea adrastaea self-requested a review April 15, 2025 16:18
@holmesworcester holmesworcester merged commit 2dabce5 into develop Apr 15, 2025
25 of 28 checks passed
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.

3 participants