Modify renderCompactMarkdown() so that it causes no flickering/scrolling in Live Preview (#2131)

* Revert 7a2b6a1 but only for src/ui/render.ts

* Fix the incorrect cursor overlapping check

* Squashed commit of the following:

commit f4e0bdd
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 06:37:10 2023 +0900

    Fix typo

commit 43f3be4
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 06:29:01 2023 +0900

    Fix formatting

commit 74e45e4
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 06:28:32 2023 +0900

    Fix the starting position of an inline field could not obtained correctly after a document change

commit 7570941
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 06:07:03 2023 +0900

    Fix formatting

commit ba7a0a5
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 06:04:27 2023 +0900

    Fix the previous commit works only for the first line

commit 08bcd32
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 05:54:44 2023 +0900

    Resolve #2101: add a click event handler to inline field widgets in live preview

* Squashed commit of the following:

commit 21591ff
Author: RyotaUshio <[email protected]>
Date:   Sat Oct 14 07:41:40 2023 +0900

    Fix #2107: Prevent inline fields inside a code block from being rendered in live preview

* Squashed commit of the following:

commit f787431
Author: RyotaUshio <[email protected]>
Date:   Sun Oct 29 00:41:52 2023 +0900

    fix formatting

commit 7205a70
Author: RyotaUshio <[email protected]>
Date:   Sun Oct 29 00:28:24 2023 +0900

    separate the inline fields rendering option into one for reading view & one for live preview

* fix the auto-scrolling/flickering issue of inline fields (#2128) and dv.paragraph not working for lists (#2099)

* fix formatting

---------

Co-authored-by: RyotaUshio <[email protected]>

Author: RyotaUshio

src/ui/render.ts

@@ -18,6 +18,33 @@ export async function renderCompactMarkdown(
     let paragraph = tmpContainer.querySelector(":scope > p");
     if (tmpContainer.childNodes.length == 1 && paragraph) {
         container.replaceChildren(...paragraph.childNodes);
+    } else {
+        /**
+         * In most cases, the condition above will be true.
+         * However, it is not always true, for example:
+         * ```dataviewjs
+         * dv.paragraph(`
+         * - list item 1
+         * - list item 2
+         *
+         * 1. list item 3
+         * 2. list item 4
+         * `)
+         * ```
+         * MarkdownRenderer.renderMarkdown will render it as:
+         * <span>
+         *   <ul>
+         *     <li>list item 1</li>
+         *     <li>list item 2</li>
+         *   </ul>
+         *   <ol>
+         *     <li>list item 3</li>
+         *     <li>list item 4</li>
+         *   </ol>
+         * </span>
+         * Notice that there is no <p> tag.
+         */
+        container.replaceChildren(...tmpContainer.childNodes);
     }
 
     tmpContainer.remove();