obsidian-reminder icon indicating copy to clipboard operation
obsidian-reminder copied to clipboard

[Feature]: Support for callouts

Open redous1 opened this issue 3 years ago • 6 comments

Goal or desired outcome of this feature

I like to hide longer lists of tasks within callouts, often making them fold by default. Reminder appears not to support them.

Callouts can prefix tasks with multiple > signs if the callout is nested within other callouts:

> - [ ] Task (@2022-06-19 12:00)
> > - [ ] Task (@2022-06-19 12:00)
> ... > - [ ] Task (@2022-06-19 12:00)

Describe the feature

Reminder should support the > ... > pattern in front of tasks.

Screenshots, mockups, or videos

The reminders here should work:

> [!note] Tasks
> > [!attention]- Reminders
> > - [x] Reminder 1 (@2022-06-19 12:00)
> > - [ ] Reminder 2 (@2022-06-19 15:00)

redous1 avatar Jun 19 '22 06:06 redous1

Please try v1.1.10.

uphy avatar Jun 20 '22 05:06 uphy

It supports one level now. I think you need to change the regex from (> )? to (> )*, allowing 0 or more matches instead of 0 or 1.

redous1 avatar Jun 21 '22 05:06 redous1

I tested it. The above change makes nested callouts work.

redous1 avatar Jun 21 '22 06:06 redous1

Any chance you could implement the change I proposed here? I haven't experienced any noticeable slowdown regardless of the repetition operator.

redous1 avatar Jun 30 '22 09:06 redous1

Thank you for testing! I forgot to fix that. I will fix later.

uphy avatar Jun 30 '22 09:06 uphy

@redous1 Supported nested-callouts with v1.1.12. Can you try it again?

uphy avatar Jul 01 '22 01:07 uphy