'getComputedStyle' on 'Window', isn't "window" with lowercase? about eruda HOT 9 CLOSED

@anaibol It's on Jsdelivr now, and thanks for your advice. https://cdn.jsdelivr.net/eruda/1.0.0/eruda.min.js

from eruda.

Judging from the error message, it is not caused by misspelling, probably the dom node currently viewing is deleted. Can you use the uncompressed one to see which line exactly is causing the problem? Since there are several places using getComputedStyle api, including some third party libs. Thanks:)

from eruda.

same problem, the error line is 3010, relative code is

Draggabilly.prototype._getPosition = function() {
  var style = getComputedStyle( this.element ); // this line
  var x = this._getPositionCoord( style.left, 'width' );
  var y = this._getPositionCoord( style.top, 'height' );
  // clean up 'auto' or other non-integer values
  this.position.x = isNaN( x ) ? 0 : x;
  this.position.y = isNaN( y ) ? 0 : y;

  this._addTransformPosition( style );
};

from eruda.

The tool will append a dom node as container to body when initialized, Looks like the dom is somehow erased afterwards. Are you guys using some frameworks that will rewrite the whole body content?

I'm planning on moving it to outside of body, hopefully it's going to solve this problem:)

from eruda.

Put the script tag at the end of document should be OK, I came across this problem just because I put it in the head tag

from eruda.

@leo108 Container element is now appended to html tag, should be ok to put it in head tag now.
@anaibol Did you put the script inside head tag too?

from eruda.

I am executing the script on componentDidMount of React.
I don't have that error now, but Eruda is replacing all my DOM.
Before

After

from eruda.

Oh, I haven't used this tool in a full-react application yet. Will try to figure out a way to solve it later. Thanks for reporting the issue.

from eruda.

Cool! I was thinking, it would be interesting also to submit it later to a CDN like Jsdelivr, because you want to async and condicionally load the debugger, it should be easier to load it from an external CDN instead of making maybe a webpack entrypoint only for the debugger.

from eruda.