#1 2005-09-24 19:22:20

nicolas
Former Piwigo Team
2004-12-30
1565

[1.5.0RC1] - Respect de html 4.01

Dans le fichier include/functions_html.inc.php, dans la fonction get_icon il faut changer la deuxième ligne de output:

Avant:

Code:

$output = '<img title="'.$title.'" src="'.$icon_url.'" style="border:0;';
$output.= 'height:'.$size[1].'px;width:'.$size[0].'px" alt=""/>';

Après:

Code:

$output = '<img title="'.$title.'" src="'.$icon_url.'" style="border:0;';
$output.= 'height:'.$size[1].'px;width:'.$size[0].'px" alt="">';

Donnez du peps à vos tags
Laissez vos visiteurs vous aidez à tagger vos images avec user_tags

Hors ligne

#2 2005-09-24 21:37:21

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12673

Re: [1.5.0RC1] - Respect de html 4.01

Est-ce que la norme HTML 4.01 dit clairement qu'il ne faut pas fermer les balises sans contenu ? Si oui, alors je vire l'autofermeture, sinon, je la laisse car elle n'empêchera pas la création d'un template en XHTML.


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#3 2005-09-24 21:46:28

volcom
Former Piwigo Team
2005-01-24
489

Re: [1.5.0RC1] - Respect de html 4.01

Je pense qu'il vaudrait mieux laisser les balises fermer...  même celle du type IMG ou INPUT.

Hors ligne

#4 2005-09-25 09:22:03

nicolas
Former Piwigo Team
2004-12-30
1565

Re: [1.5.0RC1] - Respect de html 4.01

z0rglub a écrit:

Est-ce que la norme HTML 4.01 dit clairement qu'il ne faut pas fermer les balises sans contenu ? Si oui, alors je vire l'autofermeture, sinon, je la laisse car elle n'empêchera pas la création d'un template en XHTML.

Je n'ai pas vu quoi que ce soit disant que les balises sans contenu ne devaient pas être fermées. Le validateur ne se plaint pas. Je comprends la problématique de pourvoir générer du html ou du xhtml. Je ne pensais que c'était volontaire; je pensais que c'était un oubli lors du passage de xhtml vers html 4.01 strict.

En revanche, j'ai trouvé un autre soucis: lorsqu'on a des catégories de deuxième niveau la liste n'est pas générée correctement.
(J'ai supprimé les liens dans un soucis de lisibilité)
Ce qui est généré:

Code:

<dl>
   <dt>Catégories</dt>
   <dd>
      <ul>
         <li>cat 1</li>
         <li>cat 2</li>
         <ul>
            <li>cat 2.1</li>
            <li>cat 2.2</li>
         </ul>
      </ul>
    </dd>
</dl>

Ce qui devrait être généré:

Code:

<dl>
   <dt>Catégories</dt>
   <dd>
      <ul>
         <li>cat 1</li>
         <li>cat 2      
            <ul>
               <li>cat 2.1</li>
               <li>cat 2.2</li>
            </ul>
          </li>
      </ul>
    </dd>
</dl>

J'avais fait un patch dans la version 1.4.1; je regarde si je peux encore l'appliquer.


Donnez du peps à vos tags
Laissez vos visiteurs vous aidez à tagger vos images avec user_tags

Hors ligne

#5 2005-09-25 20:32:15

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12673

Re: [1.5.0RC1] - Respect de html 4.01

Merci, je viens de corriger ce problème. Il sera livré en 1.5.0RC2 :-)


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#6 2005-09-28 21:03:41

nicolas
Former Piwigo Team
2004-12-30
1565

Re: [1.5.0RC1] - Respect de html 4.01

z0rglub a écrit:

Merci, je viens de corriger ce problème. Il sera livré en 1.5.0RC2 :-)

Juste par curiosité tu l'as corrigé comment ?


Donnez du peps à vos tags
Laissez vos visiteurs vous aidez à tagger vos images avec user_tags

Hors ligne

#7 2005-09-28 22:01:06

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12673

Re: [1.5.0RC1] - Respect de html 4.01

nicolas a écrit:

z0rglub a écrit:

Merci, je viens de corriger ce problème. Il sera livré en 1.5.0RC2 :-)

Juste par curiosité tu l'as corrigé comment ?

Code:

/**
 * returns the HTML code for a category item in the menu (for category.php)
 *
 * HTML code generated uses logical list tags ul and each category is an
 * item li. The paramter given is the category informations as an array,
 * used keys are : id, name, nb_images, date_last
 *
 * @param array categories
 * @return string
 */
function get_html_menu_category($categories)
{
  global $page, $lang;

  $ref_level = 0;
  $menu = '';
  
  foreach ($categories as $category)
  {
    $level = substr_count($category['global_rank'], '.') + 1;
    if ($level > $ref_level)
    {
      $menu.= '
             <ul>';
    }
    else if ($level == $ref_level)
    {
      $menu.= '
             </li>';
    }
    else if ($level < $ref_level)
    {
      // we may have to close more than one level at the same time...
      $menu.= str_repeat("\n</ul>",($ref_level-$level));
      $menu.= "\n</li>";
    }
    $ref_level = $level;

    $menu.= '

           <li';
    if (isset($page['cat'])
        and is_numeric($page['cat'])
        and $category['id'] == $page['cat'])
    {
      $menu.= ' class="selected"';
    }
    $menu.= '>';
  
    $url = add_session_id(PHPWG_ROOT_PATH.'category.php?cat='.$category['id']);
    $menu.= '
             <a href="'.$url.'">'.$category['name'].'</a>';

    if ($category['nb_images'] > 0)
    {
      $menu.= '
             <span class="menuInfoCat"';
      $menu.= ' title="'.$category['nb_images'];
      $menu.= ' '.$lang['images_available'].'">';
      $menu.= '['.$category['nb_images'].']';
      $menu.= '</span>';
      $menu.= get_icon($category['date_last']);
    }
  }
  
  $menu.= '</li>
             </ul>';
  
  return $menu;
}

Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

Pied de page des forums

Propulsé par FluxBB

github twitter newsletter Faire un don Piwigo.org © 2002-2024 · Contact