jPurify icon indicating copy to clipboard operation
jPurify copied to clipboard

Bug: Missing append()

Open ajinabraham opened this issue 9 years ago • 3 comments

append() breaks: https://jsfiddle.net/vno23woL/ html() works just fine: https://jsfiddle.net/vno23woL/1/

ajinabraham avatar Jan 25 '17 13:01 ajinabraham

I tracked the reason for this bug. https://github.com/cure53/jPurify/blob/master/jpurify.js#L118-L129 domManip() was a part of JQuery.fn before. Now domManip() is a outside that scope. https://github.com/jquery/jquery/blob/ee2e377494a882f043e6d8abc67ac6370ee83d9c/dist/jquery.js#L5703

ajinabraham avatar Feb 15 '17 17:02 ajinabraham

Heya, we don't really maintain the project any more - but are happy to accept PRs :)

cure53 avatar Feb 15 '17 18:02 cure53

PR is here: https://github.com/cure53/jPurify/pull/9

ajinabraham avatar Feb 16 '17 13:02 ajinabraham