ProjectVisBug icon indicating copy to clipboard operation
ProjectVisBug copied to clipboard

Idea: Attach VisBug to ShadowRoot

Open jogibear9988 opened this issue 1 year ago • 5 comments

Idea:

As adding visbug to a page could lead to side effects, wouldn't it be preferable to add it to it's own shadowroot?

Like:

      let sr = document.body.attachShadow({mode: 'open'});
      sr.innerHTML = "<slot></slot>"

And after this the VisBug overlays could be added to the ShadowRoot, without affecting the real Dom?

jogibear9988 avatar Aug 27 '24 09:08 jogibear9988

i'm willing to try out a PR and test it 👍🏻 but visbug does apply attributes to nodes and does some other things that are still "side effects", but reducing them is indeed generally nice

argyleink avatar Aug 27 '24 15:08 argyleink

I'll look to create a demo

jogibear9988 avatar Aug 27 '24 17:08 jogibear9988

i'm willing to try out a PR and test it 👍🏻 but visbug does apply attributes to nodes and does some other things that are still "side effects", but reducing them is indeed generally nice

I saw that it add's attributes for selection. Is there a reason for this? Why is the selection not stored in some varaibles?

jogibear9988 avatar Aug 27 '24 18:08 jogibear9988

i'm willing to try out a PR and test it 👍🏻 but visbug does apply attributes to nodes and does some other things that are still "side effects", but reducing them is indeed generally nice

I saw that it add's attributes for selection. Is there a reason for this? Why is the selection not stored in some varaibles?

it's never been an issue, and was using DOM state as the source of truth

argyleink avatar Aug 27 '24 19:08 argyleink

Yeah, maybe it's my thinking cause I'm working on a designer, wich exports the html and so my output should not contain own added attributes.

jogibear9988 avatar Aug 27 '24 21:08 jogibear9988