[Kde-pim] Review Request: Warn if a mail is sent to too many recipients
Ingo Klöcker
kloecker at kde.org
Wed Feb 25 18:30:44 GMT 2009
On Friday 20 February 2009, Allen Winter wrote:
> > On 2009-02-20 05:11:20, Allen Winter wrote:
> > > branches/kdepim/enterprise/kdepim/kmail/kmcomposewin.cpp, line
> > > 4356
> > > <http://reviewboard.kde.org/r/133/diff/1/?file=837#file837line435
> > >6>
> > >
> > > thomas will yell at you about coding style. in kdepim, we
> > > put the opening brace on the same lin e with the 'if'.
> >
> > wrote:
> > It is a wish only. :) Both cases are present in that very same
> > file.
>
> yep. we has been trying to unify to one style for a long time. slow
> going.
>
> > On 2009-02-20 05:11:20, Allen Winter wrote:
> > > branches/kdepim/enterprise/kdepim/kmail/kmail.kcfg, line 344
> > > <http://reviewboard.kde.org/r/133/diff/1/?file=835#file835line344
> > >>
> > >
> > > please add a <tooltip> and a <whatsthis>
> >
> > wrote:
> > Makes sense, the place from where I copied this part didn't
> > have the tooltip/whatsthis (a few lines above). ;)
>
> yep. we are missing *hundreds* of tooltips and whatsthis in kmail.
>
> > On 2009-02-20 05:11:20, Allen Winter wrote:
> > > branches/kdepim/enterprise/kdepim/kmail/configuredialog.cpp, line
> > > 2646
> > > <http://reviewboard.kde.org/r/133/diff/1/?file=833#file833line264
> > >6>
> > >
> > > I wonder if a max of 255 is enough?
> >
> > wrote:
> > Well, i cannot imagine somebody wanting this warning only if he
> > sends it to 1000+ recipients, but not to 999. :) But of course, I
> > can increase it.
>
> good point. 255 does sound like a reasonable number then.
255 is a weird number from a non-computer-scientist-user perspective.
Please, make it 250 (or 100 or 200 or 500).
Regards,
Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20090225/4b72ebf6/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
More information about the kde-pim
mailing list