om/chatcontrol-live-demo
1
0
Fork 1
Code Issues Pull requests Projects Releases Wiki Activity

Draft: chat bubbles and some smaller UI improvements #1

Merged
om merged 4 commits from a2/chatcontrol-live-demo:asbjørn-additions into master 2022-08-17 16:30:59 +00:00
Conversation 1 Commits 4 Files changed 1 +55 -25
a2 commented 2022-08-17 13:42:38 +00:00
Collaborator
Copy link

This PR has a bunch of smaller improvements:

  • improve some Danish language stuff (only minor things)
  • put contacts pane to the left of the messages pane, making it closer to typical IM UX
  • make font size smaller
  • right-align "me" text
  • make chat text areas wide enough to allow horizontal overlap
  • add chat bubbles behind text

things that still need doing:

  • hide chat bubbles when scrolling (probably just by fixing render-order and putting another element on top?)
  • differ between line height within a message, and line height between messages (should be wider between two messages than between two lines in a multi-line message)
This PR has a bunch of smaller improvements: - improve some Danish language stuff (only minor things) - put contacts pane to the left of the messages pane, making it closer to typical IM UX - make font size smaller - right-align "me" text - make chat text areas wide enough to allow horizontal overlap - add chat bubbles behind text things that still need doing: - hide chat bubbles when scrolling (probably just by fixing render-order and putting another element on top?) - differ between line height within a message, and line height between messages (should be wider between two messages than between two lines in a multi-line message)
a2 added 4 commits 2022-08-17 13:42:39 +00:00 
this is the common ui pattern in most chat apps. I think it feels more natural.
a2 commented 2022-08-17 13:46:01 +00:00
Author
Collaborator
Copy link

It looks like this rn (but gets a bit worse when bubbles scroll out of view)

image

I think we also need to have two differen line-heights somehow.

It looks like this rn (but gets a bit worse when bubbles scroll out of view) ![image](/attachments/a9f25016-cf6b-4da7-904e-9fdcf3e34890) I think we also need to have two differen line-heights somehow.
image.png
17 KiB
om merged commit 7a1c75aa99 into master 2022-08-17 16:30:59 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: om/chatcontrol-live-demo#1
No description provided.
Delete branch "a2/chatcontrol-live-demo:asbjørn-additions"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?