search

ivan-lednev / obsidian-day-planner

An Obsidian plugin for day planning with a clean UI and a simple task format
MIT License
2.11k stars 496 forks source link

Allow en-dash for time ranges #478

Open 3nt3 opened 5 months ago

3nt3 commented 5 months ago

You'd just need to change the Regex I think.

claremacrae commented 3 months ago

Why is this is important, though?

As the maintainer of the Tasks plugin, I would caution against this, though.

By having around half a million downloads, the Day Planner plugin has created a kind of de-facto standard for putting dates at the start of time lines.

So I think that changing the format here would have a much larger impact than on just this plugin.

3nt3 commented 2 months ago

Why is this is important, though?

Because that is the typographically correct way to specify ranges in many languages. If you have a grammar checker then it will flag the wrong dash as an error.

It looks bad especially if you're doing it correctly in all other documents

claremacrae commented 2 months ago

Thanks for the context - much appreciated.

Actually, I'm going to recant on the following:

So I think that changing the format here would have a much larger impact than on just this plugin.

  • It would effectively require all future plugins that want to use times like this to support en-dashes as well.
  • Also, any users of the Tasks plugin who have custom searches based on times would need to know to make their own regexes cope with both characters.

It doesn't really require those things.

Users who only use this plugin can take advantage of en-dashes if they wish, if someone volunteers to do the work, and it's merged....

Users of other related plugins are free to to stick with hyphens if they wish.

Adding en-dash support here doesn't force any user to adopt them, and it doesn't force any other developers to add support for them either...