wrong regex tag matching about summernote-cleaner HOT 10 CLOSED

@DiemenDesign done; @LoloDf I wrote your username there, alright?

from summernote-cleaner.

Sorry about that @KaiqueG it was around 3AM and way past time for sleep. Thanks for the PR, I've merged it. I hope that fixes this issue.

from summernote-cleaner.

@DiemenDesign don't worry, hehe.

from summernote-cleaner.

I got the same problem here: #42 . Your solution to the RegExp is better than mine.

from summernote-cleaner.

Sorry for the delay, exactly where do you want me to test changing the regex (ie. Line number).
I did however try changing what I thought was the closest regex to what you suggest changing, and it made no difference on my end.

from summernote-cleaner.

It was at line number 56.
However you replaced it in the last commit by (line 57-60) :

 tS=new RegExp('<'+bT[i]+'[^>]*>','gi');
            out=out.replace(tS,'');
            tS=new RegExp('</'+bT[i]+'[^>]*>','gi');
            out=out.replace(tS,'');

wich does replace only the tags and not the inner content

from summernote-cleaner.

Does it now work as you think it should, it does for me here?

from summernote-cleaner.

@DiemenDesign That's the problem I solved at #42 . It's the same solution but he made a single expression when I used two, hehe.

from summernote-cleaner.

Yes it's maybe better to take my correction for the next commit as it use only one instruction instead of two (same for the replace, I do it once instead of two)

from summernote-cleaner.

@LoloDf if you like feel free to do a PR with your changes, though you may want to grab the latest source as I've edited a lot of the formatting in the last hour or so.

from summernote-cleaner.