-
Notifications
You must be signed in to change notification settings - Fork 61
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
Can't find DOM in engine #84
Comments
@jwlawrence I'll leave this open. I think this will ultimately be addressed through the use of |
@lukemelia can you link more information about that |
@jwlawrence @lukemelia I'm seeing this issue as well when using ember-power-select, which uses ember-wormhole. @marcemira do you have a link to the issues you found regarding |
@jbailey4 if you're on ^ Ember 2.10, you can use |
@marcemira Thanks! |
This affects ember-power-select |
@jbailey4 given you are using power-select, what did the fix for this entail? About to dig in and the suggestion didn't make perfect sense to me. |
So, as mentioned above, it is erroring because the For giggles, I tossed that in as a dependency for the engine and it worked. engines: {
foo: {
dependencies: {
services: [
'-document',
]
}
}
} I'm inexperienced with some of the internals of all this, both with that particular service and with engines, so I feel like there may be some pitfalls to doing it this way? |
Heh, I'm surprised that works. Not sure what the potential pitfalls of that approach are, but it seems dirty :) I did come across this fork that leverages the Haven't tried it yet, but it looks promising. Based off the diff it looks like there might be some issues with the tests though. @mitchlloyd do you plan to open a PR for that fork at some point? |
@jwlawrence I'd be happy to open a PR based on that branch, but all of this is dependent on changes in core. The 2 failings tests are expecting the DOM nodes to be reused when moving wormhole output from 1 location to another. This will work for most use cases, but if you were editing a form and then moved the wormhole output to another location you will loose focus on the current field. This is the issue that Chris set out to solve here: emberjs/ember.js#14472. |
I'm seeing the following error with a component using ember-wormhole after upgrading to 2.11 from 2.9. It's coming from "glimmer-runtime/lib/bounds" and not from ember-wormhole directly, but perhaps they're related:
Is there any update on |
Oof. Ran straight into this one like a brick wall. :P Another |
FWIW, updating to Ember 2.11 and ember-engines 0.4 has resolved this issue for me. |
I'm using |
The same for me, with Ember 2.12.1 and ember-engines 0.5.0. The workaround worked here as well. |
I've upgraded to 3.15.1 recently and have noticed this error occurring. Does anyone know of an impact here with Glimmer components? |
I'm getting an error when using ember-wormhole inside an ember-engine:
Uncaught Error: ember-wormhole could not get DOM
Looks like the
getDOM
util doesn't find 'service:-document' and the container lookup of 'renderer:-dom' results in an 'InteractiveRenderer' object that doesn't have a '_dom' property so it throws an error.I'm not sure if this issue can be solved on the ember-wormhole side or if we need ember-engines to provide a reference to the DOM, but figured I'd leave this here for now.
The text was updated successfully, but these errors were encountered: