Header elements are not converted to Header widgets

[maks]

The PDF widget library provides a Header widget but currently HTML header (H1, H2 etc) elements are converted into TextSpans inside RichText's. I can see how that allows for more flexibility but it also means that very useful functionality of the PDF widget library TableOfContent widget cannot be used to compile a table of contents page.

@alihassan143 I'm happy to contribute a PR for this but I'm not sure if just replacing the existing mapping of Header elements to a Header widget would be ok for you as you have code that supports using AnnotationUrl to I guess allow for headers in the PDF to be url links and I'm not sure that can be preserved by using Header widget unless a child TextSpan widget was added to the Header widget perhaps?

[alihassan143]

good point
currently i added the rich text so different attributes can be parsed correctly
i think replacing with header class is good option but need to test with different scenarios is it work well width current implementation

[alihassan143]

i think header cannot have multiple child elements like paragraphs and other elements but need to do some research on that for
proper validation

[maks]

@alihassan143 Thanks for considering this! 👍🏻
I also noticed that the current code doesn't seem to handle inline css for heading elements, is that intentional or just a bug or am I just doing something wrong?

[alihassan143]

i think i missed that need to check that

[maks]

@alihassan143 to give some more context for this, I am actually using your fantastic package to add PDF output to my little static site generator picosite, so this is for Dart cli, not Flutter.

And what I am using it primarily for at the moment is the user manual for the picoTracker project, which exists in web form already but now I am also wanting to generate PDF output from the same markdown source.

For now, to allow me to make progress, I've done what amounts to a quick hack to add support headers and used that fork in a unreleased version of picosite, but I would love to be able to get something with this functionality that you would be happy to include in your package.