testomatio-editor-blocks 0.4.49 → 0.4.51

package/src/editor/blocks/stepFieldFormatting.test.ts

@@ -0,0 +1,44 @@
+import { describe, expect, it } from "vitest";
+import { buildFullMarkdown, type FormattingMeta, type LinkMeta } from "./stepField";
+
+describe("buildFullMarkdown formatting combinations", () => {
+  const noLinks: LinkMeta[] = [];
+
+  it("applies bold and italic to the same range", () => {
+    const formatting: FormattingMeta[] = [
+      { start: 0, end: 5, type: "bold" },
+      { start: 0, end: 5, type: "italic" },
+    ];
+    expect(buildFullMarkdown("hello", noLinks, formatting)).toBe("_**hello**_");
+  });
+
+  it("preserves word-level bold when sentence-level bold is applied", () => {
+    // User bolds "adipiscing", then bolds entire sentence.
+    // The word-level bold should be merged into sentence-level bold,
+    // not create nested **...**adipiscing**...** markers.
+    const text = "Lorem ipsum dolor sit amet, consectetur adipiscing elit";
+    const formatting: FormattingMeta[] = [
+      { start: 0, end: text.length, type: "bold" },
+    ];
+    expect(buildFullMarkdown(text, noLinks, formatting)).toBe(
+      `**${text}**`,
+    );
+  });
+
+  it("bold on word + italic on sentence produces correct markdown", () => {
+    const text = "hello world";
+    const formatting: FormattingMeta[] = [
+      { start: 0, end: 5, type: "bold" },
+      { start: 0, end: 11, type: "italic" },
+    ];
+    expect(buildFullMarkdown(text, noLinks, formatting)).toBe("_**hello** world_");
+  });
+
+  it("code formatting removes bold and italic", () => {
+    // When code is applied, bold/italic should not appear
+    const formatting: FormattingMeta[] = [
+      { start: 0, end: 5, type: "code" },
+    ];
+    expect(buildFullMarkdown("hello", noLinks, formatting)).toBe("`hello`");
+  });
+});

package/src/editor/createMarkdownPasteHandler.ts

@@ -10,6 +10,18 @@ type PasteHandlerContext = {
   }) => boolean | undefined;
 };
 
+const BLOCK_MARKDOWN_PREFIX = /^(\s*)(#{1,6}\s|[-*+]\s|\d+[.)]\s|>\s|```|~~~|\||!\[)/;
+
+function isInlineOnlyPaste(plainText: string, parsedBlocks: CustomPartialBlock[]): boolean {
+  if (parsedBlocks.length !== 1) return false;
+  const [block] = parsedBlocks;
+  if (block.type !== "paragraph") return false;
+  if (block.children && block.children.length > 0) return false;
+  if (/\r?\n/.test(plainText)) return false;
+  if (BLOCK_MARKDOWN_PREFIX.test(plainText)) return false;
+  return true;
+}
+
 export function createMarkdownPasteHandler(
   converter: (markdown: string) => CustomPartialBlock[],
 ) {
@@ -34,6 +46,10 @@ export function createMarkdownPasteHandler(
       const parsedBlocks = converter(plainText);
       if (parsedBlocks.length === 0) return defaultPasteHandler();
 
+      if (isInlineOnlyPaste(plainText, parsedBlocks)) {
+        return defaultPasteHandler({ plainTextAsMarkdown: false });
+      }
+
       const selection = editor.getSelection();
       const selectedIds = selection?.blocks
         ?.map((block: any) => block.id)

package/src/editor/customMarkdownConverter.test.ts

@@ -43,7 +43,7 @@ describe("blocksToMarkdown", () => {
       },
     ];
 
-    expect(blocksToMarkdown(blocks)).toBe("Hello **world***!*");
+    expect(blocksToMarkdown(blocks)).toBe("Hello **world**_!_");
   });
 
   it("places bold markers outside leading/trailing spaces", () => {
@@ -76,7 +76,7 @@ describe("blocksToMarkdown", () => {
         children: [],
       },
     ];
-    expect(blocksToMarkdown(blocks)).toBe("*word* next");
+    expect(blocksToMarkdown(blocks)).toBe("_word_ next");
   });
 
   it("places code backticks outside leading/trailing spaces", () => {
@@ -301,7 +301,7 @@ describe("blocksToMarkdown", () => {
     ];
 
     const markdown = blocksToMarkdown(blocks);
-    expect(markdown).toBe("***text***");
+    expect(markdown).toBe("_**text**_");
   });
 
    it("keeps inline formatting inside step fields", () => {

package/src/editor/customMarkdownConverter.ts

@@ -152,7 +152,7 @@ function applyTextStyles(text: string, styles: EditorStyles | undefined): string
   }
 
   if (styles.italic) {
-    wrappers.push({ prefix: "*", suffix: "*" });
+    wrappers.push({ prefix: "_", suffix: "_" });
   }
 
   if (styles.strike) {
@@ -620,10 +620,7 @@ function serializeBlocks(blocks: CustomEditorBlock[], ctx: MarkdownContext): str
 
 export function blocksToMarkdown(blocks: CustomEditorBlock[]): string {
   const lines = serializeBlocks(blocks, { listDepth: 0, insideQuote: false });
-  const cleaned = lines
-    .join("\n")
-    .replace(/\n{3,}/g, "\n\n")
-    .trimEnd();
+  const cleaned = lines.join("\n").trimEnd();
 
   return cleaned;
 }
@@ -709,8 +706,9 @@ function parseInlineMarkdown(text: string): EditorInline[] {
       }
     }
 
-    if (cleaned.startsWith("*", i)) {
-      const end = cleaned.indexOf("*", i + 1);
+    if (cleaned[i] === "*" || cleaned[i] === "_") {
+      const marker = cleaned[i];
+      const end = cleaned.indexOf(marker, i + 1);
       if (end !== -1) {
         pushPlain();
         const inner = cleaned.slice(i + 1, end);

package/src/editor/styles.css

@@ -947,6 +947,10 @@ html.dark .bn-step-image-preview__content {
   color: var(--step-input-border-focus);
 }
 
+.bn-container {
+  overflow-x: clip;
+}
+
 [data-tooltip] {
   position: relative;
 }
@@ -1079,7 +1083,8 @@ html.dark .bn-step-image-preview__content {
 }
 
 .bn-step-editor .overtype-wrapper .overtype-preview strong.step-preview-bold {
-  font-weight: 600 !important;
+  -webkit-text-stroke: 0.5px currentColor;
+  font-weight: inherit !important;
   color: inherit !important;
 }
 
@@ -1090,8 +1095,8 @@ html.dark .bn-step-image-preview__content {
 
 .bn-step-editor .overtype-wrapper .overtype-preview code.step-preview-code {
   background-color: transparent !important;
-  border-radius: 3px !important;
-  font-family: "Fira Mono", Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace !important;
+  font-family: inherit !important;
+  font-size: inherit !important;
   color: rgb(146, 64, 14) !important;
 }
 
@@ -1356,3 +1361,11 @@ html.dark .bn-step-image-preview__content {
 .bn-testcase--draft {
   --status-color: var(--status-default);
 }
+
+/* Prevent unnecessary horizontal scrollbar on BlockNote tables.
+   ProseMirror sets overflow-x: auto on .tableWrapper, but the
+   table-widgets-container extends slightly past the wrapper edge,
+   triggering a scrollbar even when table content fits. */
+.bn-editor [data-content-type="table"] .tableWrapper {
+  overflow-x: hidden;
+}