Tieson Trowbridge

Results 203 comments of Tieson Trowbridge

You'd probably want to follow the pattern used by a lot of .NET Core utilities - rather than pass in multiple parameters, pass in a configuration/options object (maybe even allow...

Would it make any sense to also provide a constructor which accepts a `Func`? You might use it like so: ```c# var sanitizer = new HtmlSanitizer(() => { var options...

@vanillajonathan I think I'm a little confused - what are you referring to, with respect to enums?

@vanillajonathan Ah, the enum is for the options class. That makes sense. I was trying to figure out where the HtmlSanitizer class would use an enum. I'd think your enums,...

@panetta-net-au Well, the HTML parser is [AngleSharp](https://anglesharp.github.io/), which in theory is supposed to give you the same "corrected" markup as most browsers (meaning it will try to correct fragments if...

Just an FYI, this has already been done in a project that uses CsQuery: https://github.com/mganss/HtmlSanitizer - perhaps it would be worth creating a combined project?

I use HtmlSanitizer for cleaning inputs where I have to allow HTML, and I've never really had problems with it. It has some nice extensibility points that allow you customize...

Seems like that should work if you added newlines. Is there a reason you have everything as a single line?

There _is_ a package, but it's not published on NuGet, since it's still considered experimental. @kristofferahl did mention on Twitter a while ago that there might be an update soon(ish),...