In article
<[email protected]>,
   Chris Young <[email protected]> wrote:
> On Thu, 11 Dec 2008 21:42:06 +0000 (GMT), Michael Drake wrote:

> I had a look at this.  I can't disable the outline, but I could
> theoretically set it to be transparent.  This doesn't work (probably
> need to enable something I don't have access to), so I've left it as
> it is for now.

That's a pity, because it would make the display look 'cleaner'.

> > There may be a similar problem with the fill plotter

> I think you're right.  I've fixed this now.

Great!

> > Also, on the BBC site, the headings like News, Sport and Weather are
> > very small.

> I don't know why this would be.  On other pages the headings are more
> correct.  Maybe it is because I'm taking the size directly from the
> style structure (style->font_size.value.length.value) and using that
> as the font height.

Ah! That's wrong, the font_size.value.length.value could be anything
depending on what the unit of the length is. EG it may be in px, pt, em,
etc. If you need the value in px, use:

    css_len2px(&style->font_size.value.length, style)

which converts whatever unit the length is in to px. There is also a
css_len2pt, if that's more useful. These are in css/css.h.

> I've set it to 72, which is what is traditionally taken as screen DPI
> here.  It's reduced the space between lines but doesn't appear to have
> affected text size.

The css_len2px and css_len2pt functions use the dpi setting, so it should
all work correctly when you use one of them.

> > Do you know if the SDK you need for that will be available by NetSurf
> > 2.0 release time?

> The information I've been given would suggest so.

Yay!

Michael

-- 

Michael Drake (tlsa)                  http://www.netsurf-browser.org/


Reply via email to