The new function automatically echos, so the templates that call the function need to be updated!

Something like:

div.entry-content img {margin: 0 24px 12px 24px;}
div.entry-content img.alignleft {display: inline; margin-left: 0; margin-right: 24px;}
div.entry-content img.alignright {display: inline; margin-left: 24px; margin-right: 0;}
div.entry-content img.aligncenter {clear: both; display: block; margin-left: auto; margin-right: auto;}

These may be present (or close to it) lower in the style.css under the /* LAYOUT */ comment, but could be improved.

Try throwing this in the footer above the rest of the scripts.

<script>window.jQuery || document.write('<script src="js/vendor/jquery-1.11.0.min.js"><\/script>')</script>

Need to update the meta description to have a fallback if the page does not have an excerpt or content.

Also remove functions that we won't be needing.

Check if it is active, else display Wordpress core next/previous page links.

Not a high priority because WP Page-Navi will not spit out an error if it is not active.

Should the Nebula template use __(...) and _e(...) for translations? All of our sites are only in English, so I think we could speed development up in general if we remove language scripts, but it's also not necessarily hurting anything.

Currently, only hard-coded text is wrapped in these scripts, so even if we wanted to localize text, we'd have some major work to do and the tags in question here are the tip of the iceberg.

Any thoughts?

Allow a post ID to be passed to nebula_the_excerpt so it can be used outside the loop if needed.

Here's something to start:

function nebula_the_excerpt( $more=false, $length=20, $hellip=false, $postID=false ) {
    $the_post = get_post($postID);

    if ( $postID ) { //Find a better way to pull the post_content without formatting. strip_tags() isn't working properly, and can't get_the_excerpt() using post ID...
        //if the post ID has an excerpt
        //else try the post ID content
        $string = $the_post->post_content;
    } else {
        if ( get_the_excerpt() ) {
            $string = strip_tags(get_the_excerpt(), '<p>');
        } else {
            $string = strip_tags(get_the_content(), '<p>');
        }
    }
    $string = string_limit_words($string, $length);
    if ( $hellip ) {
        if ( $string[1] == 1 ) {
            $string[0] .= '&hellip; ';
        }
    }
    if ( isset($more) && $more != '' ) {
        $string[0] .= ' <a class="nebula_the_excerpt" href="' . get_permalink($postID) . '">' . $more . '</a>';
    }
    return $string[0];
}

Additionally, allow a "d" to be used to for any value (except $postID) to tell nebula_the_excerpt() to use the global default value for that parameter. Maybe there is a better way to do this? My goal is to allow a user to designate a Post ID without needing to also designate a $length and $hellip (which would prevent making global changes to their theme more difficult).

Override any footer message set by plugins so the PHG footer message is the only one.

Add an example of proper (and experimental) usage of html5 video as a background element. Possibly use video.js as an optional library.

Add doubletaptogo.js to /js/libs/ and call it (commented out) in the footer.php (can be deferred, place it above main.js).

Since this js file is so small, consider moving it inside main.js as a function. @jefphg: may bounce this off you.

<script src="<?php bloginfo('template_directory');?>/js/libs/doubletaptogo.js" defer></script>

This function will prevent touch-based devices from following an href when the parent link is actually a dropdown menu. Once open, it will allow the link to work on a second click. Call it on the parent element (typically an LI)- this line should work in all cases.

jQuery('#primarynav .menu-item-has-children').doubleTapToGo();

Here is the full function:

;(function(jQuery, window, document, undefined){
    jQuery.fn.doubleTapToGo = function(params){
        if( !('ontouchstart' in window) &&
            !navigator.msMaxTouchPoints &&
            !navigator.userAgent.toLowerCase().match( /windows phone os 7/i ) ) return false;

        this.each(function(){
            var curItem = false;

            jQuery(this).on( 'click', function(e){
                var item = jQuery(this);
                if( item[0] != curItem[0] ){
                    e.preventDefault();
                    curItem = item;
                }
            });

            jQuery( document ).on( 'click touchstart MSPointerDown', function( e ){
                var resetItem = true,
                    parents   = $( e.target ).parents();

                for( var i = 0; i < parents.length; i++ )
                    if( parents[ i ] == curItem[ 0 ] )
                        resetItem = false;

                if( resetItem )
                    curItem = false;
            });
        });
        return this;
    };
})(jQuery, window, document);

The markdown document should link to the Wordpress plugins page for required plugins and optional ones too. Also include links to all helpful documentation for libraries used.

This should also link to the Google Docs testing checklist and go-live checklist.

I'm afraid there might be an issue calling it from inside the loop. I think it might use the post ID of the page it is on rather than the post ID of the queried post.

Add Jef's on click trigger for Gumby modals (with my slight modification) to main.js but also look into the Gumby docs to see if we can eventually not need the mod.

Strip localization and add bigger comments to compartmentalize it better. Re-organize if needed.

Customize the WP Admin Dashboard to include a "banner" (using the Welcome Message) for quick access to useful links.

The screenshot is overkill, but the idea is there. We should also include "Designed and developed by Pinckney Hugo Group" in it with a link back to us.

Returns an object of the viewport dimensions. Could be useful- add an additional call to it in the resize function.

viewport = updateViewportDimensions();

function updateViewportDimensions() {
    var w=window,d=document,e=d.documentElement,g=d.getElementsByTagName('body')[0],x=w.innerWidth||e.clientWidth||g.clientWidth,y=w.innerHeight||e.clientHeight||g.clientHeight;
    return { width:x,height:y }
}

Wrap the post meta in the stylized

Add the author name to the post meta section.

Set the font size to be 12px for post meta (and lighter grey than #444 too).

Add the version number to the right footer next to the PHG phone number (or another elegant way of adding it for our reference).

Retina displays on notebooks are serving the mobile styles.

May be a slow process, but we should consider adding schema.org tags globally to Nebula to have from the get-go.

The big Gumby defaults are nice for super minimal/modern sites, but we should consider smaller defaults.

Send a GA event using login.js when a user enters an incorrect password. Maybe using a setTimeout or something? Use the username and IP as the action and label.

Also track on password resets (intent if we have to).

Lets get the following formats for Open Sans into the /fonts directory so we can pull a local copy in the CSS if needed.

.eot
.woff
.ttf
.svg

Might be useful for sites with blogs to automatically cross-link pages. This just does a query for other posts with similar tags.

function bones_related_posts() {
    echo '<ul id="bones-related-posts">';
    global $post;
    $tags = wp_get_post_tags( $post->ID );
    if($tags) {
        foreach( $tags as $tag ) {
            $tag_arr .= $tag->slug . ',';
        }
        $args = array(
            'tag' => $tag_arr,
            'numberposts' => 5, /* you can change this to show more */
            'post__not_in' => array($post->ID)
        );
        $related_posts = get_posts( $args );
        if($related_posts) {
            foreach ( $related_posts as $post ) : setup_postdata( $post ); ?>
                <li class="related_post"><a class="entry-unrelated" href="<?php the_permalink() ?>" title="<?php the_title_attribute(); ?>"><?php the_title(); ?></a></li>
            <?php endforeach; }
        else { ?>
            <?php echo '<li class="no_related_post">' . __( 'No Related Posts Yet!', 'bonestheme' ) . '</li>'; ?>
        <?php }
    }
    wp_reset_postdata();
    echo '</ul>';
} /* end bones related posts function */

The gumby class is in there and the file naming convention seems correct. Whassup wit dat?

Instead of header.php or styles.css. What are the pros/cons? Also, check to see if gumby.css is loading Open Sans.

function web_fonts() {
  wp_register_style('googleFonts', 'http://fonts.googleapis.com/css?family=Open+Sans:300italic,400italic,600italic,700italic,800italic,400,300,600,700,800');
  wp_enqueue_style( 'googleFonts');
}
add_action('wp_print_styles', 'web_fonts');

Test to see if this line triggers for just IE or if other browsers are affected. Also does this trigger for just IE 10? or does IE 11 fall into this?

<!--[if gt IE 9 ]><html <?php language_attributes(); ?> class="no-js ie ie10 lte-ie10 lte-ie10 lt-ie11"><![endif]-->

Add a quick usage to the readme.

Parse error: syntax error, unexpected ':' in /home3/cblakley/public_html/nebula/wp-content/themes/WP-Nebula-master/loop.php on line 36

Need to fix asap. Can be tested on a search result page easiest.

Instead of the wordpress core older/newer text.

Add instructions for how to use mobile detect in the readme.

Description, and keywords to header.php (non-social metadata)

Double check the size of Facebook sharing thumbnails since they made the design update to the feed. Images should be minimum of 200x200, but let's make sure we use an appropriate size so they don't get scaled up.

Don't want to make them too big either because we should be able to design at the expected size (for readability, etc.).

Determine if it is needed. Test all browsers including IE7+, iOS and mobile devices. Then update header.php on ~line 21

Right now it will return an error description.

Style the core Wordpress widgets with generic styles so they at least won't look drastically different if someone wants to enable them after launching a site.

Add usage to the readme.

Also show how it would work with rgba()

IE actually follows the link, so it needs to be triggered on mobile only.

For example, wrap all event triggers in a function and initialize it at the top of the file. Do this for everything

Additional default event trackers here. Post requests or ideas here.

Window Resizes

//on ready, var is the window size
//after resize complete (using our waitforfinalevent), send event "resize" "from XXXXpx to XXXpx"
//after event, change initial var to end size

Print Intent (clean this up significantly first)

try{
(function() {
var afterPrint = function() {
ga('send', 'event', 'Print Intent', document.location.pathname); //for Universal GA
};
if (window.matchMedia) {
var mediaQueryList = window.matchMedia('print');
mediaQueryList.addListener(function(mql) {
if (!mql.matches)
afterPrint();
});
}
window.onafterprint = afterPrint;
}());
} catch(e) {}

External Links

$("a[rel*='external']").click(function(){ //change to .on('click')
//send event here
});

Currently we're just pulling the post_content from the designated post ID inside nebula_the_excerpt(). This works, but I want to pull the excerpt itself if it exists for that post.

get_the_excerpt() does not accept parameters. Currently I've got:

$the_post = get_post($postID);
$string = $the_post->post_content;

Is there an equivalent to post_content for the excerpt?

Using a conditional, detect "?debug" like we do in the footer now, but inside the false, set $defer and $async as strings. Then, in the footer, we only need to call each script tag once!

if ( array_key_exists('debug', $_GET) ) {
    $defer = '';
    $async = '';
} else {
    $defer = 'defer';
    $async = 'async';
}

Then in the header/footer for script tags (for appropriate ones), call them like this:

<script type='text/javascript' src="<?php bloginfo('template_directory');?>/js/libs/modernizr.custom.42059.js" <?php echo $defer; ?>></script>

Find an elegant way to strip out shortcodes like [caption] from showing up in the excerpt.

For example, when the excerpt is empty, the content is used, but when this has dynamic content like a Youtube player embedded within the first X words of the post, it will be generated on the listing page where nebula_the_excerpt() is used.

Somehow we need to detect any HTML code used in the content and strip it out. Furthermore, we also can't echo the raw code either.

http://wordpress.org/support/topic/changing-from-email-address-for-user-registration

Try to do it as a function.php before relying on a plugin so we can package it and enable/disable easier.

Combine and/or asynchronously load CSS without the need of a Wordpress plugin (like this one).

Save out a less compressed version, and make a retina hi-res version too.

Add a linked featured image to the query post example

The page is the "You do not have sufficient permissions to access this page." error page.

Access it by logging in as a subscriber and accessing this link:
http://gearside.com/nebula/wp-admin/plugin-editor.php

Maybe using functions.php?