[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#457169: patch for qa.debian.org



On Fri, Dec 21, 2007 at 04:50:31PM +0100, Yves-Alexis Perez wrote:
> The simple patch attached fixes the problem on high dpi displays and looks
> good on a 96 dpi one. I guess removing all fonts: ##pt (and maybe replace them
> with a px one) would help but I don't know enough css/designing.

No, sorry, this is not an acceptable patch (at least not by me), this is
the removal of the current font size choices which, if it is one, is a
different bug than that related to this bugreport.

A patch for this bug would be changing the font sizes so that they use
relative units instead of absolute ones (as you suggest in the end of
your last message) preserving the current look and feel. The tricky part
of doing that is that according to my tests "em" units are not
cross-browsers compatible: I've tried with epiphany and firefox when the
font size choices where in "em" and the pages looked remarkably
different. It is possible that there were issues elsewhere though ...

Cheers.

-- 
Stefano Zacchiroli -*- PhD in Computer Science ............... now what?
zack@{cs.unibo.it,debian.org,bononia.it} -%- http://www.bononia.it/zack/
(15:56:48)  Zack: e la demo dema ?    /\    All one has to do is hit the
(15:57:15)  Bac: no, la demo scema    \/    right keys at the right time

Attachment: signature.asc
Description: Digital signature


Reply to: