Skip to content
This repository has been archived by the owner on Feb 7, 2020. It is now read-only.

Emoji Picker feature #55

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Emoji Picker feature #55

wants to merge 5 commits into from

Conversation

DevFlex
Copy link

@DevFlex DevFlex commented Apr 21, 2018

No description provided.

DevFlex and others added 5 commits April 9, 2018 18:35
… that when you refresh it shows all of the notifications again. It also needs some touch up work around the requesting user permission flow for the notifications.
…use this code already completed the notification feature and mine was just half complete and therefore no longer needed.
… changed the anchor tag to a button because I forgot to change it back when I was debugging.
@DevFlex
Copy link
Author

DevFlex commented Apr 21, 2018

Hey there are some updates we could add to this to make it better and if I have the time I definitely will, but right now if you put an emoji in the text input by itself it is still very small instead of having it get bigger like you see with slack. However, I have been pretty busy as I am just moving so @lukejacksonn or if anyone else wants to make those improvements that be cool.

})
}

updateCursorPostion(e) {

Choose a reason for hiding this comment

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

Looks like this name is missing the letter i

Should be updateCursorPosition

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants