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

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