Tasks now adds all known obsidian attributes

The way it currently works means that it will not automatically pick up
new attributes as they get added by obsidian.

Fixes #45

src/InlineRenderer.ts

@@ -83,7 +83,14 @@ export class InlineRenderer {
                 continue;
             }
 
-            const cachedElement = await task.toLi({ parentUlElement: element });
+            const dataLine: string =
+                renderedElement.getAttr('data-line') ?? '0';
+            const listIndex: number = Number.parseInt(dataLine, 10);
+            const cachedElement = await task.toLi({
+                parentUlElement: element,
+                listIndex,
+            });
+
             // If the rendered element contains a sub-list, we need to keep it.
             renderedElement.findAll('ul').map((renderedSubUl) => {
                 cachedElement.appendChild(renderedSubUl);

src/QueryRenderer.ts

@@ -90,16 +90,25 @@ class QueryRenderChild extends MarkdownRenderChild {
             tasks = tasks.filter(filter);
         }
 
+        const tasksSorted = Sort.byDateThenPath(tasks);
+        const tasksCount = tasksSorted.length;
+
         const taskList = content.createEl('ul');
         taskList.addClass('contains-task-list');
-        for (const task of Sort.byDateThenPath(tasks)) {
+        for (let i = 0; i < tasksCount; i++) {
+            const task = tasksSorted[i];
+
             let fileName: string | undefined;
             const fileNameMatch = task.path.match(/([^/]+)\.md$/);
             if (fileNameMatch !== null) {
                 fileName = fileNameMatch[1];
             }
 
-            const listItem = await task.toLi({ parentUlElement: taskList });
+            const listItem = await task.toLi({
+                parentUlElement: taskList,
+                listIndex: i,
+            });
+
             if (fileName !== undefined) {
                 const link = listItem.createEl('a');
                 link.href = fileName;

src/Task.ts

@@ -157,12 +157,14 @@ export class Task {
 
     public async toLi({
         parentUlElement,
+        listIndex,
     }: {
         parentUlElement: HTMLElement;
+        /** The nth item in this list (including non-tasks). */
+        listIndex: number;
     }): Promise<HTMLLIElement> {
         const li: HTMLLIElement = parentUlElement.createEl('li');
         li.addClass('task-list-item');
-        li.setAttr('data-task', this.originalStatusCharacter.trim()); // Trim to ensure empty attribute for space. Same way as obsidian.
 
         let taskAsString = this.toString();
         const { globalFilter, removeGlobalFilter } = getSettings();
@@ -204,6 +206,11 @@ export class Task {
 
         li.prepend(checkbox);
 
+        // Set these to be compatible with stock obsidian lists:
+        li.setAttr('data-task', this.originalStatusCharacter.trim()); // Trim to ensure empty attribute for space. Same way as obsidian.
+        li.setAttr('data-line', listIndex);
+        checkbox.setAttr('data-line', listIndex);
+
         return li;
     }