Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix the bug where note revisions cannot be previewed. #1392

Merged
merged 1 commit into from
Mar 11, 2025

Conversation

SiriusXT
Copy link
Member

@SiriusXT SiriusXT commented Mar 11, 2025

Related to [https://github.com//issues/1379], enabling Canvas and Mind Map previews.

The main change is replacing .html() with .prop("outerHTML").

Reason: <img> is a self-closing tag with no child elements, so .html() returns empty, which prevents the preview.

The .html() method was introduced during the conversion from .js to .ts, leading to the bug: 8f6376e.

Copy link
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.

LGTM.

@eliandoran eliandoran merged commit 971b388 into develop Mar 11, 2025
5 checks passed
@eliandoran eliandoran deleted the Note_Revisions branch March 11, 2025 11:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants