Merge pull request #161 from Ammarpad/patch-1

Use hook to add body attributes

includes/Hooks.php

@@ -924,5 +924,13 @@ class TweekiHooks {
 		}
 		return true;
 	}
+	
+	public static function onOutputPageBodyAttributes( $out, $sk, &$bodyAttrs ) {
+		if ( isset( $bodyAttrs['class'] ) && strlen( $bodyAttrs['class'] ) > 0 ) {
+			$bodyAttrs['class'] .= ' ' . implode( ' ', SkinTweeki::$bodyClasses );
+		} else {
+			$bodyAttrs['class'] = implode( ' ', SkinTweeki::$bodyClasses );
+		}
+	}
 
 }

includes/SkinTweeki.php

@@ -42,7 +42,7 @@ class SkinTweeki extends SkinTemplate {
 			->makeConfig( 'tweeki' );
 	}
 
-	protected static $bodyClasses = array( 'tweeki-animateLayout' );
+	public static $bodyClasses = array( 'tweeki-animateLayout' );
 
 
 	/**
@@ -87,21 +87,7 @@ class SkinTweeki extends SkinTemplate {
 		Hooks::run( 'SkinTweekiStyleModules', array( $this, &$styles ) );
 		$out->addModuleStyles( $styles );
 	}
-
-	/**
-	 * Adds classes to the body element.
-	 *
-	 * @param $out OutputPage object
-	 * @param &$bodyAttrs Array of attributes that will be set on the body element
-	 */
-	function addToBodyAttributes( $out, &$bodyAttrs ) {
-		if ( isset( $bodyAttrs['class'] ) && strlen( $bodyAttrs['class'] ) > 0 ) {
-			$bodyAttrs['class'] .= ' ' . implode( ' ', static::$bodyClasses );
-		} else {
-			$bodyAttrs['class'] = implode( ' ', static::$bodyClasses );
-		}
-	}
-
+	
 	/**
 	 * Override to pass our Config instance to it
 	 * @param string $classname

skin.json

@@ -37,7 +37,8 @@
 		"MagicWordMagicWords": "TweekiHooks::onMagicWordMagicWords",
 		"MagicWordwgVariableIDs": "TweekiHooks::onMagicWordwgVariableIDs",
 		"InternalParseBeforeLinks": "TweekiHooks::onInternalParseBeforeLinks",
-		"OutputPageBeforeHTML": "TweekiHooks::onOutputPageBeforeHTML"
+		"OutputPageBeforeHTML": "TweekiHooks::onOutputPageBeforeHTML",
+		"OutputPageBodyAttributes": "TweekiHooks::onOutputPageBodyAttributes"
 	},
 	"ResourceModules": {
 		"skins.tweeki.bootstrap.styles": {