Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tables should be passed through instead of being stripped out #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fmarier
Copy link
Contributor

@fmarier fmarier commented Nov 5, 2012

In addition, links inside tables need to be HTML otherwise they
will not be recognized as links.

In addition, links inside tables need to be HTML otherwise they
will not be recognized as links.
@aaronsw
Copy link
Owner

aaronsw commented Nov 5, 2012

Can we make this an option, default off? I don't think most users expect to get big chunks of HTML in their output.

@fmarier
Copy link
Contributor Author

fmarier commented Nov 5, 2012

Sure, if you want to turn that into an option, that's not a bad idea. I think you're right that people don't expect lots of HTML, but at the same time, stripping out the tables and just outputting broken formatting is not ideal either.

So, personally I'm not sure which one should be the default, but in my use case, I certainly didn't want to lose my tables after converting from HTML to Markdown.

@mcepl
Copy link

mcepl commented Nov 6, 2013

Wouldn't it be more useful to convert to Markdown tables? Issue #50 seems to be much more interesting than this.

pombredanne pushed a commit to pombredanne/html2text that referenced this pull request Oct 10, 2015
<a> tags with no text or <img> tags no longer discarded.

Thanks @mgontav
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants