-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error while opening content in browser #3925
Comments
Apparently there is no default browser on your phone and neither there is an application able to handle that url. Do other apps open things correctly in browser? |
Imho we should catch that exception and fall back to a regular share |
@test2a @DrovGamedev With my debug apk above, when setting no default browser on your device, did you get a list of apps to open the URL (when using open in browser command), or something else? |
@TiA4f8R hi. i just checked your debug app but just like .20.4, if i click on globe icon, a new newpipe app opens and video plays. i think it might be due to url intent given to newpipe for opening all youtube apps. because i still have no default browser, it is falling back to url intent. clicking on any other link in the descrption for example opens the browser intent popup correctly. |
@test2a does it opens the NewPipe release or my debug APK? Here is my behavior on my device: Thanks for your fast reply! |
@test2a Can you go to Settings, Apps, NewPipe release v0.20.4, Open by default and check if the Remove default settings button is clickable? If yes, tap it and retry and let me know. |
@test2a If you want to test if my fix is working in an other way, you can also share a URL which NewPipe can't open to my debug APK and use Open in browser command. If a list of apps is showing, it means that my fix is working. |
OK. I checked the settings. I have set "open supported links" as "aways in this app". If I set it to don't open, then it works as expected. Dunno. Is there a way around this? If I set to always open, this would always happen unless there is some way to break the URL intent somehow |
Sounds a nice compromise. Thanks. |
Version
0.19.6
Steps to reproduce the bug
Expected behavior
Opening url in browser.
Actual behaviour
UI crashes
Logs
Exception
Crash log
The text was updated successfully, but these errors were encountered: