joaomgcd / JoinChrome

Chrome Extension for Join
164 stars 20 forks source link

Join pop out window not working #120

Closed betterbydesign closed 2 years ago

betterbydesign commented 2 years ago

I noticed after updating Chrome now when I try to open Join in its own pop out window it doesn't do anything. Theres no Chrome error about it being blocked either.

joaomgcd commented 2 years ago

Which version of Chrome are you using?

betterbydesign commented 2 years ago

Version 102.0.5005.61 (Official Build) (arm64)

joaomgcd commented 2 years ago

Hhm, I just tried that on my Windows PC and it still works.. Is that on Linux? Mac?

betterbydesign commented 2 years ago

It's on Mac machines.

On Wed, Jun 1, 2022, 3:59 AM João Dias @.***> wrote:

Hhm, I just tried that on my Windows PC and it still works.. Is that on Linux? Mac?

— Reply to this email directly, view it on GitHub https://github.com/joaomgcd/JoinChrome/issues/120#issuecomment-1143248344, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZJ5YIO7B3ZLWFGEC4QMQDVM4J7RANCNFSM5XOE736Q . You are receiving this because you authored the thread.Message ID: @.***>

medlir commented 2 years ago

I was having this issue on Windows 10 as well but I WAS getting an error in the inspect dev console for the extension about it being out of bounds.

Resolved by Chrome menu, More Tools, Extensions, Join, Details, Inspect Views, "join.html", Application tab in dev console, Local Storage, wiped out the popoutWidth and popoutHeight values, which allowed it to fall back on the default hard-coded values, and it successfully was able to open the popout immediately after nuking those.

Edit: Though resizing the window off the default size immediately breaks it again. Not sure what that's about as the error is Unchecked runtime.lastError: Invalid value for bounds. Bounds must be at least 50% within visible screen space. and values of 886x819 are certainly within my screen bounds and visible.

Edit 2: Okay, looking further, seems to be a problem with the top and left attributes of the popout window create... it's setting the left to screen.width - 230 and the top to Math.round((screen.height / 2) - (height /2)) so presumably if you resize the popout to wider than 460, more than 50% will be off-screen regardless of height? Though on switching my HDMI switch for second monitor back to my desktop from my laptop it's now working seemingly after being resized... only it's creating the resized popup on the second screen, leading me to think it's getting confused about available screen space in general.

joaomgcd commented 2 years ago

Oh, thanks for the tip. Seems like I need to try and fix that! Does that fix your issue though @Satori83 ?

betterbydesign commented 2 years ago

Interestingly enough one of Macbooks was working still, the other had the issue but following the steps listed to remove pop out values fixed it. Thanks guys!

joaomgcd commented 2 years ago

Glad it worked! :) Thanks for the input!

clubtech commented 2 years ago

@joaomgcd will you be releasing a fix for this? I'm still having this issue with version 1.8.23 of the extension.

joaomgcd commented 2 years ago

@clubtech did the solution mentioned above not work for you?

clubtech commented 2 years ago

@joaomgcd it fixed it for me but would it be possible to release a permanent fix for this via the chrome store?

joaomgcd commented 2 years ago

Yes, I need to put out a permanent fix for this. Ideally the window should always be created in a usable space...

betterbydesign commented 2 years ago

Yes it worked for me. I thought I replied back to let you know.

Thanks,

Scott Foster

Founder / Project Coordinator

Office: 215-550-1405

http://upcity.com/local-marketing-agencies/profiles/satori-design http://upcity.com/local-marketing-agencies/profiles/satori-design

This e-mail message may contain confidential or legally privileged information and is intended only for the use of the intended recipient(s). Any unauthorized disclosure, dissemination, distribution, copying or the taking of any action in reliance on the information herein is prohibited. E-mails are not secure and cannot be guaranteed to be error free as they can be intercepted, amended, or contain viruses. Anyone who communicates with us by e-mail is deemed to have accepted these risks. Satori Design is not responsible for errors or omissions in this message and denies any responsibility for any damage arising from the use of e-mail. Any opinion and other statement contained in this message and any attachment are solely those of the author and do not necessarily represent those of the company.

On Tue, Jun 14, 2022 at 8:38 AM João Dias @.***> wrote:

Closed #120 https://github.com/joaomgcd/JoinChrome/issues/120 as completed.

— Reply to this email directly, view it on GitHub https://github.com/joaomgcd/JoinChrome/issues/120#event-6803975558, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZJ5YI5REPR7CIWR7JIEMDVPB4MJANCNFSM5XOE736Q . You are receiving this because you were mentioned.Message ID: @.***>