search

bastibe / org-journal

A simple org-mode based journaling mode
BSD 3-Clause "New" or "Revised" License
1.25k stars 124 forks source link

Use inactive timestamp in headline rather than CREATED property #346

Open xeruf opened 3 years ago

xeruf commented 3 years ago

Is your feature request related to a problem? Please describe. This is sort of a follow-up to https://github.com/bastibe/org-journal/issues/188 Currently, journal entries look like this:

* Monday, 2021-05-03
:PROPERTIES:
:CREATED:  20210503
:END:

or, with the customization from the aforementioned issue, this:

* Monday, 2021-05-03
:PROPERTIES:
:CREATED:  [2021-05-03]
:END:

Describe the solution you'd like How about, instead of a separate created property, embedding it right in the headline? For example, my handcrafted file I used before org-journal looked like this:

* [2021-04-16 Fri] New Creations
** 10 Browsing
** 11 Creativity LU Reflection
** 13 Administration :shell:
** 18 Zsh completion :shell:
* [2021-04-15 Thu] Independence
** Read in Walden
** 10 Haskell linking :haskell:

Describe alternatives you've considered I can customize both org-journal-date-format and org-journal-created-property-timestamp-format, but I dislike redundant information ;)

bastibe commented 3 years ago

I like the look of this.

But I think some users will still prefer the property for their org-mode or agenda integration.

xeruf commented 3 years ago

This could be implemented by not setting the created property when org-journal-created-property-timestamp-format is nil. Then it expects the respective constraints to be fulfilled in org-journal-date-format instead so it can search these.

casch-at commented 3 years ago

This could be implemented by not setting the created property when org-journal-created-property-timestamp-format is nil. Then it expects the respective constraints to be fulfilled in org-journal-date-format instead so it can search these.

Good idea. We wouldn't need to change to much than.

There is also the possibility to mark text as invisible. However, I currently don't know how this behaves when there follows some more text on the same line....

xeruf commented 3 years ago

Why would we make it invisible? I like having the timestamp there.

casch-at commented 3 years ago

I missed the fact that you meant to put it right in the header instead of the date. Anyway, the property is just used internally by org-journal to find the journals, hence we could hide it. We can of course add a variable to make it optionally...

xeruf commented 3 years ago

No need to add a separate variable. As said, simply add the possibility for org-journal-created-property-timestamp-format to be nil.