On Monday 08 December 2008, zhangqiang wrote:
> In order to help general users.I make a patch.I don't know whether it is
> needed and suitable.That is, if the user deletes the last panel,give the
> tip "This is the last panel! Make sure you know how to add it.",that is
> unlike the usual tip "Do you really want to remove the panel ?" I do it in
> the containment, and my patch attached. Thanks very much.

first, we can't make such a change until trunk opens for 4.3 development as it 
contains new strings.

the patch, however, is also incorrect in a few ways: not all panel 
containments are called "panel", and keeping a count is unecessary.

instead, simply loop through Corona::containments() and look for 
containmentType() == PanelContainment or CustomPanelContainment

moreover, i don't think this belongs in libplasma since not all plasma based 
applications will care about this. it really belongs in the desktop shell 
application.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to