From a24fdac9f75765aa555ef0ebe1535799daafdbf2 Mon Sep 17 00:00:00 2001
From: Chrissi2812 <cf@pixel-kraft.de>
Date: Fri, 17 May 2019 15:50:59 +0200
Subject: [PATCH] toggle between unordered_list and ordered_list

---
 .../src/commands/toggleList.js                | 66 +++++++------------
 1 file changed, 24 insertions(+), 42 deletions(-)

diff --git a/packages/tiptap-commands/src/commands/toggleList.js b/packages/tiptap-commands/src/commands/toggleList.js
index 7fd1703e5..ad7fde5c7 100644
--- a/packages/tiptap-commands/src/commands/toggleList.js
+++ b/packages/tiptap-commands/src/commands/toggleList.js
@@ -1,49 +1,31 @@
-import { nodeIsActive } from 'tiptap-utils'
 import { wrapInList, liftListItem } from 'prosemirror-schema-list'
 
-export default function toggleList(type, itemType) {
-  return (state, dispatch, view) => {
-    const isActive = nodeIsActive(state, type)
-
-    if (isActive) {
-      return liftListItem(itemType)(state, dispatch, view)
-    }
-
-    return wrapInList(type)(state, dispatch, view)
-  }
+function isList(node, schema) {
+  return (node.type === schema.nodes.bullet_list || node.type === schema.nodes.ordered_list)
 }
 
-// https://discuss.prosemirror.net/t/list-type-toggle/948
+export default function toggleList(listType) {
+  return (state, dispatch) => {
+    const { schema } = state
+    const lift = liftListItem(schema.nodes.list_item)
+    const wrap = wrapInList(listType)
+    const { $from, $to } = state.selection
+    const range = $from.blockRange($to)
+    if (!range) {
+      return false
+		}
 
-// import { wrapInList, liftListItem } from 'prosemirror-schema-list'
+    if (range.depth >= 1 && $from.node(range.depth).type === listType) {
+      return lift(state, dispatch)
+    }
 
-// function isList(node, schema) {
-//   return (node.type === schema.nodes.bullet_list || node.type === schema.nodes.ordered_list)
-// }
+    if (range.depth >= 1 && isList($from.node(range.depth), schema)) {
+      const { tr } = state
+      tr.setNodeMarkup(range.start - 1, listType)
+      if (dispatch) dispatch(tr)
+      return false
+    }
 
-// export default function toggleList(listType, schema) {
-//   const lift = liftListItem(schema.nodes.list_item)
-// 	const wrap = wrapInList(listType)
-
-//   return (state, dispatch) => {
-//     const { $from, $to } = state.selection
-//     const range = $from.blockRange($to)
-//     if (!range) {
-//       return false
-// 		}
-
-//     if (range.depth >= 2 && $from.node(range.depth - 1).type === listType) {
-//       return lift(state, dispatch)
-//     } else if (range.depth >= 2 && isList($from.node(range.depth - 1), schema)) {
-//       const tr = state.tr
-// 			const node = $from.before(range.depth - 1)
-// 			console.log({node})
-//       // TODO: how do I pass the node above to `setNodeType`?
-//       // tr.setNodeType(range.start, listType);
-//       if (dispatch) dispatch(tr)
-//       return false
-//     } else {
-//       return wrap(state, dispatch)
-//     }
-//   }
-// }
+    return wrap(state, dispatch)
+  }
+}