zadam / trilium

Build your personal knowledge base with Trilium Notes
GNU Affero General Public License v3.0
27.52k stars 1.88k forks source link

(Bug report) The image in the list subitem will reset the logical hierarchy between the subitems #4508

Open flysaber opened 12 months ago

flysaber commented 12 months ago

Trilium Version

v0.62.4

What operating system are you using?

Windows

What is your setup?

Local + server sync

Operating System Version

Windows 10,Ubuntu 22.04

Description

The image in the list subitem will reset the logical hierarchy between the subitems。 1 Item 1.3 should be at the same level as Item 1.2, indented to the right. However, due to the presence of images in Item 1.2, Item 1.3 cannot be indented to the right (if Item 1.3 is selected, the right indent button is grayed out). This means that the image resets the hierarchy of sub items in the list. For ordered lists, this error is also similar, so I provide a similar graph. 2

Addendum to the question: After sending this question, I found a way to bypass this error. We cannot directly upload or paste images in sub items. However, if we first upload the image to the attachment and then copy the address of the image in the attachment to the sub item. we can temporarily bypass this error. BUT,This is just a temporary flawed handling method. The images in the sub items processed in this way cannot be centered, right aligned, or left aligned, otherwise formatting errors will occur. The most fatal thing is that if the image link in the attachment is used to display the image in the note, it cannot be displayed on the sharing page and can only be displayed as blank.( this is the independent bug , I report it in #4509 )

Error logs

There are no error logs, this should be a bug

zadam commented 11 months ago

This is because CKEditor by default does not support block elements in lists. But this is going to change soon: ckeditor/ckeditor5#14767

flysaber commented 11 months ago

I'm glad to see your reply. I will wait for the editor's update. Also, please reply to bug report #4509

BoKKeR commented 1 month ago

I am assuming this is solved?