search

vocatwk / CodeU-Nemo

Chat web app for CodeU
Apache License 2.0
1 stars 3 forks source link

Added profile pictures #129

Closed vocatwk closed 6 years ago

vocatwk commented 6 years ago

The page looks a little awkward when there's not that much information to display. Let me know if you guys have any suggestions or changes.

Also I forgot to mention that you only get a badge next to the conversation with the number of messages not yet seen only if you've ever opened the conversation before.

User1's view

screen shot 2018-08-03 at 1 51 25 am

User4's view

screen shot 2018-08-03 at 1 55 07 am screen shot 2018-08-03 at 1 52 21 am

User1's view

screen shot 2018-08-03 at 1 51 52 am screen shot 2018-08-03 at 1 52 14 am
jasminechau2 commented 6 years ago

I think it looks good

lealgamb commented 6 years ago

LGTM. Perhaps adding Bootstrap's <ul> with the class list-group can make the messages stand out more. If you want I can work on this since I have to update the profiles to make sure messages only appear if the logged in user is part of the conversation.

vocatwk commented 6 years ago

@lealgamb Yeah, please feel free to change things up 👍