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
F3396083: D14018.id45958.diff
Sun, Dec 1, 10:17 AM
Unknown Object (File)
Sun, Dec 1, 12:55 AM
Unknown Object (File)
Sat, Nov 30, 7:48 PM
Unknown Object (File)
Sat, Nov 30, 5:57 PM
Unknown Object (File)
Sat, Nov 30, 5:56 PM
Unknown Object (File)
Sat, Nov 30, 12:54 PM
Unknown Object (File)
Sat, Nov 30, 10:48 AM
Unknown Object (File)
Fri, Nov 29, 11:39 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
Lint Not Applicable
Unit
Tests Not Applicable

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