-
Notifications
You must be signed in to change notification settings - Fork 6
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
upstream: Add config option to turn off all snapshotting and related observers #163
Conversation
…rweb-io#1311) * Add config option to turn off all snapshotting and related observers - allows RRWEB to be used for click/movement tracking alone, e.g. for a heatmaps use case - could also be used if there was a separate process for recording the DOM (in which case a 3rd party library like https://github.com/antonmedv/finder could be added to record targets instead of the mirror) --------- Authored-by: eoghanmurray <[email protected]> Co-authored-by: Justin Halsall <[email protected]>
size-limit report 📦
|
Ref rrweb-io#1311 |
if (o.recordDOM) { | ||
styleSheetObserver = initStyleSheetObserver(o, { win: currentWindow }); | ||
adoptedStyleSheetObserver = initAdoptedStyleSheetObserver(o, o.doc); | ||
styleDeclarationObserver = initStyleDeclarationObserver(o, { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's this for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same thing as https://github.com/getsentry/rrweb/pull/163/files#diff-b7a42c375ac5ab2de196b5e95e0b2f82b723969c5836fac0ff6f7ca6ad66ebd7L1306 except this observer only runs when o.recordDOM
is true
Authored-by: eoghanmurray [email protected]
Co-authored-by: Justin Halsall [email protected]