Merge pull request obsidian-tasks-group#3073 from ilandikov/feat-post…

…pone-item-check

feat: check postpone item if its date matches the one in the task

src/ui/Menus/PostponeMenu.ts

@@ -38,11 +38,24 @@ export class PostponeMenu extends TaskEditingMenu {
  itemNamingFunction: NamingFunction,
  postponingFunction: PostponingFunction,
  ) => {
+ // TODO some of the code below is duplicated in postponeOnClickCallback() and may be refactored
+ let isCurrentValue = false;
+ const dateFieldToPostpone = getDateFieldToPostpone(task);
+ if (dateFieldToPostpone) {
+ const { postponedDate } = postponingFunction(task, dateFieldToPostpone, timeUnit, amount);
+
+ if (task[dateFieldToPostpone]?.isSame(postponedDate, 'day')) {
+ isCurrentValue = true;
+ }
+ }
+
  const title = itemNamingFunction(task, amount, timeUnit);
- // TODO Call setChecked() to put a checkmark against the item, if it represents the current task field value.
- item.setTitle(title).onClick(() =>
- PostponeMenu.postponeOnClickCallback(button, task, amount, timeUnit, postponingFunction, taskSaver),
- );
+
+ item.setChecked(isCurrentValue)
+ .setTitle(title)
+ .onClick(() =>
+ PostponeMenu.postponeOnClickCallback(button, task, amount, timeUnit, postponingFunction, taskSaver),
+ );
  };
 
  const fixedTitle = fixedDateMenuItemTitle;

tests/ui/Menus/PostponeMenu.test.ts

@@ -88,7 +88,7 @@ describe('PostponeMenu', () => {
  const itemsAsText = contentsOfPostponeMenuForTask(new TaskBuilder().startDate(today));
  expect(itemsAsText).toMatchInlineSnapshot(`
  "
-   Start today, on Sun 3rd Dec
+ x Start today, on Sun 3rd Dec
  Start tomorrow, on Mon 4th Dec
  ---
  Start in 2 days, on Tue 5th Dec
@@ -114,7 +114,7 @@ describe('PostponeMenu', () => {
  expect(itemsAsText).toMatchInlineSnapshot(`
  "
  Scheduled today, on Sun 3rd Dec
-   Scheduled tomorrow, on Mon 4th Dec
+ x Scheduled tomorrow, on Mon 4th Dec
  ---
  Postpone scheduled date by 2 days, to Wed 6th Dec
  Postpone scheduled date by 3 days, to Thu 7th Dec