responsively-app icon indicating copy to clipboard operation
responsively-app copied to clipboard

Responsively closes when navigates to twitter.com

Open jjavierdguezas opened this issue 4 years ago • 6 comments
trafficstars

🐞 bug report

✍️ Description

Responsively closes when navigates to twitter.com. (maybe it happens with other URLs)

🕵🏼‍♂️ Is this a regression?

idk

🔬 Minimal Reproduction

1- Open ResponsivelyApp 2- Go to twitter.com

🌍 Your Environment

Version: 0.16.0
Electron: 9.3.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: 8.3.110.13-electron.0
OS: Windows_NT x64 10.0.19043

🔥 Exception or Error or Screenshot

N/A

jjavierdguezas avatar May 12 '21 18:05 jjavierdguezas

Hey @jjavierdguezas :wave:,

Thank you for opening an issue. We will get back to you as soon as we can. Have you seen our Open Collective page? Please consider contributing financially to our project. This will help us involve more contributors and get to issues like yours faster.

https://opencollective.com/responsively

We offer priority support for all financial contributors. Don't forget to add priority label once you become one! :smile:

Confirmed, a possible reason for this is URL is switching between mobile.twitter.com to twitter.com. then electron exiting with ERR_ABORTED error.

durgakiran avatar May 15 '21 09:05 durgakiran

One more example is youtube.com, but the behaviour is slightly different, but the problem should be the same.

durgakiran avatar May 15 '21 09:05 durgakiran

I managed to let it not crash by commenting this.props.onAddressChange(url); in desktop-app\app\components\WebView\index.js

const urlChangeHandler = async ({url, isMainFrame = true}) => { if (!isMainFrame || url === this.props.browser.address) { return; } await new Promise(r => setTimeout(r, 200)); // this.props.onAddressChange(url); };

not sure if this can be already a fix but I hope the hint can help.

Ladvace avatar Aug 10 '22 06:08 Ladvace

It happens with https://www.facebook.com/ as reported by @HartLarsson on #727

jjavierdguezas avatar Sep 15 '22 12:09 jjavierdguezas