Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

description of todo.txt format in plain text #68

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

dinosv
Copy link

@dinosv dinosv commented Dec 30, 2021

No description provided.

README.md Outdated Show resolved Hide resolved
Co-authored-by: Waldir Pimenta <waldyrious@gmail.com>
@rachmadaniHaryono
Copy link

maybe keep the image?

i sometime forget the rule/format and but always remember the image.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@waldyrious
Copy link

maybe keep the image?

@rachmadaniHaryono isn't the image redundant with the text-based diagram? They contain exactly the same content 🤔

Compare the image:

...and the text-based diagram:

┌─────────────────── Optional - Marks completion 
│  ┌──────────────── Optional - Marks priority 
│  │      ┌─────────────── Optional - Completion date 
│  │      │          ┌──── Optional - Creation date
│  │      │          │     (must be specified if completion date is)
│  │      │          │        ┌─── Mandatory - Description
│  │      │          │        │    Tags (optional) can be placed anywhere within it.
│  │  ┌───┴────┐ ┌───┴────┐ ┌─┴────────────────────────────────────────────────────┐
x (A) 2016-05-20 2016-04-30 measure space for +chapelShelving @chapel due:2016-05-30
                                              └──────┬──────┘ └──┬──┘ └──────┬─────┘       
                                    project tag ─────┘           │           │
                                    context tag ─────────────────┘           │
                                    arbitrary key-value tag ─────────────────┘

@rachmadaniHaryono
Copy link

rachmadaniHaryono commented Jan 1, 2022

i thought about it once again and you are correct. if the target is to have a text-only format, it is the correct way to go. so just ignore my last comment


other than that

  • please add the top bracket to x and (A).
  • should the top text be left-aligned just like the bottom text?
  • does the image will be deleted?

dinosv and others added 2 commits January 1, 2022 09:44
Co-authored-by: Waldir Pimenta <waldyrious@gmail.com>
@dinosv
Copy link
Author

dinosv commented Jan 1, 2022

i thought about it once again and you are correct. if the target is to have a text-only format, it is the correct way to go. so just ignore my last comment

other than that

* please add the top bracket to `x` and `(A)`.

* should the top text be left-aligned just like the bottom text?

* does the image will be deleted?

Thanks, @rachmadaniHaryono! Nice suggestions. I put a new bracket on priority and modified the text alignment. I do not know if putting a bracket over x could be done without changing too much the rest of the block or using some special Unicode character.

About the image, I guess it can be decided by the repository owner. My original idea was to have a file that could be parsed through Pandoc in order to produce and print a txt output.

Thanks everyone, specially @waldyrious for the changes.

@waldyrious
Copy link

@dinosv any thoughts about removing trailing whitespace (suggested at the end of this comment) and replacing "special" with "arbitrary" (suggested in this comment)?

@dinosv
Copy link
Author

dinosv commented Jan 1, 2022

@waldyrious yes, sure. Do you mean something like this?

┌───────────────────────────────── Optional - Marks completion 
│  ┌────────────────────────────── Optional - Marks priority 
│  │   ┌────────────────────────── Optional - Completion date 
│  │   │          ┌─────────────── Optional - Creation date
│  │   │          │                (must be specified if completion date is)
│  │   │          │           ┌─── Mandatory - Description
│  │   │          │           │    Tags (optional) can be placed anywhere within it
│ ┌┴┐ ┌┴───────┐ ┌┴───────┐ ┌─┴────────────────────────────────────────────────────┐
x (A) 2016-05-20 2016-04-30 measure space for +chapelShelving @chapel due:2016-05-30
                                              └────────────┬┘ └────┬┘ └───────────┬┘       
                                   project tag ────────────┘       │              │
                                   context tag ────────────────────┘              │
                                   special key-value tag ─────────────────────────┘

About the change from special to arbitrary, I agree in the view of the "standard" of todotxt, because any tag with key:value format is arbitrary, but some implementations have nested some of those tags (like due: for example), so they are not arbitrary anymore. I would leave that to the developers as well, because it also involves modifying the last section to clarify this aspect.

@waldyrious
Copy link

yes, sure. Do you mean something like this?

Sorry, I don't understand what you changed in that snippet 😅 there's a single space character at the end of the first three lines of the snippet, and seven space characters at the end of the 10th line.

About the change from special to arbitrary, I agree in the view of the "standard" of todotxt, because any tag with key:value format is arbitrary, but some implementations have nested some of those tags (like due: for example), so they are not arbitrary anymore.

I understand your point, but even if some implementations do assign special meaning to specific key-value tags, those implementations still (should) support arbitrary such tags with prefixes they don't recognize, lest they violate the spec of what a valid todo.txt line can be.

That said, I can see that one could argue that such tags only make sense in the context of an implementation that does recognize the specific key prefix, so I guess it's a matter of whether we want to describe what the tags are in terms of formal specification, vs. how they are (expected to be) used in practice. A reasonable argument can be made for either position, so I'm OK with whatever choice you make.

@dinosv
Copy link
Author

dinosv commented Jan 2, 2022

Sorry, I don't understand what you changed in that snippet sweat_smile there's a single space character at the end of the first three lines of the snippet, and seven space characters at the end of the 10th line.

Sorry! Lost in translation! Now I see that you referred to the end-of-line whitespace. I deleted it.

About the change from special to arbitrary, I agree in the view of the "standard" of todotxt, because any tag with key:value format is arbitrary, but some implementations have nested some of those tags (like due: for example), so they are not arbitrary anymore.

I understand your point, but even if some implementations do assign special meaning to specific key-value tags, those implementations still (should) support arbitrary such tags with prefixes they don't recognize, lest they violate the spec of what a valid todo.txt line can be.

That said, I can see that one could argue that such tags only make sense in the context of an implementation that does recognize the specific key prefix, so I guess it's a matter of whether we want to describe what the tags are in terms of formal specification, vs. how they are (expected to be) used in practice. A reasonable argument can be made for either position, so I'm OK with whatever choice you make.

Although I agree with you on this now, maybe it is best to stick with special for now. I suppose @karbassi can give us his insight on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants