search

twilio / twilio-video-app-ios

A collaboration application built with the Twilio Video iOS SDK
Apache License 2.0
248 stars 54 forks source link

VIDEO-3827: Message list UI #150

Closed timrozum closed 3 years ago

timrozum commented 3 years ago

https://issues.corp.twilio.com/browse/VIDEO-3891

Changes

  1. Added message list UI to chat screen. Grouping messages by sender and time is not included in this ticket so there is always one message per section.
  2. Reorganized other cell files in project.

Testing

  1. Sent messages from two devices to verify message bubbles are different colors.
  2. Sent different length messages to verify message bubble width and height adjust correctly.
  3. Sent many messages to verify list automatically scrolls to new message.
  4. Verified author is truncated correctly if it is really long
  5. This ticket does not include logic to group messages by sender and time but I did hack it to verify the UI handles grouping well and spacing is correct.

Simulator Screen Shot - iPhone 12 Pro - 2021-03-04 at 13 44 00

timrozum commented 3 years ago

I fixed the UI warning.

The list should be scrolled to the bottom when first displayed. But it's a little tricky so I will add it as acceptance criteria for the send message UI ticket. That ticket will involve other interesting list interactions with keyboard.