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

Fix typing indicator - Round #2 #136

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

Conversation

heatherbooker
Copy link
Contributor

Description

This is the second try to merge #127 - the first try had a bug so we reverted without having time to investigate.

@kaxi1993 Really did all the heavy lifting here! Sorry for the delay in getting it out!

Checklist

  • [ x] Additions/changes follow the current code structure
  • [ x] All changes are related to a single topic
  • [ x] Demo tested (npm start) to ensure it still functions as expected, and/or
  • [ x] New features are demonstrated to the demo (leave checkbox blank if not applicable)
  • [ x] Readme/docs are updated
  • [ x] npm run lint passes (npm run lint -- --fix will autofix where possible)

kaxi1993 and others added 2 commits October 8, 2019 16:08
* Add typing indicator

* Update readme to include typing feature

* fix sample code issue in readme

* Update demo to include typing example

* Remove vendor prefixes from typing indicator style

* Update typing indicator prop name

* Add missing comma
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants