testomatio-editor-blocks 0.4.48 → 0.4.50

package/package/editor/customMarkdownConverter.js

@@ -776,6 +776,7 @@ function parseTestStep(lines, index, allowEmpty = false, snippetId) {
     let expectedResult = "";
     let next = index + 1;
     let inExpectedResult = false;
+    let blankLineSeenOutsideCodeBlock = false;
     const stepIndent = current.length - current.trimStart().length;
     while (next < lines.length) {
         const line = lines[next];
@@ -789,6 +790,7 @@ function parseTestStep(lines, index, allowEmpty = false, snippetId) {
                 }
                 else {
                     stepDataLines.push("");
+                    blankLineSeenOutsideCodeBlock = true;
                 }
             }
             next += 1;
@@ -894,6 +896,10 @@ function parseTestStep(lines, index, allowEmpty = false, snippetId) {
             next += 1;
             continue;
         }
+        // After a blank line outside a code block, stop adding to step data
+        if (blankLineSeenOutsideCodeBlock) {
+            break;
+        }
         if (STEP_DATA_LINE_REGEX.test(rawTrimmed)) {
             const content = unescapeMarkdown(rawTrimmed);
             stepDataLines.push(content);

package/package/styles.css

@@ -1079,7 +1079,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 +1091,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 +1357,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;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testomatio-editor-blocks",
-  "version": "0.4.48",
+  "version": "0.4.50",
   "description": "Custom BlockNote schema, markdown conversion helpers, and UI for Testomatio-style test cases and steps.",
   "type": "module",
   "main": "./package/index.js",

package/src/editor/blocks/step.tsx

@@ -51,10 +51,10 @@ const writeStepViewMode = (mode: StepViewMode) => {
 /**
  * Returns true when a normalised (lowercased, trailing-punctuation-stripped)
  * heading text looks like a "Steps" heading.
- * Accepted forms: steps, step, step(s).
+ * Accepted forms: steps, step, step(s), test steps, test step, test step(s).
  */
 export function isStepsHeading(text: string): boolean {
-  return /^step(s|\(s\))?$/.test(text);
+  return /^(test\s+)?step(s|\(s\))?$/.test(text);
 }
 
 export const isEmptyParagraph = (b: any): boolean =>
@@ -337,12 +337,6 @@ export const stepBlock = createReactBlockSpec(
         [block.id, combinedStepValue, editor],
       );
 
-      useEffect(() => {
-        if (dataHasContent && !isDataVisible) {
-          setIsDataVisible(true);
-        }
-      }, [dataHasContent, isDataVisible]);
-
       const handleStepTitleChange = useCallback(
         (next: string) => {
           if (next === stepTitle) {
@@ -380,7 +374,8 @@ export const stepBlock = createReactBlockSpec(
 
       const handleHideData = useCallback(() => {
         setIsDataVisible(false);
-      }, []);
+        editor.updateBlock(block.id, { props: { stepData: "" } });
+      }, [editor, block.id]);
 
       const handleExpectedChange = useCallback(
         (next: string) => {
@@ -457,16 +452,8 @@ export const stepBlock = createReactBlockSpec(
       const handleHideExpected = useCallback(() => {
         setIsExpectedVisible(false);
         writeExpectedCollapsedPreference(true);
-      }, []);
-
-      useEffect(() => {
-        if (expectedHasContent && !isExpectedVisible) {
-          setIsExpectedVisible(true);
-        }
-      }, [expectedHasContent, isExpectedVisible]);
-
-      const canToggleData = !dataHasContent;
-      const canToggleExpected = !expectedHasContent;
+        editor.updateBlock(block.id, { props: { expectedResult: "" } });
+      }, [editor, block.id]);
 
       const viewToggleButton = (
         <button
@@ -523,6 +510,7 @@ export const stepBlock = createReactBlockSpec(
               placeholder={STEP_TITLE_PLACEHOLDER}
               onChange={handleStepTitleChange}
               autoFocus={stepTitle.length === 0}
+              multiline
               enableAutocomplete
               fieldName="title"
               suggestionFilter={(suggestion) => (suggestion as StepSuggestion).isSnippet !== true}
@@ -556,17 +544,15 @@ export const stepBlock = createReactBlockSpec(
                 label="Step data"
                 placeholder={STEP_DATA_PLACEHOLDER}
                 labelAction={
-                  canToggleData ? (
-                    <button
-                      type="button"
-                      className="bn-step-field__dismiss"
-                      data-tooltip="Hide step data"
-                      onClick={handleHideData}
-                      aria-label="Hide step data"
-                    >
-                      ×
-                    </button>
-                  ) : undefined
+                  <button
+                    type="button"
+                    className="bn-step-field__dismiss"
+                    data-tooltip="Hide step data"
+                    onClick={handleHideData}
+                    aria-label="Hide step data"
+                  >
+                    ×
+                  </button>
                 }
                 value={stepData}
                 onChange={handleStepDataChange}
@@ -585,18 +571,16 @@ export const stepBlock = createReactBlockSpec(
                 label="Expected result"
                 placeholder={EXPECTED_RESULT_PLACEHOLDER}
                 labelAction={
-                  canToggleExpected ? (
-                    <button
-                      type="button"
-                      className="bn-step-field__dismiss"
-                      data-tooltip="Hide expected result"
-                      onClick={handleHideExpected}
-                      tabIndex={-1}
-                      aria-label="Hide expected result"
-                    >
-                      ×
-                    </button>
-                  ) : undefined
+                  <button
+                    type="button"
+                    className="bn-step-field__dismiss"
+                    data-tooltip="Hide expected result"
+                    onClick={handleHideExpected}
+                    tabIndex={-1}
+                    aria-label="Hide expected result"
+                  >
+                    ×
+                  </button>
                 }
                 value={expectedResult}
                 onChange={handleExpectedChange}