imterah/element-portable
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #6593 from SimonBrandner/fix/correct-button

Browse source 
Use the correct button component for the Call View sidebar button
This commit is contained in:
Travis Ralston 2021-08-12 08:54:53 -06:00 • committed by GitHub
commit 772b8fb3fa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/components/views/voip/CallView/CallViewButtons.tsx
View file

@ -21,7 +21,6 @@ import classNames from "classnames";
import AccessibleTooltipButton from "../../elements/AccessibleTooltipButton"; import AccessibleTooltipButton from "../../elements/AccessibleTooltipButton";
import CallContextMenu from "../../context_menus/CallContextMenu"; import CallContextMenu from "../../context_menus/CallContextMenu";
import DialpadContextMenu from "../../context_menus/DialpadContextMenu"; import DialpadContextMenu from "../../context_menus/DialpadContextMenu";
import AccessibleButton from "../../elements/AccessibleButton";
import { MatrixCall } from "matrix-js-sdk/src/webrtc/call"; import { MatrixCall } from "matrix-js-sdk/src/webrtc/call";
import { Alignment } from "../../elements/Tooltip"; import { Alignment } from "../../elements/Tooltip";
import { import {
@ -211,10 +210,12 @@ export default class CallViewButtons extends React.Component<IProps, IState> {
let sidebarButton; let sidebarButton;
if (this.props.buttonsVisibility.sidebar) { if (this.props.buttonsVisibility.sidebar) {
sidebarButton = ( sidebarButton = (
<AccessibleButton <AccessibleTooltipButton
className={sidebarButtonClasses} className={sidebarButtonClasses}
onClick={this.props.handlers.onToggleSidebarClick} onClick={this.props.handlers.onToggleSidebarClick}
aria-label={this.props.buttonsState.sidebarShown ? _t("Hide sidebar") : _t("Show sidebar")} title={this.props.buttonsState.sidebarShown ? _t("Hide sidebar") : _t("Show sidebar")}
alignment={Alignment.Top}
yOffset={TOOLTIP_Y_OFFSET}
/> />
); );
} }