-
-
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
Do not steal focus to report a guru meditation #10135
Comments
This comment was marked as spam.
This comment was marked as spam.
Just in case, guru meditation error occurs you are supposed to scroll down a little, click on |
I was driving and not using NewPipe at all. Since I needed to get to my destination on time I did not bother to copy the error log. No matter what the content of the stack trace might be, NewPipe should not ever steal focus like this. |
Basically, if the app is not in the foreground, it should simply generate an error notification, instead of a full-screen one. I think that's the most straightforward solution, @Stypox? |
A notification could be reasonable to cover the use case where someone needs to know that NewPipe is no longer running in the background. Ideally, DND mode would suppress that notification as well: in my case, I did not care at all what was going on with NewPipe at that time, which made the focus-stealing especially heavy-handed and frustrating. |
It's not clear how this is replicated. The error will display when NewPipe is completely closed and hasn't been operating. First, fix the issue of the app stealing focus to display errors, second we can go through and determine what the issues actually are. |
Yes, although I am not sure there is a simple way to distinguish whether the app was in foreground or not AFTER it has crashed. |
Is that place the point of intervention? I was thinking that the moment the app goes into the background (which I think you can query programmatically?), you could tell the app to send all potential full screen error reports as error notifications instead. |
This comment was marked as duplicate.
This comment was marked as duplicate.
I know a way to reproduce this problem. Step 1: Step 3: Step 4: Step 5: |
The core of the issue is NewPipe popping up with a full screen crash report while the app isn't in the foreground. For me this usually happens hours or even days after I last opened the app, and often repeatedly until I go and kill NewPipe manually. Most users won't care enough or have the skills to report it properly, making the unprompted focus-stealing pop-up solely a major nuisance. |
Another annoying thing (though less annoying than the crash jumpscare) is that the button press gets stolen even if there was an eligible background media app that was used more recently. You have to open that app and resume playback manually. |
That's new. Some versions ago this used to be a workaround for another bug where headphones or Bluetooth would cause the last media app to resume, and if that happened to be Newpipe it would show an empty notification. Making sure another media app was the last one opened was the way to bypass the bug. |
I have created a temporary fork that has guru code commented out. Branches off of |
I'm experiencing this same issue on Android 14 (happened on Android 13 too) and have been for many months now. I'll try to remember to attach a log to this ticket next time. I find it difficult since it's usually when I'm driving. I wonder if it has to do with Google Maps or Waze since it never happens with OSM or OrganicMaps for me... |
I use OSMAND and it happens frequently. This repro seems to be accurate. |
What log are you talking about? adb logcat? Otherwise it doesn't matter what specific crash report Newpipe generates. The idea is to redirect them all to an error notification. |
Checklist
Affected version
0.25.1
Steps to reproduce the bug
I'm not sure unfortunately. I was watching videos last night and today I have the problem described below with the background process. I'm happy to experiment if anyone can make suggestions.
Expected behavior
When an unhandled exception occurs, NewPipe should print a guru meditation without stealing focus or switching me out of the app I'm working on.
Actual behavior
Today I'm driving my car navigating with a maps app. NewPipe repeatedly takes over the screen every few minutes to print a guru meditation. I had to pull off the road and force kill the app to get it to stop. This is dangerous and annoying.
To be clear, it's not important what the error was about. Errors happen. When they happen, NewPipe should not ever steal focus.
Screenshots/Screen recordings
No response
Logs
No response
Affected Android/Custom ROM version
grapheneOS.org
Affected device model
Pixel 6
Additional information
No response
The text was updated successfully, but these errors were encountered: