data-cacheid aren't cleaned up before code snippets are taken
WilcoFiers opened this issue · comments
Wilco Fiers commented
I've been finding a lot of data-cacheid attributes in the code snippets returned by Quail. I think this is caused by the color tests. This may be confusing to users.
Chris Baldwin commented
Using elem.dataset
or jQuery.data()
should keep these out of the DOM.
Chris Baldwin commented
Tried using dataset but it threw TypeError: 'undefined' is not an object (evaluating 'element.dataset.cacheId')
during testing.
Pretty sure I'm missing something obvious here, but the jQuery .data() method seems to work and gets rid of these data-cacheid attributes in reports.
J. Renée Beach commented
Fixed by PR #360 .