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

Lukas Appelhans l.appelhans at gmx.de
Fri Aug 14 15:30:25 CEST 2009


Am Freitag 14 August 2009 15:11:51 schrieb 潘卫平(Peter Pan):
> Lukas Appelhans 写道:
> > 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...
>
> done. I use m_isBusy. Thanks a lot.
That looks nice to me! Thanks for your patch! If you have an SVN-Account you 
can commit it, else I will do it :)

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


More information about the Plasma-devel mailing list