Changeset 71544 in spip-zone

Timestamp:

Apr 3, 2013, 9:40:26 PM (8 years ago)

Author:

cedric@…

Message:

On favorise la semantique des balises : <code> pour code inline et <pre> pour un bloc de code, <p> pour le lien de telechargement
Cela permet de beneficier des stylages natifs de ces balises quand ils sont là et d'avoir des blocs qui se tiennent et ont une apparence correcte avec une base CSS complète, même sans stylage spécifique pour le plugin.

Corrolairement on laisse tomber les .spip_code et .spip_cadre qui generent plus de confusion qu'autre chose (et ne s'emboitent pas bien par rapport au pre, ajoutant cadre au cadre)

La css privée est enrichie des fioritures qui rendent le tout plus joli, et on les propose dans la css publique aussi, pour le cas ou elle servirait.

Voir si ça ne casse pas trop les sites qui utilisent le plugin.

Location:

_plugins_/coloration_code/trunk

Files:

• coloration_code_fonctions.php (modified) (4 diffs)
• css/coloration_code.css (modified) (1 diff)
• paquet.xml (modified) (1 diff)
• plugin.xml (modified) (1 diff)
• prive/themes/spip/coloration_code.css (modified) (1 diff)

_plugins_/coloration_code/trunk/coloration_code_fonctions.php

@@ -52 +52 @@
         if ($cadre == 'cadre')
                 $englobant = 'div';
+        $balise_code = ($englobant == 'div' ? "div":"code");
 
         // Supprime le premier et le dernier retour chariot
@@ -77 +78 @@
         $geshi->set_overall_style('');
         $geshi->set_code_style('');
-        
+
         $stylecss = "";
         if (!PLUGIN_COLORATION_CODE_STYLES_INLINE OR PLUGIN_COLORATION_CODE_SANS_STYLES) {
@@ -120 +121 @@
                 $datatext_content = ' data-clipboard-text="'.attribut_html($code).'"';
                 
-        if ($cadre == 'cadre') {
-          $spip_cadre = ' spip_cadre';
-          $geshi->set_header_type(GESHI_HEADER_DIV);
+        if ($cadre == 'cadre' OR $englobant=="div") {
+          $geshi->set_header_type(GESHI_HEADER_PRE);
           $geshi->enable_line_numbers(GESHI_NORMAL_LINE_NUMBERS);
         } else {
-          $spip_cadre = '';
           $geshi->set_header_type(GESHI_HEADER_NONE);
           $geshi->enable_line_numbers(GESHI_NO_LINE_NUMBERS);
@@ -133 +132 @@
         // And echo the result!
         //
-        $rempl = $stylecss . '<' . $englobant . ' class="coloration_code"><' . $englobant . ' class="spip_'.$language.' '.$cadre.$spip_cadre.'"'.$datatext_content.'>'.$geshi->parse_code().'</' . $englobant . '>';
+        $rempl = $stylecss . '<' . $englobant . ' class="coloration_code '.$cadre.'"><' . $balise_code . ' class="spip_'.$language.' '.$cadre.'"'.$datatext_content.'>'.$geshi->parse_code().'</' . $balise_code . '>';
 
         if ($telecharge) {
-                $rempl .= "<div class='" . $cadre . "_download'><a href='$fichier'>"._T('bouton_download')."</a></div>";
+                $rempl .= "<p class='download " . $cadre . "_download'><a href='$fichier'>"._T('bouton_download')."</a></p>";
         }
         return $rempl.'</' . $englobant . '>';

_plugins_/coloration_code/trunk/css/coloration_code.css

@@ -1 @@
-.cadre_download{
-        text-align:right;
+.coloration_code pre {
+  background: #eee;
+  padding: 0;
+  -webkit-box-shadow: inset 40px 0 0 #f7f7f7, inset 41px 0 0 #ececf0;
+     -moz-box-shadow: inset 40px 0 0 #f7f7f7, inset 41px 0 0 #ececf0;
+          box-shadow: inset 40px 0 0 #f7f7f7, inset 41px 0 0 #ececf0;
+  white-space: pre-wrap;
+  word-break: break-all;
+  word-wrap: break-word;
+  padding:1em 0;
 }
+.coloration_code .download {text-align: right}
 
-.cadre_download a{
-        font-family: verdana, arial, sans; font-weight: bold; font-style: normal;
+.coloration_code ol{
+        list-style-type: decimal;
+    margin-left:43px;
 }

_plugins_/coloration_code/trunk/paquet.xml

@@ -2 +2 @@
         prefix="coloration_code"
         categorie="edition"
-        version="0.7.2"
+        version="0.8.0"
         etat="stable"
         compatibilite="[2.0.0;3.0.99]"

_plugins_/coloration_code/trunk/plugin.xml

@@ -16 +16 @@
         <icon>coloration-32.png</icon>
         <auteur>Pierre Andrews (Mortimer) - ARNO*</auteur>
-        <version>0.7.2</version>
+        <version>0.8.0</version>
         <etat>stable</etat>
         <description>

_plugins_/coloration_code/trunk/prive/themes/spip/coloration_code.css

@@ -1 @@
-.coloration_code .spip_cadre{
-        margin-left:2em;        
+.coloration_code.cadre {
+
 }
+
+.coloration_code pre {
+  background: #eee;
+  padding: 0;
+  -webkit-box-shadow: inset 40px 0 0 #f7f7f7, inset 41px 0 0 #ececf0;
+     -moz-box-shadow: inset 40px 0 0 #f7f7f7, inset 41px 0 0 #ececf0;
+          box-shadow: inset 40px 0 0 #f7f7f7, inset 41px 0 0 #ececf0;
+  white-space: pre-wrap;
+  word-break: break-all;
+  word-wrap: break-word;
+  padding:1em 0;
+}
+.coloration_code .download {text-align: right}
 
 .coloration_code ol{
         list-style-type: decimal;
-        margin-left: 0em;
+    margin-left:43px;
 }