Skip to content

Allowlist common html tags

Andrew Nichols requested to merge fr3nata/taguette:allow-more-html-tags into master

I broke this into two commits -- the first allows the table tags and includes a test, this should directly close #267 (closed) and I'll prep some screenshots demonstrating correct behavior via Calibre.

The next introduces a larger set of common html tags, draw largely from the list in #150 (closed) but notably not the generic inline and block containers, div and span. I don't think too many of the common source documents for taguette should need these generic tags so I felt it was better to err on the side of not including them.

I'm not certain how much test coverage is desirable and could easily add another test.

A simple table in a word doc: word-table

After uploading it to taguette on my branch: taguette-table

After uploading it to taguette on master: taguette-current

Edited by Andrew Nichols

Merge request reports