-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
add ignore element option #1
Comments
We may have two approaches to customize ignore element. One is to ignore certain class name/data attribute, such as Another one is to add an ignore class name option like The first one needs users to update their source code when the second one may cause extra complexity and potential performance issue. Personally, I tend to implement the first one because most apps do not already have some markups for privacy usage. And if we provided the second solution, users may define a class name like Any feedback is welcomed. |
|
merge rrweb master into our fork
…ument perf: only call createHTMLDocument when it is needed
If some elements were in the ignore list, stop observe and record its value.
We also need a default to ignore list, including some elements like password input and so on.
The text was updated successfully, but these errors were encountered: