More on the to-do list

[rms39 columbia edu (Russell Steinthal)]

As those of you who follow cvs-commits-list probably already know, 
I've committed the beginnings of some of the to-do list enhancements 
I mentioned on the list a few weeks ago, specifically for priorities. 
 I think they work reasonably at this point, but feel free to suggest 
improvements...

While making those changes, though, I came across some anomalous 
code, but am unsure which of two simple fixes is the desired behavior:

The following is from prop.c (build_list_options_frame):
   due_date_show_button = gtk_check_button_new_with_label 
(_("Summary"));
   due_date_show_button = gtk_check_button_new_with_label (_("Due 
Date"));
   priority_show_button = gtk_check_button_new_with_label 
(_("Priority"));

The odd line is the first one: it seems that someone started to make 
Summary hidable, but there's no other code (that I can find) in CVS 
which follows through on that idea.  While it seems like a good idea 
for maximum configurability, I'm not sure what good it would do: it 
would then leave a list of due dates/priorities, without any idea of 
what they are attached with... As an implementation decision, it's 
simpler as well to force it to be shown, since it removes the need 
for code for odd cases like sorting a CList which has no visible 
columns, etc.

So what do people think: should I just remove the anomalous line, or 
add the code necessary to actually allow summaries to be hidden?

BTW, in case people are curious, I think the next changes I'm going 
to work on will be Miguel's request for Percent Completion (since 
it's easy and it's nice to be able to fill one of Miguel's requests, 
since he gets to deal with everyone else's) and then categories...

-Russell




-- 
Russell Steinthal
<rms39@columbia.edu>		Columbia College Class of 1999
<steintr@avnet.org>		System Administrator, AV-Network