Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Move call buttons to the room header #5693

Merged
merged 3 commits into from
Feb 26, 2021
Merged

Conversation

turt2live
Copy link
Member

This is to make some room in the composer for voice messages. The hangup behaviour is intentionally lost by this change as the VOIP UX is intended to rely on dedicated hangup buttons instead.

New look:
image
image

This is to make some room in the composer for voice messages. The hangup behaviour is intentionally lost by this change as the VOIP UX is intended to rely on dedicated hangup buttons instead.
@turt2live turt2live requested a review from a team February 26, 2021 20:50
Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I dislike it but code looks sane :(

@t3chguy
Copy link
Member

t3chguy commented Feb 26, 2021

DOUBLY APPROVED

@turt2live
Copy link
Member Author

let's hope the CI agrees

@turt2live
Copy link
Member Author

oh wow, it does.

/me merges before it changes its mind

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

Successfully merging this pull request may close these issues.

3 participants