Optionally keep inner HTML of table cells

[jakewilliami]

Hi Serge,

Firstly, thank you for your work on this package. I use this library often, after first reading about it here.

I'm wondering if there is (or could be) an option to retain the inner HTML of table cells. Currently, <br> tags will be stripped, so foo<br>bar will become foobar.

Could potentially use a combination of UnescapeString and k3a/html2text/jaytaylor/html2text (or allow user to use this downstream).

Thanks again!

Jake

[nfx]

the idea of this library is to rely on nothing but stdlib, which golang.org/x kinda falls into. So using the standard library is a hard requirement.

cell text gets extracted by:

https://github.com/nfx/go-htmltable/blob/main/page.go#L202-L207

and

https://github.com/nfx/go-htmltable/blob/main/page.go#L372-L383

I am in favor of the community contributing a PR, as long as it adds a configurable option to Page struct and modifies all the exported NewSliceXXX and NewXXX functions

[jakewilliami]

Hey Serge, sorry for such a delayed response to this. Life got really busy!

I have implemented some configurable options (and fixed the tests), so that my requirement to keep inner HTML is met. I also added an option to specify user agent, as this has been an ongoing issue for me when using this package (more and more sites care about security nowadays. For good reason, I suppose!).

Please let me know what you think, and modify as needed, in #44