Opened 12 years ago

Last modified 12 years ago

#306 closed defect (FIXED)

wiki @font-face links incorrectly

Reported by: Odin Hørthe Omdal Owned by: Jef van Schendel
Priority: minor Milestone:
Component: ui Keywords:
Cc: Parent Tickets:

Description

The style on the wiki:

::

            @font-face {
              font-family: 'Lato';
              font-style: italic;
              font-weight: 700;
              src: local('Lato Bold Italic'), local('Lato-BoldItalic'), url(http://wiki.mediagoblin.org/skins/mediagoblin/fonts/Lato-BoldItalic.ttf?2011-11-02T15:46:40Z) format('truetype');
            }
            @font-face {
              font-family: 'Lato';
              font-style: normal;
              font-weight: 400;
              src: local('Lato Regular'), local('Lato-Regular'), url('fonts/Lato-Regular.woff') format('truetype');
            }

The last line should probably be:

::

    src: local('Lato Regular'), local('Lato-Regular'), url('http://wiki.mediagoblin.org/skins/mediagoblin/fonts/Lato-Regular.woff') format('truetype');

Right now **all** browsers show the wrong font, because noone can
get the Lato-regular one. However, it does look worse in Opera,
because Opera chooses to display the italic font instead of the
bold, whilst the other browsers bolds everything.

Well, looks crappy in all of them, but you can see it's wrong
easier in Opera ;-)



Change History (4)

comment:1 by Jef van Schendel, 12 years ago

Status: NewIn Progress
Thanks for reporting this.

I changed the last line from "Lato-Regular.woff" to
"Lato-Regular.*ttf*". To be honest I don't know why it didn't say
that before, because we also use .ttf everyone on MediaGoblin
itself and the fonts directory only contains .ttf files.

Could you check if it works correctly now?



comment:2 by Odin Hørthe Omdal, 12 years ago

Working well now, yes :-)

I'll also file a bug at work for Operas not exactly incorrect, but
not too smart font fallback selection.



comment:3 by Jef van Schendel, 12 years ago

Status: In ProgressClosed
Cool. Closing this.



comment:4 by Will Kahn-Greene, 12 years ago

The original url for this bug was http://bugs.foocorp.net/issues/654 .

Note: See TracTickets for help on using tickets.