gtaubman / doneyet

A NCurses based To-Do list manager!
MIT License
31 stars 6 forks source link

Change Hierarchical List to be rooted at a task instead of at a vector #14

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
At the beginning I made a bad decision to store top level tasks in a vector, 
but all subtasks go inside parent tasks.  This has required separate 
functions for root tasks and all other tasks.  Allowing the list view to be 
rooted at a root Node instead of a root Vector will simplify everything, and 
allow us to fix other issues such as issue 1.

Original issue reported on code.google.com by gtaub...@gmail.com on 15 Jul 2009 at 1:46

stweise commented 5 years ago

Hey @gtaubman do you have any thoughts on this? Could you please look at the PR?

stweise commented 5 years ago

The proposed changes introduced a lot of other issues. I could not solve this correctly and I still think that representing the initial list of listroots, which are then hierarchical lists is not a bad decision in terms of matching model to code.