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

Convert HTML tables to Markdown #56

Merged
merged 2 commits into from
Jul 29, 2020
Merged

Convert HTML tables to Markdown #56

merged 2 commits into from
Jul 29, 2020

Conversation

MattiSG
Copy link
Member

@MattiSG MattiSG commented Jul 24, 2020

The following services contain at least one <table>. They have a checkbox associated with them that should be checked when the result of running the filters on them has been double-checked.

  • AskFM
  • Badoo
  • deviantART
  • MouthShut
  • StackOverflow
  • WeChat
  • Yelp

@MattiSG MattiSG requested a review from Ndpnt July 24, 2020 10:09
@MattiSG
Copy link
Member Author

MattiSG commented Jul 24, 2020

@AdrienFines could you please double-check the impact on the last two services? 🙂 I already checked MouthShut's Privacy Policy, no change.

@AdrienFines
Copy link
Contributor

I checked each service and everything seems fine. There is only one little flaw : when no column headers are found, one empty line is added at the beginning of the table (for example in StackOverflow Privacy Policy). Yet I don't think it will be that annoying.

@MattiSG
Copy link
Member Author

MattiSG commented Jul 27, 2020

when no column headers are found, one empty line is added

Yup, saw that too. It is a workaround added in mixmark-io/turndown-plugin-gfm#10.

Thanks for the review! 😃

MattiSG added 2 commits July 29, 2020 10:34
Convert HTML tables and strikethrough
Official plugin is not maintained, the joplin-turndown-plugin-gfm fork contains fixes for HTML tables conversions
See mixmark-io/turndown-plugin-gfm#3
@Ndpnt Ndpnt force-pushed the turndown-extension branch from 5852bb9 to 0c97f5b Compare July 29, 2020 08:34
@Ndpnt Ndpnt merged commit a1ec305 into master Jul 29, 2020
@Ndpnt Ndpnt deleted the turndown-extension branch July 29, 2020 08:35
martinratinaud pushed a commit that referenced this pull request Aug 6, 2021
Convert HTML tables to Markdown
martinratinaud pushed a commit that referenced this pull request Aug 6, 2021
Convert HTML tables to Markdown
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