[patch] Bug 157226: "Plain Clock Font" only changes the time

[Sebastian Kügler]

On Thursday 27 March 2008 15:55:51 Beaudoin Gael wrote:
> This patch fixes bug #157226 : "Plain Clock Font" only changes the time.
>
> Now It changes the date too.
> Seams trivial but who knows ...

I had thought about that, too. The reason why I (intentionally) left it at 
smallestReadableFont is because the pointSize() is only valid for this 
particular font (you set it in system settings). Other fonts might not be 
readable with this pointSize(). Also, for the digits one might want to use a 
different, maybe more fancy font. That font might not look so well with text. 
To change the font of the date, one could:

- Make it configurable in the config dialogue (overkill?)
- Have it configurable in system settings (currently) (discoverability?)
- Make it the same font as the clock face (your patch) (readability?)

> ... ok to commit ?

I'd rather first have comment from others.
-- 
sebas

 http://www.kde.org | http://vizZzion.org |  GPG Key ID: 9119 0EF9 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 481 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080327/e7819dfc/attachment.pgp