MMM-Screencast icon indicating copy to clipboard operation
MMM-Screencast copied to clipboard

Fix issue #128

Open goldyfruit opened this issue 2 years ago • 10 comments

This fix is inspired by https://github.com/amirthapa27/MMM-Screencast

When autoCloseScript is called, it sends a message (mmm-screencast.exited) to the console. If the message is received then an IPC is emitted as well as a call to the app.quit() method.

goldyfruit avatar Oct 10 '22 17:10 goldyfruit

Hi @goldyfruit, I tried your version of MMM-Screencast but when I disconnect from sharing with MM (which I do with my smartphone) the YuoTube window is always visible. Am I wrong something?

Thanks.

Egnos87 avatar Nov 05 '22 10:11 Egnos87

@Egnos87 works, for me. Did you try the correct branch of my fork (fix_window_still_visible)?

goldyfruit avatar Nov 05 '22 11:11 goldyfruit

I did the git clone from here https://github.com/smartgic/MMM-Screencast/tree/fix_window_still_visible

When I press "Disconnect" from the YouTube app it actually disconnects but the YouTube window always remains open. image

Egnos87 avatar Nov 05 '22 11:11 Egnos87

@Egnos87 did you restart the mirror after cloning the repo?

goldyfruit avatar Nov 05 '22 13:11 goldyfruit

Yes, I have tried many ways but it always remains the youtube window open. What O.S. do you have? I have Buster ...

This is my config:

{ module: 'MMM-Screencast', position: 'bottom_right', // This position is for a hidden

and not the screencast window config: { position: 'center', height: 300, width: 500, x:710, y: 200, //200 castName: 'MagicMirror' } },

Egnos87 avatar Nov 05 '22 22:11 Egnos87

I'm running inside Debian Buster container; this is my configuration for the module:

        {
            classes: 'default',
            config: {
                castName: 'smartgic_mirror',
		position: 'center',
		height: 720,
		width: 1024
            },
            module: 'MMM-Screencast',
            position: 'middle_center'
        },

How do you do your git clone? This is how you should do it:

git clone https://github.com/smartgic/MMM-Screencast.git -b fix_window_still_visible

goldyfruit avatar Nov 06 '22 01:11 goldyfruit

@kevinatown could you please have a quick look to the PR?s I think this could help few people having the same issue. :+1:

goldyfruit avatar Nov 08 '22 15:11 goldyfruit