Change the logic for calculating item position

mr-mig · Kristján Oddsson · commit 3eba7a6e4a91 · 2021-08-31T09:24:59.000+01:00
Use tagName === `LI` to calculate item position within a list
diff --git a/src/task-lists-element.ts b/src/task-lists-element.ts
@@ -132,7 +132,7 @@ function position(checkbox: HTMLInputElement): [number, number] {
   const list = taskList(checkbox)
   if (!list) throw new Error('.contains-task-list not found')
   const item = checkbox.closest('.task-list-item')
-  const listItems = Array.from(list.children).filter(el => el.classList.contains('task-list-item'))
+  const listItems = Array.from(list.children).filter(el => el.tagName === 'LI')
   const index = item ? listItems.indexOf(item) : -1
   return [listIndex(list), index]
 }

Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,7 @@ function position(checkbox: HTMLInputElement): [number, number] {`
`132`	`132`	`const list = taskList(checkbox)`
`133`	`133`	`if (!list) throw new Error('.contains-task-list not found')`
`134`	`134`	`const item = checkbox.closest('.task-list-item')`
`135`		`- const listItems = Array.from(list.children).filter(el => el.classList.contains('task-list-item'))`
	`135`	`+ const listItems = Array.from(list.children).filter(el => el.tagName === 'LI')`
`136`	`136`	`const index = item ? listItems.indexOf(item) : -1`
`137`	`137`	`return [listIndex(list), index]`
`138`	`138`	`}`