Page MenuHomePhabricator

[native] Move render functions in ChatInputBar to function component
ClosedPublic

Authored by angelika on Fri, Nov 22, 4:44 AM.
Tags
None
Referenced Files
F3389112: D14018.id45959.diff
Fri, Nov 29, 5:36 PM
F3387939: D14018.id.diff
Fri, Nov 29, 11:53 AM
F3385496: D14018.id45958.diff
Fri, Nov 29, 12:40 AM
Unknown Object (File)
Wed, Nov 27, 3:06 PM
Unknown Object (File)
Tue, Nov 26, 10:07 PM
Unknown Object (File)
Tue, Nov 26, 9:05 PM
Unknown Object (File)
Tue, Nov 26, 9:05 PM
Unknown Object (File)
Tue, Nov 26, 9:05 PM
Subscribers

Details

Summary
Test Plan

Tested this diff stack by playing around with ChatInputBar on both iOS simulator and Android device:

  • focus the text input and verify the keyboard is up and the camera buttons are animated correctly
  • try to send the message, verify the send button is animated correctly
  • verify the text input can be unfocused and keyboard is hidden
  • verify the draft works: write text, navigate from the chat, navigate into the chat again, verify the text is kept and the $
  • verify the edit mode works by editing a message
  • try to close the chat while editing a message - the alert should be shown
  • try to join a thread
  • try to select typed text

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat requested changes to this revision.Sun, Nov 24, 6:50 PM
ashoat added inline comments.
native/chat/chat-input-bar.react.js
1210 ↗(On Diff #45958)

There's an error here... in the old version the !isEditMode check was checking the result of invoking the isEditMode function, but here you're just checking the function itself, which is always defined (so this check will always be false)

1260 ↗(On Diff #45958)

Same here

This revision now requires changes to proceed.Sun, Nov 24, 6:50 PM
This revision is now accepted and ready to land.Mon, Nov 25, 8:25 AM