From: Razzloss <Raz...@gm...> - 2007-12-15 00:51:48
|
On Friday 14 December 2007 22:10:25 Steven Sheehy wrote: > Jacek Sieka wrote: > >> Fixes the disconnected string showing in the transfers pane before the > >> no slots available status. > > > > I discussed this patch with razzloss and we both came to the conclusion > > that its the responsability of the gui to tell which error messages to > > display from where...besides, I'm anticipating some changes to the > > transfer view now that we have segmented downloading in the core... > > Great, razzloss failed to mention this... > In my defense it was only few (three at most) weeks ago... And I've been pretty busy with school & work lately and so I haven't got any time to do anything related to (Linux)DC++. I've been meaning to talk to you about it, but... yeah. IMHO the state wether to show the disconnect should be added to GUI-part as there are few other things that would benefit from it (setting the sensitivity of some context menu options to false/true atleast, though clearly it is not the killer feature everyone is waiting for, but gives a more finished look for the app). And also I'm assuming that the TTH/rollback failures lead to similar disconnect? So it would be easier just to add one hidden column to transferview and use that to determine if disconnect should be shown. --RZ |