Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Sujith H
> On 2009-12-12 14:02:03, Marco Martin wrote: > > replacing something that doesn't interrupt the workflow with something that > > does? > > it's just the opposite of what we're trying to do everywhere else You are absolutely right. Hence I am closing this request. - Sujith -

Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/#review3406 --- replacing something that doesn't interrupt the workflow with something

Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Sujith H
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 13:13:42.435258) Review request for Plasma and Anne-Marie Ma

Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Sujith H
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 13:11:26.536267) Review request for Plasma and Anne-Marie Ma

Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Sujith H
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 12:13:50.835279) Review request for Plasma and Anne-Marie Ma

Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Sujith H
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 12:04:32.331590) Review request for Plasma and Anne-Marie Ma

Re: Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Anne-Marie Mahfouf
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/#review3404 --- A few comments: - keep the same string if you want to get this in 4.4

Review Request: Adding KMessageBox instead of KNotification

2009-12-12 Thread Sujith H
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- Review request for Plasma and Anne-Marie Mahfouf. Summary --- Implementing