refactoring

packages/core/src/helpers/getMarksBetween.ts

@@ -1,19 +1,13 @@
-import { Mark } from 'prosemirror-model'
 import { EditorState } from 'prosemirror-state'
+import { MarkRange } from '../types'
 
-export type MarkPosition = {
-  mark: Mark,
-  start: number,
-  end: number,
-}
+export default function getMarksBetween(from: number, to: number, state: EditorState): MarkRange[] {
+  let marks: MarkRange[] = []
 
-export default function getMarksBetween(start: number, end: number, state: EditorState): MarkPosition[] {
-  let marks: MarkPosition[] = []
-
-  state.doc.nodesBetween(start, end, (node, pos) => {
+  state.doc.nodesBetween(from, to, (node, pos) => {
     marks = [...marks, ...node.marks.map(mark => ({
-      start: pos,
-      end: pos + node.nodeSize,
+      from: pos,
+      to: pos + node.nodeSize,
       mark,
     }))]
   })

packages/core/src/helpers/isMarkActive.ts

@@ -1,14 +1,8 @@
 import { EditorState } from 'prosemirror-state'
-import { Mark, MarkType } from 'prosemirror-model'
+import { MarkType } from 'prosemirror-model'
 import objectIncludes from '../utilities/objectIncludes'
 import getMarkType from './getMarkType'
-import { AnyObject } from '../types'
-
-export type MarkRange = {
-  mark: Mark,
-  from: number,
-  to: number,
-}
+import { AnyObject, MarkRange } from '../types'
 
 export default function isMarkActive(
   state: EditorState,

packages/core/src/helpers/isNodeActive.ts

@@ -1,14 +1,8 @@
 import { EditorState } from 'prosemirror-state'
-import { Node, NodeType } from 'prosemirror-model'
+import { NodeType } from 'prosemirror-model'
 import objectIncludes from '../utilities/objectIncludes'
 import getNodeType from './getNodeType'
-import { AnyObject } from '../types'
-
-export type NodeRange = {
-  node: Node,
-  from: number,
-  to: number,
-}
+import { AnyObject, NodeRange } from '../types'
 
 export default function isNodeActive(
   state: EditorState,