From 3ac436bdb9bd7a30463eae44a9c37ddab0784609 Mon Sep 17 00:00:00 2001 From: sylenien Date: Tue, 13 Dec 2022 16:38:37 +0100 Subject: [PATCH] change(player): dont show cursor icon if session is played on mobvile --- frontend/app/player/web/Screen/Cursor.ts | 3 ++- frontend/app/player/web/Screen/Screen.ts | 4 ++-- frontend/app/player/web/WebPlayer.ts | 4 ++-- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/frontend/app/player/web/Screen/Cursor.ts b/frontend/app/player/web/Screen/Cursor.ts index 4d8094b4e..83dba4ab1 100644 --- a/frontend/app/player/web/Screen/Cursor.ts +++ b/frontend/app/player/web/Screen/Cursor.ts @@ -5,9 +5,10 @@ import styles from './cursor.module.css'; export default class Cursor { private readonly cursor: HTMLDivElement; private tagElement: HTMLDivElement; - constructor(overlay: HTMLDivElement) { + constructor(overlay: HTMLDivElement, isMobile: boolean) { this.cursor = document.createElement('div'); this.cursor.className = styles.cursor; + if (isMobile) this.cursor.style.backgroundImage = 'unset' overlay.appendChild(this.cursor); } diff --git a/frontend/app/player/web/Screen/Screen.ts b/frontend/app/player/web/Screen/Screen.ts index b1ceff509..043be5357 100644 --- a/frontend/app/player/web/Screen/Screen.ts +++ b/frontend/app/player/web/Screen/Screen.ts @@ -57,7 +57,7 @@ export default class Screen { private readonly screen: HTMLDivElement; private parentElement: HTMLElement | null = null; - constructor() { + constructor(isMobile: boolean) { const iframe = document.createElement('iframe'); iframe.className = styles.iframe; this.iframe = iframe; @@ -73,7 +73,7 @@ export default class Screen { screen.appendChild(overlay); this.screen = screen; - this.cursor = new Cursor(this.overlay) // TODO: move outside + this.cursor = new Cursor(this.overlay, isMobile) // TODO: move outside } attach(parentElement: HTMLElement) { diff --git a/frontend/app/player/web/WebPlayer.ts b/frontend/app/player/web/WebPlayer.ts index 8df6214d4..e80964b0a 100644 --- a/frontend/app/player/web/WebPlayer.ts +++ b/frontend/app/player/web/WebPlayer.ts @@ -31,7 +31,7 @@ export default class WebPlayer extends Player { private targetMarker: TargetMarker constructor(private wpState: Store, session, config: RTCIceServer[], live: boolean) { - + const isMobile = session.userOs === 'iPhone' || session.userOs === 'Android' let initialLists = live ? {} : { event: session.events.toJSON(), stack: session.stackEvents.toJSON(), @@ -46,7 +46,7 @@ export default class WebPlayer extends Player { ), } - const screen = new Screen() + const screen = new Screen(isMobile) const messageManager = new MessageManager(session, wpState, screen, initialLists) super(wpState, messageManager) this.screen = screen