Table formatting not useful about html2text HOT 7 CLOSED

@matcornic I'm working on a PR for this, may have something later today or if I don't get to it before Friday I'll have some time this weekend.

Once I open the PR I'll select you both as reviewers so we can ensure it's good for everyone :)

Thanks!

cc @MarcGrol

from html2text.

Oh no! I'm sorry to hear that we've inadvertently broken you. Appreciate you reaching out in such a timely manner!

I'm confident we can find a way to accommodate all our collective needs in this case.

My first impulse is to add some config options so that we can control the rendering behavior. Perhaps it'd look like an optional extra parameter with a flag to disalbe ASCII table rendering.

Does this sound like a reasonable compromise / approach? Any other ideas on how we can best resolve this issue? I'm all ears! :D

from html2text.

Thanks for your response. Your suggested approach would allow me to keep using the latest and greatest version of html2text.

from html2text.

Hey @MarcGrol,
I'm the one responsible of breaking your usage :(. I agree with @jaytaylor solution.

In my opinion, I think the default behavior should be the old display. And an option like "PrettyTable" could be enabled.

Moreover, this kind of options could be used for example to replace the character used for converting the lists "* vs -"

@MarcGrol do you plan to make the PR or should I do it ?

from html2text.

@matcornic @MarcGrol GH currently not letting me request reviews from you, so if you're able please try / check it out and let me know what you think!

Cheers,
Jay

from html2text.

@jaytaylor looks fine to me, i commented the PR :)

from html2text.

Sorry for the delay.
@jaytaylor @matcornic: Reviewed and tested this specific version against my software and everything works fine again.
I am glad I can keep using the latest and greatest version of the html2text package.
Thanks a lot for creating the PR.

from html2text.