Re: [Deskbar] Urgent patch, Fixing ModuleLoader

[Mikkel Kamstrup Erlandsen <kamstrup daimi au dk>]

On Sat, 2005-10-15 at 15:59 +0200, Raphael Slinckx wrote:
> On Sat, 2005-10-15 at 15:10 +0200, Mikkel Kamstrup Erlandsen wrote:
> > ModuleLoader in cvs is broken. It contains a line:
> > 
> > self.set_value(iter, self.NAME_COL, "<b>%s</b>\n%s" % context.name)
> 
> It's not a bug, it's a feature...
With that kind of rhetoric you could get a job at Microsoft :)

> Maybe it wasn't such a good idea, then please suggest another way to do
> it, it's for the preference panel (look at it, the title is in bold)
I see the purpose of this but I'm not sure I like the implementation. My
main concern is that if we do

set_value(iter, NAME_COL, context.name)
then
get_value (iter, NAME_COL) != context.name 

 - but I have no better suggestions atm (that is if we want to have the
popular "<b>Name</b>\nShort description"-layout). If we want to keep it
really simple we could just use a name column and a description column.
But that not really trendy...

Cheers
Mikkel