Fix #4184: Added fractional time parsing when creating a new task

Fixed the issue where you could not insert 1.5h or any decimal value
when inputing time stamp in the quick add method of creating a task. To
test execute project with instructions given in the repository and add a
new task (shift+A) and write the title of a new task such as "task 1.5h"
or "task 1.5h/3.5h". Also added test cases for this fix.

Author: Sofia Lucas

Diff for: src/app/features/tasks/short-syntax.spec.ts

@@ -171,6 +171,44 @@ describe('shortSyntax', () => {
       });
     });
 
+    it('', () => {
+      const t = {
+        ...TASK,
+        title: 'Fun title whatever 1.5h',
+      };
+      const r = shortSyntax(t, CONFIG);
+      expect(r).toEqual({
+        newTagTitles: [],
+        remindAt: null,
+        projectId: undefined,
+        taskChanges: {
+          title: 'Fun title whatever',
+          timeEstimate: 5400000,
+        },
+      });
+    });
+
+    it('', () => {
+      const t = {
+        ...TASK,
+        title: 'Fun title whatever 1.5h/2.5h',
+      };
+      const r = shortSyntax(t, CONFIG);
+      expect(r).toEqual({
+        newTagTitles: [],
+        remindAt: null,
+        projectId: undefined,
+        taskChanges: {
+          title: 'Fun title whatever',
+          // timeSpent: 7200000,
+          timeSpentOnDay: {
+            [getWorklogStr()]: 5400000,
+          },
+          timeEstimate: 9000000,
+        },
+      });
+    });
+
     it('should ignore time short syntax when disabled', () => {
       const t = {
         ...TASK,

Diff for: src/app/features/tasks/short-syntax.ts

@@ -19,7 +19,8 @@ type DueChanges = {
   plannedAt?: number;
 };
 
-const SHORT_SYNTAX_TIME_REG_EX = / t?(([0-9]+(m|h|d)+)? *\/ *)?([0-9]+(m|h|d)+)/;
+const SHORT_SYNTAX_TIME_REG_EX =
+  /(?:\s|^)t?((\d+(?:\.\d+)?[mhd])(?:\s*\/\s*(\d+(?:\.\d+)?[mhd]))?(?=\s|$))/i;
 // NOTE: should come after the time reg ex is executed so we don't have to deal with those strings too
 
 const CH_PRO = '+';
@@ -317,25 +318,28 @@ const parseTimeSpentChanges = (task: Partial<TaskCopy>): Partial<Task> => {
   }
 
   const matches = SHORT_SYNTAX_TIME_REG_EX.exec(task.title);
-
   if (matches && matches.length >= 3) {
     const full = matches[0];
-    const timeSpent = matches[2];
-    const timeEstimate = matches[4];
-
+    const timeSpent = matches[2]; // First part (before slash)
+    const timeEstimate = matches[3]; // Second part (after slash)
+    // If no slash, use the single value as timeEstimate only
+    const hasSlashFormat = matches[3] !== undefined;
     return {
-      ...(timeSpent
+      ...(hasSlashFormat && timeSpent
         ? {
             timeSpentOnDay: {
               ...(task.timeSpentOnDay || {}),
               [getWorklogStr()]: stringToMs(timeSpent),
             },
           }
         : {}),
-      timeEstimate: stringToMs(timeEstimate),
-      title: task.title.replace(full, ''),
+      ...(timeEstimate
+        ? { timeEstimate: stringToMs(timeEstimate) }
+        : timeSpent
+          ? { timeEstimate: stringToMs(timeSpent) }
+          : {}),
+      title: task.title.replace(full, '').trim(),
     };
   }
-
   return {};
 };

Diff for: src/app/ui/duration/string-to-ms.pipe.ts

@@ -5,6 +5,24 @@ export const stringToMs = (strValue: string, args?: any): number => {
     return 0;
   }
 
+  // First try to parse simple formats like "1.5h", "30m", etc.
+  const simpleFormatMatch = strValue.trim().match(/^([0-9]*\.?[0-9]+)([smhd])$/i);
+  if (simpleFormatMatch) {
+    const amount = parseFloat(simpleFormatMatch[1]);
+    const unit = simpleFormatMatch[2].toLowerCase();
+
+    switch (unit) {
+      case 's':
+        return amount * 1000;
+      case 'm':
+        return amount * 1000 * 60;
+      case 'h':
+        return amount * 1000 * 60 * 60;
+      case 'd':
+        return amount * 1000 * 60 * 60 * 24;
+    }
+  }
+
   let d: number | undefined;
   let h: number | undefined;
   let m: number | undefined;