Comment 14 for bug 396736

Revision history for this message
Chris Jones (cmsj) wrote :

Per IRC request from Mirco, I have rebased my patch against lp:notify-osd/karmic and also taken the opportunity to tidy it up.

[11:21] <MacSlow> compare the reported "title-height: ..." and "body-height: ..."
                  from you patched notify-osd with unpatched lp:notify-osd/karmic

After setting my application font to 7pt:

-(cmsj@kiryo)-(/tmp/notify-osd-orig)- killall -15 notify-osd ; DEBUG=1 ./src/notify-osd
notify-osd: no process found
** Message: DEBUG mode enabled
font-size: 7pt
dpi: 127.0
pixels/EM: 12.35
width: 296 px
title-height: 7.00 pt
body-height: 6.30 pt

^C
-(cmsj@kiryo)-(/tmp/notify-osd-orig)- cd ../notify-osd-cmsj/
-(cmsj@kiryo)-(/tmp/notify-osd-cmsj)- killall -15 notify-osd ; DEBUG=1 ./src/notify-osd
** Message: DEBUG mode enabled
font-size: 7.000000pt
dpi: 127.0
pixels/EM: 12.35
width: 296 px
title-height: 7.00 pt
body-height: 6.30 pt

^C
-(cmsj@kiryo)-(/tmp/notify-osd-cmsj)-

Which seems like a good sign :)

With the setting back at 6.5pt:

-(cmsj@kiryo)-(/tmp/notify-osd-orig)- killall -15 notify-osd ; DEBUG=1 ./src/notify-osd
notify-osd: no process found
** Message: DEBUG mode enabled
font-size: 5pt
dpi: 127.0
pixels/EM: 8.82
width: 211 px
title-height: 5.00 pt
body-height: 4.50 pt

^C
-(cmsj@kiryo)-(/tmp/notify-osd-orig)- cd ../notify-osd-cmsj/
-(cmsj@kiryo)-(/tmp/notify-osd-cmsj)- killall -15 notify-osd ; DEBUG=1 ./src/notify-osd
notify-osd: no process found
** Message: DEBUG mode enabled
font-size: 6.500000pt
dpi: 127.0
pixels/EM: 11.47
width: 275 px
title-height: 6.50 pt
body-height: 5.85 pt

^C
-(cmsj@kiryo)-(/tmp/notify-osd-cmsj)-

and the notifications look better :)