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

💡 [FEAT] Support for for nesting of TODOs #70

Closed
TOMES-TT opened this issue Sep 11, 2022 · 7 comments
Closed

💡 [FEAT] Support for for nesting of TODOs #70

TOMES-TT opened this issue Sep 11, 2022 · 7 comments
Labels
enhancement New feature or request

Comments

@TOMES-TT
Copy link

Is your feature request related to a problem? 😢 Please describe.
For my use case, I find that having only one layer of nesting for child todos is very limiting (ie, I often need to do tasks 1, 2, and 3 in order to accomplish task A, which itself is one of multiple tasks required to complete todo foo)

Describe the solution you'd like 🤔
I would like A to create a child under any todo, including a child todo.

@TOMES-TT TOMES-TT added the enhancement New feature or request label Sep 11, 2022
@kraanzu
Copy link
Owner

kraanzu commented Sep 11, 2022

Hey @TOMES-TT ,

I had a discussion on this before here

The point about code complications seems vague now.
It was just bad design decisions that I plan to work on soon.

However, the first point still remains valid
I tried nesting too much which made the date column something like this:

The upper/parent tasks' date rendered ideally as they were supposed to be
But in the nested regions the letters got separated into different lines creating something like this:

No
 Due
Date

Idea 💡:
What I can do now is to allow the users to tweak the UI and provide the respected ratios they want to assign to each column and the number of columns they want to see (for e.g. they don't want the urgency column they can remove that)

Example config:

tasks: 70
date: 20
urgency: 10

Where the number shows the ratio to divide the columns into ( according to the terminal's width ) where 0 means don't show the column

What are your thoughts on this?

@TOMES-TT
Copy link
Author

Hi,

Thanks for the response! My mistake for not realizing there was already discussion on this topic, I thought I had checked everywhere but I'm still pretty new to github so I must have missed it. I think that would be a good solution.

@kraanzu
Copy link
Owner

kraanzu commented Sep 12, 2022

My mistake for not realizing there was already discussion on this topic

No problemo!

I think that would be a good solution.

I'm a bit busy with my work and am not getting too much time for github stuff, so might be a while 😞 .
I'll keep this thread open and will update once implemented

@kraanzu kraanzu changed the title 💡 [FEAT] 💡 [FEAT] Support for for nesting of TODOs Sep 12, 2022
@TOMES-TT
Copy link
Author

Yeah, no worries; obviously I'm just requesting stuff from you, so I have no expectations about it :)
Something that just occurred to me; due dates could be made optional, which would let the todo column stretch out to basically fill the entire terminal window

@kraanzu
Copy link
Owner

kraanzu commented Sep 12, 2022

Something that just occurred to me; due dates could be made optional, which would let the todo column stretch out to basically fill the entire terminal window

Yes, I did mention that:

where 0 means don't show the column

@kraanzu
Copy link
Owner

kraanzu commented Sep 12, 2022

I am also thinking to make the Workspace or the left section toggleable (with animation).
The current code wouldn't allow that but I'll implement it soon.

@kraanzu kraanzu added this to the v1.0 milestone Oct 5, 2022
@kraanzu
Copy link
Owner

kraanzu commented Mar 2, 2023

Implemented in v1.0

@kraanzu kraanzu closed this as completed Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants