Changeset 5005

Timestamp:

08/18/2011 10:29:48 PM (14 years ago)

Author:

johnjamesjacoby

Message:

Add some responsiveness to bp-default. See #3494.

Location:

trunk/bp-themes/bp-default

Files:

• _inc/css/default.css (modified) (5 diffs)
• _inc/css/responsive.css (added)
• functions.php (modified) (4 diffs)

trunk/bp-themes/bp-default/_inc/css/default.css

@@ -73 +73 @@
     font-family: Arial, Tahoma, Verdana, sans-serif;
     line-height: 170%;
-    min-width: 960px;
     max-width: 1250px;
     margin: 0 auto;
-    width: 90%;
+    width: 95%;
 }
 .padder {
@@ -152 +151 @@
     border-bottom-right-radius: 6px;
     color: #fff;
-    height: 100px;
+    height: auto;
     margin-bottom: 20px;
     padding-top: 25px;
@@ -165 +164 @@
 #header #search-bar .padder {
     padding: 10px 15px 0 20px;
+    height: auto;
 }
 #header #search-bar input[type=text] {
@@ -504 +504 @@
     outline: 1px dotted #ccc;
 }
+#navigation {
+    position: relative;
+    text-align: right;
+    width: 100%;
+}
 #nav {
-    margin: 0;
+    margin: 40px 20px 0 0;
     list-style: none;
-    max-width: 95%;
     padding: 0;
-    position: absolute;
-    right: 15px;
+    position: relative;
+    right: auto;
     bottom: 0;
 }
@@ -519 +523 @@
     padding: 5px 15px;
     text-decoration: none;
+    text-align: left;
 }
 #nav li {
     margin-left: 5px;
-    float: left;
+    display: inline-block;
 }
 #nav li li {

trunk/bp-themes/bp-default/functions.php

@@ -73 +73 @@
     add_theme_support( 'automatic-feed-links' );
 
+    // Add responsive layout support to bp-default without forcing child
+    // themes to inherit it if they don't want to
+    add_theme_support( 'bp-default-responsive' );
+
     // This theme uses wp_nav_menu() in one location.
     register_nav_menus( array(
@@ -87 +91 @@
         // The height and width of your custom header. You can hook into the theme's own filters to change these values.
         // Add a filter to bp_dtheme_header_image_width and bp_dtheme_header_image_height to change these values.
-        define( 'HEADER_IMAGE_WIDTH', apply_filters( 'bp_dtheme_header_image_width', 1250 ) );
-        define( 'HEADER_IMAGE_HEIGHT', apply_filters( 'bp_dtheme_header_image_height', 125 ) );
+        define( 'HEADER_IMAGE_WIDTH',  apply_filters( 'bp_dtheme_header_image_width',  1250 ) );
+        define( 'HEADER_IMAGE_HEIGHT', apply_filters( 'bp_dtheme_header_image_height', 125  ) );
 
         // We'll be using post thumbnails for custom header images on posts and pages. We want them to be 1250 pixels wide by 125 pixels tall.
@@ -136 +140 @@
 function bp_dtheme_enqueue_scripts() {
     // Bump this when changes are made to bust cache
-    $version = '20110804';
+    $version = '20110818';
 
     // Enqueue the global JS - Ajax will not work without it
@@ -167 +171 @@
 function bp_dtheme_enqueue_styles() {
     // Bump this when changes are made to bust cache
-    $version = '20110804';
+    $version = '20110818';
 
     // Default CSS
     wp_enqueue_style( 'bp-default-main', get_template_directory_uri() . '/_inc/css/default.css', array(), $version );
 
+    // Responsive Layout
+    if ( current_theme_supports( 'bp-default-responsive' ) ) {
+        wp_enqueue_style( 'bp-default-responsive', get_template_directory_uri() . '/_inc/css/responsive.css', array( 'bp-default-main' ), $version );
+    }
+
     // Right to left CSS
-    if ( is_rtl() )
+    if ( is_rtl() ) {
         wp_enqueue_style( 'bp-default-main-rtl',  get_template_directory_uri() . '/_inc/css/default-rtl.css', array( 'bp-default-main' ), $version );
+    }
 }
 add_action( 'wp_print_styles', 'bp_dtheme_enqueue_styles' );