Update jQuery to add +/- signs to mobile menu. Fixes #33.

Add support for retina logos

functions.php

@@ -49,7 +49,7 @@ function trestle_theme_setup() {
 	// Child theme definitions (do not remove).
 	define( 'TRESTLE_THEME_NAME', 'Trestle' );
 	define( 'TRESTLE_THEME_URL', 'http://demo.mightyminnow.com/theme/trestle/' );
-	define( 'TRESTLE_THEME_VERSION', '2.0.1' );
+	define( 'TRESTLE_THEME_VERSION', '2.1.0' );
 	define( 'TRESTLE_SETTINGS_FIELD', 'trestle-settings' );
 
 	// Setup default theme settings.

includes/admin/admin.php

@@ -19,8 +19,8 @@ function trestle_settings_defaults() {
 	// Trestle default key/value pairs.
 	$trestle_defaults = array(
 		'layout'				=> 'solid',
-		'logo_url'				=> '',
+		'logo_id'				=> '',
-		'logo_url_mobile'		=> '',
+		'logo_id_mobile'		=> '',
 		'favicon_url'			=> '',
 		'nav_primary_location'	=> 'full',
 		'mobile_nav_toggle'		=> 'small-icon',
@@ -210,6 +210,12 @@ function trestle_register_required_plugins() {
 			'required' 	=> false,
 		),
 
+		array(
+			'name' 		=> 'RICG Responsive Images',
+			'slug' 		=> 'ricg-responsive-images',
+			'required' 	=> false,
+		),
+
 		array(
 			'name' 		=> 'Widget Context',
 			'slug' 		=> 'widget-context',

includes/admin/customizer.php

@@ -55,44 +55,46 @@ function trestle_customizer_controls( $wp_customize ) {
 
 	// Upload a logo.
 	$wp_customize->add_setting(
-		'trestle-settings[logo_url]',
+		'trestle-settings[logo_id]',
 		array(
-			'default'     		=> trestle_get_option( 'logo_url' ),
+			'default'           => trestle_get_option( 'logo_id' ),
 			'type'              => 'option',
 			'capability'        => 'edit_theme_options',
-			'sanitize_callback'	=> 'esc_url_raw',
+			'sanitize_callback' => 'absint',
 		)
 	);
 	$wp_customize->add_control(
-		new WP_Customize_Image_Control(
+		new WP_Customize_Media_Control(
 			$wp_customize,
 			'trestle_logo_control',
 			array(
 				'label'     => __( 'Upload a logo', 'trestle' ),
 				'section'   => 'trestle_settings_section',
-				'settings'   => 'trestle-settings[logo_url]',
+				'settings'  => 'trestle-settings[logo_id]',
+				'mime_type' => 'image',
 			)
 		)
 	);
 
 	// Upload a mobile logo.
 	$wp_customize->add_setting(
-		'trestle-settings[logo_url_mobile]',
+		'trestle-settings[logo_id_mobile]',
 		array(
-			'default'     		=> trestle_get_option( 'logo_url_mobile' ),
+			'default'           => trestle_get_option( 'logo_id_mobile' ),
 			'type'              => 'option',
 			'capability'        => 'edit_theme_options',
-			'sanitize_callback'	=> 'esc_url_raw',
+			'sanitize_callback' => 'absint',
 		)
 	);
 	$wp_customize->add_control(
-		new WP_Customize_Image_Control(
+		new WP_Customize_Media_Control(
 			$wp_customize,
 			'trestle_mobile_logo_control',
 			array(
 				'label'      => __( 'Upload a mobile logo', 'trestle' ),
 				'section'    => 'trestle_settings_section',
-			   'settings'   => 'trestle-settings[logo_url_mobile]',
+				'settings'   => 'trestle-settings[logo_id_mobile]',
+				'mime_type'  => 'image',
 			)
 		)
 	);
@@ -210,6 +212,7 @@ function trestle_customizer_controls( $wp_customize ) {
 			'default'    => trestle_get_option( 'revisions_number' ),
 			'type'       => 'option',
 			'capability' => 'edit_theme_options',
+			'sanitize_callback' => 'absint',
 		)
 	);
 	$wp_customize->add_control(

includes/functions/theme-functions.php

@@ -216,42 +216,55 @@ add_filter( 'genesis_seo_title', 'trestle_do_logos', 10, 3 );
  */
 function trestle_do_logos( $title, $inside, $wrap ) {
 
-	$logo_url = trestle_get_option( 'logo_url' );
+	$logo_id = trestle_get_option( 'logo_id' );
-	$logo_url_mobile = trestle_get_option( 'logo_url_mobile' );
+	$logo_id_mobile = trestle_get_option( 'logo_id_mobile' );
 	$logo_html = '';
 
 	// Regular logo.
-	if ( $logo_url ) {
+	if ( $logo_id ) {
 
-		// Default logo class.
+		// Default logo classes.
-		$classes = array('logo-full');
+		$classes = array(
+			'logo',
+			'logo-full'
+		);
 
 		// If no mobile logo is specified, make regular logo act as mobile logo too.
-		if( ! $logo_url_mobile )
+		if( ! $logo_id_mobile ) {
 			$classes[] = 'show';
+		}
 
-		$logo_html .= sprintf( '<img class="logo %s" alt="%s" src="%s" />',
+		// Prepare the classes.
-			implode(' ', $classes),
+		$logo_attr = array(
-			esc_attr( get_bloginfo( 'name' ) ),
+			'class'	=> implode( $classes, ' ' ),
-			$logo_url
 		);
+
+		// Build the <img> tag.
+		$logo_html .= wp_get_attachment_image( $logo_id, 'full', false, $logo_attr );
+
 	}
 
 	// Mobile logo.
-	if ( $logo_url_mobile ) {
+	if ( $logo_id_mobile ) {
 
 		// Default mobile logo class.
-		$classes = array('logo-mobile');
+		$classes = array(
+			'logo',
+			'logo-mobile'
+		);
 
 		// If no regular logo is specified, make mobile logo act as regular logo too.
-		if( ! $logo_url )
+		if( ! $logo_id )
 			$classes[] = 'show';
 
-		$logo_html .= sprintf( '<img class="logo %s" alt="%s" src="%s" />',
+		// Prepare the classes.
-			implode(' ', $classes),
+		$logo_attr = array(
-			esc_attr( get_bloginfo( 'name' ) ),
+			'class'	=> implode( $classes, ' ' ),
-			$logo_url_mobile
 		);
+
+		// Build the <img> tag.
+		$logo_html .= wp_get_attachment_image( $logo_id_mobile, 'full', false, $logo_attr );
+
 	}
 
 	if ( $logo_html ) {

includes/js/theme-jquery.js

@@ -107,8 +107,8 @@ jQuery( document ).ready( function( $ ) {
 	var openIcon = '-';
 
 	// Insert the icons into the nav where appropriate.
-	$( '.nav-primary' ).find( '.genesis-nav-menu .parent:not( .current-menu-item, .current_page_item, .current_page_parent, .current_page_ancestor) > a' ).after( '<a class="sub-icon" href="#">' + closedIcon + '</a>' );
+	$( '.nav-primary' ).find( '.genesis-nav-menu .parent:not( [class*="current"] ) > a' ).after( '<a class="sub-icon" href="#">' + closedIcon + '</a>' );
-	$( '.nav-primary' ).find( '.genesis-nav-menu .parent.current-menu-item > a, .genesis-nav-menu .parent.current_page_item > a, .genesis-nav-menu .parent.current_page_parent > a, .genesis-nav-menu .parent.current_page_ancestor > a' ).after( '<a class="sub-icon" href="#">' + openIcon + '</a>' );
+    $( '.nav-primary' ).find( '.genesis-nav-menu .parent[class*="current"] > a' ).after( '<a class="sub-icon" href="#">' + openIcon + '</a>' );
 
 	// Mobile navigation expand/contract functionality.
 	$( '.sub-icon' ).click( function( event ) {