Hi, On Wed, 2002-12-04 at 03:35, Jason R. Mastaler wrote: > David Guerizec <[EMAIL PROTECTED]> writes: > > > Added Files: > > tmda-gui > > This is pretty swank, I think users will enjoy this. It might also > give me an excuse to blow the dust off my copy of John Grayson's > _Python and Tkinter Programming_ :-) > > Thanks David.
My pleasure ;) > > NOTE: This is early alpha software, so please try it, and send your > > comments on the tmda-gui or tmda-workers list. > > Even though Gre7g named his list ``tmda-gui'', I thought his intention > was only to discuss his CGI application. Is this not correct? I think > the normal TMDA lists would be a better place to discuss the Tkinter > app, particularly if we keep it in bin/. No problem, I should have asked before writing this, but it was two in the morning and I really needed to go to bed. Either list will do for me since I'm subscribed to both. > I'll look at tmda-gui more carefully tomorrow when I have more time, > but I do have a few initial comments: > > * tmda-gui should probably use the getopt module for its command-line > parsing as the other bin/tmda-* programs do. I've put some of the > supporting code in already. OK, although it shouldn't have much command switches. In fact, at first I planned to make two different executables, but now I think it's more practical to have only one. The reason is that I intend to be able to use it without TMDA library, ie. through a network connection, so it'll be easier to copy only one file to any client. > * You do `from TMDA import Pending' many times throughout the > code. Why not just once at the top of the program? I've fixed > this. If there was a particular reason why you did this, I can back > this out, but it seems to work fine. No, there was no particular reason, just an ersatz from an earlier design phase ;) As I said above, I planned to make it TMDA library "independent", so I'll need to make a wrapper class to be able to switch to a networked communication with Pending and Address modules. For this reason, it makes sense actually to have it in only one place. > * Shouldn't the pending mode ignore already delivered messages? > Currently it includes them in the list. I changed this as well, and > again I can back it out if this wasn't what you intended. Well, in fact I planned to make this an option in the Settings box, but it's fine for the moment. Note that the messages were first sorted by their status, so the recent and non-confirmed are on top of the list, hence directly visible. Also, I plan to hide the message id and replace it by the status of the message (R, C, ' '). I think the message id would be too confusing for an average user. David _________________________________________________ tmda-workers mailing list ([EMAIL PROTECTED]) http://tmda.net/lists/listinfo/tmda-workers
