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

Fix DialPad and Call Menu buttons not working when a call is fullscreened. #6496

Merged
merged 2 commits into from
Jul 29, 2021

Conversation

anoadragon453
Copy link
Member

Fixes element-hq/element-web#17175

This PR:

  • Moves the dialpad and hold/transfer menu buttons into the part of the DOM that's included when a call is fullscreen'd.
  • Tweaks ContextMenu to allow menu content to be mount as a child of the current component, rather than at the root of the DOM (which was not included in the full-screen'd content).

Signed-off-by: Andrew Morgan <[email protected]>

…urr. container

By default, ContextMenu's are children of mx_ContextualMenu_Container, which is placed at the
root of the DOM. A new prop, mountAsChild, allows for ContextMenus to be mounted as children
of the current container instead.

This is necessary as mx_ContextualMenu_Container won't be included in the content that's
shown when a call is fullscreened, and could potentially be useful elsewhere.
@anoadragon453 anoadragon453 added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems label Jul 28, 2021
@anoadragon453 anoadragon453 requested a review from a team July 28, 2021 15:52
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.

Looks sane to me!! :shipit:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When 1:1 video call is full screen, the dialpad and "three dots" buttons don't work
2 participants