Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New room list: VoIP + Jitsi #14177

Closed
2 tasks done
turt2live opened this issue Jun 24, 2020 · 5 comments · Fixed by matrix-org/matrix-react-sdk#4904
Closed
2 tasks done

New room list: VoIP + Jitsi #14177

turt2live opened this issue Jun 24, 2020 · 5 comments · Fixed by matrix-org/matrix-react-sdk#4904
Assignees

Comments

@turt2live
Copy link
Member

turt2live commented Jun 24, 2020

Has design:

  • Incoming call box (1:1 calls only)
  • Jitsi floating conference box (bottom right)

This issue does not scope in decorations for in-progress calls, doesn't change the ringing noise, and doesn't deal with anything that isn't feature parity.

@niquewoodhouse
Copy link

Attaching incoming call box for reference.

Screenshot 2020-06-25 at 19 10 19

Figma link

@JorikSchellekens
Copy link
Contributor

JorikSchellekens commented Jul 1, 2020

We need some design for the active calls as well. The following is what we currently have:

image
One to one voice call which is currently active ^. Clicking on this will take you to the right room.

image
One to one video call. The black box above is a placeholder for video

We're decided to have jitsi calls floating to the bottom right of the room view (see initial comment). This raises the question of where the new call box should be: Should it be where the call view will be (bottom right) or should it be closer to the room list which groups it cognitively with the list of people.
Also where should the active call box be?

@turt2live
Copy link
Member Author

We're going for parity. Do 1:1 calls currently show up in the room list?

@JorikSchellekens
Copy link
Contributor

Yes they do

@turt2live
Copy link
Member Author

godwhy.jpg

I suppose we stick them in the lower right with the jitsi then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants