zfkun / ComfyUI_zfkun

A collection of nodes for common tools, including text preview, text translation (multi-platform, multi-language), image loader, webcamera capture, share screen capture
MIT License
19 stars 1 forks source link

screen node no popup on operaGX #2

Closed Ferniclestix closed 7 months ago

Ferniclestix commented 8 months ago

hi, love your screen share node, i think its really cool. in fact I did a tutorial on using it :D https://youtu.be/DZ-n2izkYYI Unfortunately I've just updated comfyUI and the popup no longer appears when clicking share screen button. the clip area also appears not to function.

Fully up to date comfyUI running on operaGX

Checked on Firefox as working

zfkun commented 7 months ago

During development, I found that if sharing is initiated and closed too frequently, there is a certain probability that chrome will not pop up the sharing confirmation pop-up.

There is not much time for in-depth analysis of this problem, the easiest way is to restart the operating system. It's not a great experience, but it works for me.

zfkun commented 7 months ago

i tried operaGX, I found that there is indeed a certain compatibility problem. I will test and correct operaGX as soon as possible.

Thank you for your reminder. because i usually use chrome more. operaGX has not been tested in advance. I am very sorry. - -!

@Ferniclestix

zfkun commented 7 months ago

@Ferniclestix

the main reason for the problem is that operaGX does not have screen recording permission. when clicking the Start Share Screen button, other windows or screens other than the opera browser window cannot be obtained.

if your are mac

  1. open setting window

  2. find Privacy & Security -> Screen Recording image

  3. turn on for Opera GX

    make sure operagx has screen recording permissions. image

then, ShareScreen node will working after click Start Share Screen button

image

I'll also refine the code, adding friendly hints and error handling to remind users that they need to manually set system permissions when they need them.

thanks ~

Ferniclestix commented 7 months ago

The odd thing about it erroring in operaGX is that the previous version worked fine in operaGX. screensharing was fine. then I updated and it no longer worked. Ill check to see if there is some system permission that got flipped but this is me checking on two previously working computers after doing various restarts and having a previously working setup using operaGX.

It may be an issue that my mouse has intermittant left click missfires after clicking which causes instant double clicks every now and again so maybe two issues are combining to make it fail.

zfkun commented 7 months ago

Is your operating system MacOS?

Today I also released a simple update, adding some necessary error tips, and I suggest you try it out.

To test this out, I downloaded the brand new operaGX today, and after completing the system authorization for screen recording, everything works fine.

If it doesn't work, you can open developer tools and see if there are error messages in the console.

Or, like me, make sure there are no other custom plugins, make sure there are only a few custom plugins in comfyui, and run a test to see if it works

image

image

image

image

Ferniclestix commented 7 months ago

I use windows, update fixed it, thanks :D also have a new mouse so single clicking for sure lol.