Fix bug with legacy task list position calculation

primer · Jul 6, 2023 · ac211f0 · ac211f0
1 parent 70cc227
commit ac211f0
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/drafts/MarkdownViewer/_useListInteraction.ts b/src/drafts/MarkdownViewer/_useListInteraction.ts
@@ -4,6 +4,8 @@ import {ListItem, listItemToString, parseListItem} from '../MarkdownEditor/_useL
 
 type TaskListItem = ListItem & {taskBox: '[ ]' | '[x]'}
 
+const isCodeBlockDelimiter = (line: string) => line.trimStart().startsWith('```')
+
 const isTaskListItem = (item: ListItem | null): item is TaskListItem => typeof item?.taskBox === 'string'
 
 const toggleTaskListItem = (item: TaskListItem): TaskListItem => ({
@@ -43,11 +45,17 @@ export const useListInteraction = ({
   const onToggleItem = useCallback(
     (toggledItemIndex: number) => () => {
       const lines = markdownRef.current.split('\n')
+      let inCodeBlock = false
 
       for (let lineIndex = 0, taskIndex = 0; lineIndex < lines.length; lineIndex++) {
+        if (isCodeBlockDelimiter(lines[lineIndex])) {
+          inCodeBlock = !inCodeBlock
+          continue
+        }
+
         const parsedLine = parseListItem(lines[lineIndex])
 
-        if (!isTaskListItem(parsedLine)) continue
+        if (!isTaskListItem(parsedLine) || inCodeBlock) continue
 
         if (taskIndex === toggledItemIndex) {
           const updatedLine = listItemToString(toggleTaskListItem(parsedLine))