[PATCH] BUG 203716 give user a hint when adding same application into quicklaunch

Lukas Appelhans l.appelhans at gmx.de
Fri Aug 14 11:53:54 CEST 2009


Am Freitag 14 August 2009 11:10:42 schrieb 潘卫平(Peter Pan):
> Hi,all
>
> I think we should give user a hint when he/she adds a application which
> already exists in quicklaunch, so I made this patch.
>
>
> Regards
Hey!

Good thing! But 2 things: I'm not 100% into the code currently, but why not 
implementing the loop inside addProgram()?

And the second one: "Cancel or Continue?" is not needed as words inside the 
MessageBox imo, as the buttons are named like it... "\"%1\" is already in 
quicklaunch!"  should be enough!

Regards,

Lukas


More information about the Plasma-devel mailing list