web-dev-qa-db-fra.com

Revue de thème: vignette de publication, image d'en-tête, largeur du contenu

Un suivi de ma question précédente à propos de mon thème qui a été rejeté pour le répertoire de thèmes WordPress.org :

  • RECOMMANDÉ: Aucune référence à the_post_thumbnail () n’a été trouvée dans le thème. Il est recommandé que le thème implémente cette fonctionnalité au lieu d'utiliser des champs personnalisés pour les vignettes.
  • RECOMMANDÉ: Aucune référence à add_custom_image_header n'a été trouvée dans le thème. Il est recommandé que le thème implémente cette fonctionnalité si une image est utilisée pour l'en-tête.

Comment puis-je le corriger? Ceci est ma fonction:

// thumbnail list 
function retImage($content){
     $pattern="/<img.*?src=[\'|\"](.*?(?:[\.gif|\.jpg]))[\'|\"].*?[\/]?>/";
     preg_match_all($pattern,$content,$match); 
     if(empty($match[0][0])){
                 echo "<img src=\"";
                 bloginfo('template_url');
                 echo "/images/thumbnail.png\" />";
     }else{
         echo  $match[0][0];
     }
}

J'ai utilisé une image pour l'en-tête par css, pourquoi disent-ils ceci:

  • RECOMMANDÉ: Aucune référence à add_custom_image_header n'a été trouvée dans le thème.

Header image CSS rules

  • Les vidéos incorporées se chevauchent. Veuillez définir la variable content_width

Dans mon functions.php de mon thème, j'ai fait ceci:

if ( ! isset( $content_width ) )
    $content_width = 992;

Pourquoi me donnent-ils toujours ce pourboire?

theme-developmentwordpress.orgtheme-review
3
enjoylife

RECOMMANDÉ: Aucune référence à the_post_thumbnail () n’a été trouvée dans le thème. Il est recommandé que le thème implémente cette fonctionnalité au lieu d'utiliser des champs personnalisés pour les vignettes.

En effet, vous n'utilisez pas the_post_thumbnail() dans votre thème, vous essayez d'obtenir une image à partir du contenu du message. Cela signifie qu’il n’ya aucun moyen pour un utilisateur de définir explicitement quelle image est affichée où que vous utilisiez retImage(). J'inclurais la possibilité pour retImage() d'essayer d'utiliser la vignette, par exemple:

// thumbnail list 
function retImage($content) {

    if( has_post_thumbnail() )
         return the_post_thumbnail( 'thumbnail' ); 

    $pattern="/<img.*?src=[\'|\"](.*?(?:[\.gif|\.jpg]))[\'|\"].*?[\/]?>/";
    preg_match_all($pattern,$content,$match); 
    if(empty($match[0][0])){
             echo "<img src=\"";
             bloginfo('template_url');
             echo "/images/thumbnail.png\" />";
    } else {

        echo  $match[0][0];
    }
}

Vous devrez peut-être également inclure add_theme_support( 'post-thumbnails' ) dans votre functions.php

RECOMMANDÉ: Aucune référence à add_custom_image_header n'a été trouvée dans le thème. Il est recommandé que le thème implémente cette fonctionnalité si une image est utilisée pour l'en-tête.

Si votre thème a une image d'en-tête, il est recommandé d'utiliser l'API d'image d'en-tête de WordPress. Pour en savoir plus, cliquez ici: http://codex.wordpress.org/Function_Reference/add_custom_image_header

Cela permettrait à l'utilisateur de changer l'image d'en-tête via l'admin (Apparence -> En-tête personnalisé) ou quelque chose de similaire.

Les vidéos incorporées se chevauchent. Veuillez définir la variable content_width Le $content_width doit être une variable globale:

global $content_width;
$content_width = 960;
5
Joe Hoyle