Fonts need to be disabled to work with theme-defined fonts? [#1449982]

Just upgraded from 7-1.x and found all my TypeKit fonts missing. I went into the settings and found the new fonts[] directives that you're supposed to add to your theme to make fonts load without using the CSS selectors in the UI (I use my theme's CSS for this). So I added those – still nothing.

A look at the code revealed that fontyourface_add_theme_info_fonts() wouldn't add my fonts to fontyourface_font_registry() – unless they're disabled on the Browse tab. Is that the intended behavior or is this a bug?

I'm guessing that fontyourface_font_registry($font); should be moved below the if statement that's currently surrounding it (line 65-ish).

Apologies if this has already been reported – couldn't find a similar issue.

Comments

Comment #1

sreynen CreditAttribution: sreynen commented 22 February 2012 at 13:23

Assigned:

Unassigned

» sreynen

Yep, that's a bug, introduced in #1444708: Fonts enabled in theme .info are never disabled. Previously, fonts in the .info were enabled automatically if they weren't already enabled. Now they're added to the registry if they're not already enabled. They should just be added to the registry whether or not they're enabled.