Changeset 21668

Timestamp:

08/30/2012 04:59:22 PM (12 years ago)

Author:

lancewillett

Message:

Twenty Twelve: move Open Sans custom font loading to a body class to allow easier overriding, fixes #21694.

• Use wp_style_is() to check for the enqueued CSS file before adding the body class value.
• Add a sample dequeue method in the comments as documentation for child themers and their ilk.

Props viniciusmassuchetto, nacin, and obenland.

Location:

trunk/wp-content/themes/twentytwelve

Files:

• functions.php (modified) (2 diffs)
• style.css (modified) (1 diff)

trunk/wp-content/themes/twentytwelve/functions.php

@@ -103 +103 @@
     /*
      * Loads our special font CSS file.
+     *
+     * To disable in a child theme, use wp_dequeue_style()
+     * function mytheme_dequeue_fonts() {
+     *     wp_dequeue_style( 'twentytwelve-fonts' );
+     * }
+     * add_action( 'wp_enqueue_scripts', 'mytheme_dequeue_fonts', 11 );
      */
     $protocol = is_ssl() ? 'https' : 'http';
@@ -377 +383 @@
         $classes[] = 'custom-background-white';
 
+    // Enable custom font class only if the font CSS is queued to load.
+    if ( wp_style_is( 'twentytwelve-fonts', 'queue' ) )
+        $classes[] = 'custom-font-enabled';
+
     return $classes;
 }

trunk/wp-content/themes/twentytwelve/style.css

@@ -438 +438 @@
     font-size: 14px;
     font-size: 1rem;
-    font-family: "Open Sans", Helvetica, Arial, sans-serif;
+    font-family: Helvetica, Arial, sans-serif;
     text-rendering: optimizeLegibility;
     color: #444;
+}
+body.custom-font-enabled {
+    font-family: "Open Sans", Helvetica, Arial, sans-serif;
 }
 a {