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

Lukas Appelhans l.appelhans at gmx.de
Fri Aug 14 14:51:56 CEST 2009


Am Freitag 14 August 2009 14:00:40 schrieb 潘卫平(Peter Pan):
> Lukas Appelhans 写道:
> > 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()?
>
> In the beginning, I want to quit as soon as possible, to save CPU cycles.
> But I agree that addProgram() is a more elegant place, but we should pay
> attention to initializing and sorting.
>
> > 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!
>
> agree with you.
> And I modify my patch.
Ok that looks much nicer... :) Now I think we can merge m_isSorting and 
m_isInitializing to m_isBusy or m_notAdding or sth like this...

Lukas
>
> > Regards,
> >
> > Lukas
> > _______________________________________________
> > Plasma-devel mailing list
> > Plasma-devel at kde.org
> > https://mail.kde.org/mailman/listinfo/plasma-devel


More information about the Plasma-devel mailing list