-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
To support Shadow DOM #31
Comments
Is anyone familiar with Shadow DOM and what we need to do to support it? I imagine the issues are being caused by the Or if someone could provide me a reproduction of it failing I could try to debug further for |
Here is a reproduction: When i drag the PaneResizer, the method Ideas how this could be fixed:
|
I tried to fix this issue but there are still problems. There is a test project in packages/shadow-dom-test. I added a prop called getRood to PaneGroup. getResizeHandleElement works. There is an effect in paneforge.svelte.ts in line 610. Outside a shadow dom, this effect runs when I start to drag the resizer. The effect does not run when paneforge runs inside a shadow dom. |
Before the changes are merged into the npm i https://pkg.pr.new/svecosystem/paneforge/paneforge@1d8c1528a2107a74fab6925f3b567fe85835e659 You can view the documentation for setting the root node here: https://ed0479da.paneforge.pages.dev/docs/utils/config |
This does not work. Reproduction with PaneForgeConfig (with and without Shadow Dom): https://stackblitz.com/edit/vitejs-vite-qm3fnazy?file=src%2FApp.svelte I also tried this project on my machine (outside of Stackblitz). As far as i understand, in the example on https://ed0479da.paneforge.pages.dev/docs/utils/config Do we really need PaneForgeConfig to support shadow dom? The Component PaneGroup uses a div. Every element that belongs to the PaneGroup is a descendant of this div. Because of this, PaneForge could call Most of the apps I program are embedded into a website. Therefore, multiple apps that use Paneforge may be embedded in the same HTML document. Because of this it could be that a certain ID appears several times in the HTML document. In order to avoid such problems with ids, I would actually prefer if paneforge (and all other libraries that I use) would work completely without ids. |
Describe the feature in detail (code, mocks, or screenshots encouraged)
The component is failed to be used in a Shadow DOM.
What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response
The text was updated successfully, but these errors were encountered: