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 _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
