Skip to content

Commit

Permalink
No need to try and de-fullscreen before showing settings dialog.
Browse files Browse the repository at this point in the history
  • Loading branch information
jamesremuscat committed Jul 7, 2024
1 parent bdc1cc2 commit e340ceb
Showing 1 changed file with 1 addition and 15 deletions.
16 changes: 1 addition & 15 deletions src/modules/timingScreen/components/ServiceParameters.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import styled from "styled-components";
import { ListAlt } from "styled-icons/material";

import { useServiceManifest, useServiceState } from "../../../components/ServiceContext";
import { useFullscreenContext } from "../../../components/FullscreenContext";
import { Dialog, DialogBackdrop } from "../../menu/components/Dialog";
import { stopEventBubble } from "../../../utils";
import { Input } from "../../../components/Forms";
Expand Down Expand Up @@ -137,23 +136,10 @@ const MyButton = styled(Button)`
export const ServiceParameters = () => {
const dialog = useDialogState();

const fs = useFullscreenContext();

const show = useCallback(
() => {
if (fs.active) {
fs.exit();
}
dialog.toggle();
},
[dialog, fs]
);


return (
<>
<MyButton
onClick={show}
onClick={dialog.toggle}
title='Set session parameters'
>
<SettingsIcon />
Expand Down

0 comments on commit e340ceb

Please sign in to comment.