[RÉGLÉ] spécifiez dimensions de l'image

[michel68100]

Lors d'un test avec gtmetrix il ne demande de spécifiez dimensions de l'image c'est les images du lien que j'ai inséré avec les menus de l'administration


ou je dois spécifiez dimensions de l'image mais je sais pas ou, je sais avec les balises width et height mais pas de trace sur les fichiers contenant les images du lien ou sont il ranger. Voir pièce jointe.

[SimonG]

Re : spécifiez dimensions de l'image

Bonjour,

je ne pense pas qu'il soit possible (simplement) de renseigner la taille de ces images.
Ceci dit, en regardant les indications de GTMetrix, tu te rendras compte, à mon avis, que le fait de renseigner ces tailles ne fait pas partie des conseils ayant le plus d'importance (et donc de répercussion sur le temps de chargement des pages).

[ManuelVoileux]

Re : spécifiez dimensions de l'image

je ne pense pas qu'il soit possible (simplement) de renseigner la taille de ces images.

Bonjour,
si tu utilises JCE comme éditeur, tu peux lui demander de spécifier sytématiquement les dimensions des images intégrées.
voir image jointe (menu backend Composants/JCE administration/Profils JCE/Default/Paramètres des plugin/gestionnaire d'images)

[jisse03]

Re : spécifiez dimensions de l'image

Bonjour,

Il est possible, assez simplement, de préciser la taille des images en surchargeant les divers tmpl de mod_menu: default_component.php, default_separator.php et default_url.php.
Tous ont ce code:

if ($item->menu_image)
    {
        $item->params->get('menu_text', 1) ?
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" /><span class="image-title">'.$item->title.'</span> ' :
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" />';
}
else { $linktype = $item->title;
} 

Pour avoir la taille des images il suffit de modifier ainsi:

if ($item->menu_image)
    {
        $dims = getimagesize( $item->menu_image );
        $item->params->get('menu_text', 1) ?
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" width="'.$dims[0].'" height="'.$dims[1].'" /><span class="image-title">'.$item->title.'</span> ' :
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title..'" width="'.$dims[0].'" height="'.$dims[1]'" />';
}
else { $linktype = $item->title;
} 

[jisse03]

Re : spécifiez dimensions de l'image

si tu utilises JCE comme éditeur, tu peux lui demander de spécifier sytématiquement les dimensions des images intégrées.
voir image jointe (menu backend Composants/JCE administration/Profils JCE/Default/Paramètres des plugin/gestionnaire d'images)

Valide pour les contenus, mais pas pour les menus iconiques

[michel68100]

Re : spécifiez dimensions de l'image

Bonjour jisse
Si je modifie le code j'ai
Parse error : syntax error, T_VARIABLE inattendue / home2/wb67267/modules/mod_menu/tmpl/default_component.php sur la ligne 16

voici la modification que j'ai apporter par tes soins:

<?php
/**
 * @package        Joomla.Site
 * @subpackage    mod_menu
 * @copyright    Copyright (C) 2005 - 2013 Open Source Matters, Inc. All rights reserved.
 * @license        GNU General Public License version 2 or later; see LICENSE.txt
 */

// No direct access.
defined('_JEXEC') or die;

// Note. It is important to remove spaces between elements.
$class = $item->anchor_css ? 'class="'.$item->anchor_css.'" ' : '';
$title = $item->anchor_title ? 'title="'.$item->anchor_title.'" ' : '';
if ($item->menu_image) {
        $dims = getimagesize( $item->menu_image ); 
        $item->params->get('menu_text', 1) ? 
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" width="'.$dims[0].'" height="'.$dims[1].'" /><span class="image-title">'.$item->title.'</span> ' : 
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title..'" width="'.$dims[0].'" height="'.$dims[1]'" />'; 
}
else { $linktype = $item->title;
}

switch ($item->browserNav) :
    default:
    case 0:
?><a <?php echo $class; ?>href="<?php echo $item->flink; ?>" <?php echo $title; ?>><?php echo $linktype; ?></a><?php
        break;
    case 1:
        // _blank
?><a <?php echo $class; ?>href="<?php echo $item->flink; ?>" target="_blank" <?php echo $title; ?>><?php echo $linktype; ?></a><?php
        break;
    case 2:
    // window.open
?><a <?php echo $class; ?>href="<?php echo $item->flink; ?>" onclick="window.open(this.href,'targetWindow','toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes');return false;" <?php echo $title; ?>><?php echo $linktype; ?></a>
<?php
        break;
endswitch;

ou ce situe l'erreur

[lomart]

Re : spécifiez dimensions de l'image

Il ne faut pas recopier bêtement les fautes de frappe de Jisse
les 2 points après title dans $linktype= ...

[jisse03]

Re : spécifiez dimensions de l'image

Faute de frappe en mode "je tape trop vite sans tester le code localement"...
version débuguée:

if ($item->menu_image) {
        $dims = getimagesize( $item->menu_image );
        $item->params->get('menu_text', 1) ?
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" width="'.$dims[0].'" height="'.$dims[1].'" /><span class="image-title">'.$item->title.'</span> ' :
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" width="'.$dims[0].'" height="'.$dims[1].'" />';

} 

et cette fois testé sur un Joomla local.

[michel68100]

Re : spécifiez dimensions de l'image

Désoler mais maintenant c'est une erreur:
Parse error : syntax error, T_STRING inattendue / home2/wb67267/modules/mod_menu/tmpl/default_component.php sur la ligne 15

<?php
/**
 * @package        Joomla.Site
 * @subpackage    mod_menu
 * @copyright    Copyright (C) 2005 - 2013 Open Source Matters, Inc. All rights reserved.
 * @license        GNU General Public License version 2 or later; see LICENSE.txt
 */

// No direct access.
defined('_JEXEC') or die;

// Note. It is important to remove spaces between elements.
$class = $item->anchor_css ? 'class="'.$item->anchor_css.'" ' : '';
$title = $item->anchor_title ? 'title="'.$item->anchor_title.'" ' : '';
if ($item->menu_image) { 
        $dims = getimagesize( $item->menu_image ); 
        $item->params->get('menu_text', 1) ? 
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" width="'.$dims[0].'" height="'.$dims[1].'" /><span class="image-title">'.$item->title.'</span> ' : 
        $linktype = '<img src="'.$item->menu_image.'" alt="'.$item->title.'" width="'.$dims[0].'" height="'.$dims[1].'" />'; 

}  

else { $linktype = $item->title;
}

switch ($item->browserNav) :
    default:
    case 0:
?><a <?php echo $class; ?>href="<?php echo $item->flink; ?>" <?php echo $title; ?>><?php echo $linktype; ?></a><?php
        break;
    case 1:
        // _blank
?><a <?php echo $class; ?>href="<?php echo $item->flink; ?>" target="_blank" <?php echo $title; ?>><?php echo $linktype; ?></a><?php
        break;
    case 2:
    // window.open
?><a <?php echo $class; ?>href="<?php echo $item->flink; ?>" onclick="window.open(this.href,'targetWindow','toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes');return false;" <?php echo $title; ?>><?php echo $linktype; ?></a>
<?php
        break;
endswitch;

[jisse03]

Re : spécifiez dimensions de l'image

Si je compte bien, ton erreur est sur

if ($item->menu_image) { 

et en comptant plus mal

$dims = getimagesize( $item->menu_image ); 

Dans les 2 cas, il n'y a pas de T_STRING...

Bizarre, puisque sur mon site de test cette modif fonctionne bien (bon après avoir rectifié les fautes d efrappe de la première mouture non testée sur un site )

[michel68100]

Re : spécifiez dimensions de l'image

Bien compter mais je vois pas ou mettre les ' ou le / dans cette phrase

[jisse03]

Re : spécifiez dimensions de l'image

Ce que je ne vois pas est où est ton erreur, le code fonctionne bien sur un site de test, et génère du HTML avec les dimensions de l'image.

[michel68100]

Re : spécifiez dimensions de l'image

Salut jisse

Je viens de refaire un test avec ton code et tout est rentré dans l'ordre à rien comprendre.

Merci du coup de mains.