[PR #5257] [CLOSED] Export: convert tables from HTML to Markdown #6457

Closed
opened 2026-02-05 10:32:49 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/5257
Author: @WilliamDEdwards
Created: 10/11/2024
Status: Closed

Base: developmentHead: feature/convert-tables-md


📝 Commits (1)

  • 8bc4c13 Export: convert tables from HTML to Markdown

📊 Changes

1 file changed (+3 additions, -0 deletions)

View changed files

📝 app/Entities/Tools/Markdown/HtmlToMarkdown.php (+3 -0)

📄 Description

When exporting pages to Markdown, tables were still HTML. This is because league/html-to-markdown does not convert tables to Markdown by default, "because it is not part of the original Markdown syntax" (source: https://packagist.org/packages/league/html-to-markdown)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/5257 **Author:** [@WilliamDEdwards](https://github.com/WilliamDEdwards) **Created:** 10/11/2024 **Status:** ❌ Closed **Base:** `development` ← **Head:** `feature/convert-tables-md` --- ### 📝 Commits (1) - [`8bc4c13`](https://github.com/BookStackApp/BookStack/commit/8bc4c130cc4aa54899c8a84eb9ed1a668e13770f) Export: convert tables from HTML to Markdown ### 📊 Changes **1 file changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `app/Entities/Tools/Markdown/HtmlToMarkdown.php` (+3 -0) </details> ### 📄 Description When exporting pages to Markdown, tables were still HTML. This is because `league/html-to-markdown` does not convert tables to Markdown by default, "because it is not part of the original Markdown syntax" (source: https://packagist.org/packages/league/html-to-markdown) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:32:49 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6457