-
Notifications
You must be signed in to change notification settings - Fork 212
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
[CIS-907] Add Working with the Composer Guide #1195
Conversation
34d676e
to
2e905a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good!
The guide should also show how to update the composer height, to show more lines (3 instead of 2), related message in slack |
2e905a1
to
60f42f4
Compare
Codecov Report
@@ Coverage Diff @@
## main #1195 +/- ##
=======================================
Coverage 91.40% 91.40%
=======================================
Files 219 219
Lines 9419 9419
=======================================
Hits 8609 8609
Misses 810 810
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
0dff854
to
08aedbd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really solid!
962a16a
to
0b54782
Compare
To follow the pattern of the rest of the view controllers
Disabling the suggestions should be on the recognizing of the commands, so it does not have to do the work for nothing
This will make sure there's enough padding on the right side of the composer
It includes an example on how to create a custom typing suggestion
b3673ac
to
e4f6ef5
Compare
Description of the pull request
How to Test
https://stream-composer-guide.loca.lt/chat/docs/sdk/ios/guides/working-with-the-composer (In case it is offline, please ask me on Slack to put it online)