#27 Facility to add current interval to previous tasks

open
None
5
2009-06-15
2007-09-20
Hans Baier
No

I use gnotime every day for work, and implemented a feature I was sorely missing:

At work I frequently have to go back to previous tasks and work on it, so
I implemented a Task Chooser in the notes area
to attach the current interval to previous tasks.

The implementation adds a pointer current_task to
the project structure which can be a past task, e.g.
that is not at the head of task_list.

It seems to work fine for me here,
except it displays the head task in the task notes
area and in the ctree and not the task currently
added to, which is fine for me,
but probably needs to be changed for public use.

Please let me know what you think about it.
And whether I should continue hacking.

(For remarks about style I am greatful too
since I didn't do C programming for some 10 years now,
only Java......)

Thanks for the nice software,

Cheers,
Hans

Discussion

  • Hans Baier
    Hans Baier
    2007-09-20

    Logged In: YES
    user_id=1666622
    Originator: YES

    typo

     
  • Hans Baier
    Hans Baier
    2007-09-20

    • summary: Facility to add currend interval to previous tasks --> Facility to add current interval to previous tasks
     
  • Hans Baier
    Hans Baier
    2007-10-12

    Logged In: YES
    user_id=1666622
    Originator: YES

    Now the active task shows up in the notes area.
    I did some refactoring to clean up the code, esp. added getters and setters
    for current_task in the project structure.
    The patches are against current HEAD.

    I sure would appreciate some feedback.
    File Added: gnotime-choose-active-task.diff

     
  • Hans Baier
    Hans Baier
    2007-10-12

    Logged In: YES
    user_id=1666622
    Originator: YES

    File Added: task_select.glade

     
  • Hans Baier
    Hans Baier
    2007-10-12

    Logged In: YES
    user_id=1666622
    Originator: YES

    File Added: gnotime-choose-active-task.diff

     
  • Hans Baier
    Hans Baier
    2007-10-12

    Logged In: YES
    user_id=1666622
    Originator: YES

    File Added: notes.glade.diff

     
  • Hans Baier
    Hans Baier
    2007-10-13

    Logged In: YES
    user_id=1666622
    Originator: YES

    File Added: gnotime-choose-active-task.diff

     
  • Hans Baier
    Hans Baier
    2007-10-13

    put all the changes in one single diff

     
    • assigned_to: nobody --> goedson
     
  • Thank you for the patch. I've added it to our source code repository in the choose-task branch.
    I've changed it so the selection is done directly in the notes area through a combobox instead of opening a dialog.

    There are still some bits to be polished (like the display of "current task time" in the project tree) but looks fine overall.