Skip to content
This repository was archived by the owner on Jun 24, 2025. It is now read-only.

fix(import): Unable to handle multi line mathematical formulas when i…#1984

Merged
eliandoran merged 3 commits intodevelopfrom
markdown-math
May 21, 2025
Merged

fix(import): Unable to handle multi line mathematical formulas when i…#1984
eliandoran merged 3 commits intodevelopfrom
markdown-math

Conversation

@SiriusXT
Copy link
Copy Markdown
Member

fix(import): Unable to handle multi line mathematical formulas when importing markdown

Previously, handling math formulas within paragraphs had the following limitations:

  1. Multiline math formulas could not be imported.
  2. Underscores in formulas might be incorrectly converted into <em> tags. For example, when importing This is a formula: $\\mathcal{L}_{task} + \\mathcal{L}_{od}$ inside a sentence.,
  3. Math formulas inside list items could not be converted properly.

To address these issues, formulas are now first replaced with placeholders before converting Markdown to HTML. After the conversion, the placeholders are replaced back with the actual formulas.

@SiriusXT SiriusXT marked this pull request as ready for review May 20, 2025 14:12
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 20, 2025

Folder/File Previous size New size Difference
/upload/TriliumNextNotes-Server-1984-merge-linux-x64.tar.xz 67.96MB +67.96MB (+100.00%)
TOTAL +67.96MB

@SiriusXT SiriusXT marked this pull request as draft May 21, 2025 07:19
@SiriusXT SiriusXT marked this pull request as ready for review May 21, 2025 09:26
@SiriusXT SiriusXT marked this pull request as draft May 21, 2025 10:36
@SiriusXT SiriusXT marked this pull request as ready for review May 21, 2025 12:26
@eliandoran eliandoran added this to the v0.94.0 milestone May 21, 2025
@eliandoran eliandoran merged commit 1001841 into develop May 21, 2025
11 checks passed
@eliandoran eliandoran deleted the markdown-math branch May 21, 2025 20:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants