-
Notifications
You must be signed in to change notification settings - Fork 286
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
App Switcher breaks inspector #1039
Comments
I think this may be fixed in https://github.com/emberjs/ember-inspector/pull/1103/files (specifically this), @nummi any chance you can re-test? (no rush) |
@chancancode, it's getting better: no empty white screen. Unfortunately Inspector doesn't switch apps and now I'm getting this error:
Here's the test app I've been using: https://github.com/nummi/inspector-test It embeds the deprecation app in an iframe. |
Thanks! I’ll take a look! |
@chancancode is this still an issue? |
Yes |
I haven’t had a chance to look into how it’s supposed to work with iframes yet, but it doesn’t work properly. It may be the case that switching between two apps on the same page and on the same frame works but switching between iframes doesn’t. |
@chancancode was changing the app picker a requirement of the new render node stuff? If not, maybe we can change it back? |
It was already broken before my change. This issue predated my changes. My change made it crash less badly, but it still doesn’t work. |
@chancancode are you sure? It worked the last time @nummi and I tried it, but we don't try it often. |
The issue is from September and the description mentioned:
But yes I also confirmed it locally. I can try to figure this out but probably have a few other things to look into before getting to this. I also find that the inspector's detection of apps on the current page could be pretty spotty and timing dependent, so probably going to try and fix that as well. |
@chancancode sorry, you are correct, it's been a long couple of weeks 😅 |
No problem! Same 😄 |
should be fixed by #2157 |
Error in event handler: Error: Assertion Failed: You attempted to remove a function listener which did not exist on the instance, which means you may have attempted to remove it before it was added.
The entire Inspector turns into a white screen.
Seems to be coming from this line: https://github.com/emberjs/ember-inspector/blob/master/app/routes/app-detected.js#L52
I believe updating to 3.11 is the culprit. This PR causes the error:
#1000
The text was updated successfully, but these errors were encountered: