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

feat(math): support multi-line formula editing#2003

Merged
eliandoran merged 5 commits intodevelopfrom
math-edit
May 26, 2025
Merged

feat(math): support multi-line formula editing#2003
eliandoran merged 5 commits intodevelopfrom
math-edit

Conversation

@SiriusXT
Copy link
Copy Markdown
Member

Related to :
TriliumNext/Trilium#2088
https://github.com/orgs/TriliumNext/discussions/1731
This issue has also existed in ckeditor5-math for years without being resolved, so it is directly fixed in this repository.

Before:
图片

After:
图片

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 23, 2025

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

Copy link
Copy Markdown
Contributor

@eliandoran eliandoran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First, I'm really glad to see the monorepo bear fruit, with changes to the CKEditor plugins.

It looks pretty good but there's two topics to address:

  1. I think we need to figure out a way to actually store the newlines entered as part of the equation since they are lost if you refresh the page or go to another note. This would especially by relevant for big equations where whitespace might play a role such as in matrices.

  2. I would personally make the textbox slightly wider by default, like this:
    image

@eliandoran eliandoran added this to the v0.94.0 milestone May 25, 2025
@eliandoran eliandoran marked this pull request as draft May 25, 2025 18:12
@SiriusXT SiriusXT marked this pull request as ready for review May 26, 2025 11:03
@SiriusXT
Copy link
Copy Markdown
Member Author

Really appreciate the move to a monorepo — it’s made editing these plugins so much more convenient!
I’ve implemented changes based on both of your suggestions:

Newlines inside equations are now preserved using a placeholder approach.

The default width of the equation editor has been slightly increased for better usability.

@eliandoran eliandoran merged commit 6a29fae into develop May 26, 2025
9 checks passed
@eliandoran eliandoran deleted the math-edit branch May 26, 2025 13:43
@eliandoran
Copy link
Copy Markdown
Contributor

Great job. :)

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