[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: two small patches
From:       Ingo =?utf-8?q?Kl=C3=B6cker?= <kloecker () kde ! org>
Date:       2005-02-28 23:48:14
Message-ID: 200503010048.16474 () erwin ! ingo-kloecker ! de
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Monday 28 February 2005 20:50, Aaron Seigo wrote:
> hi all..
>
> attached is a diff for two small patches:
>
> 0. make the border around the presence information use mid() instead
> of the foreground colour. currently it has a "box in box" visual
> effect that's less pleasing IMHO.

a) Your patch changes the printing CSS which is not what you intended to 
do, or did you?
b) I've already committed a corresponding change for the screen CSS 
before the freeze. So you are beating a dead horse. (Is this metaphor 
applicable in this situation?)

> 1. change the icon from refresh to mail_get for the "Check mail in
> this folder" item in the folder context menu. right now there are
> _two_ refresh icons (which means one can't just rely on the icon for
> identification, one needs to read), and since it says "Check mail"
> (even though it does actually refresh, technically) it makes sense
> IMHO to sync this icon with the other appearances of "Check mail in"
> actions in kmail.

I'll leave that one for Till to decide.

Regards,
Ingo

[Attachment #5 (application/pgp-signature)]

_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel


[prev in list] [next in list] [prev in thread] [next in thread]