Changeset 2647:bd67628faadf

Timestamp:

02/05/14 15:53:27 (12 years ago)

Author:

franck <carnet.franck.paul@…>

Branch:

default

Message:

Visually split each group of files in three sub-groups (if any) : theme's files, parent's theme's file, template set's files, fixes #1704

Files:

• locales/en/plugins.po (modified) (1 diff)
• locales/fr/plugins.po (modified) (1 diff)
• plugins/themeEditor/class.themeEditor.php (modified) (2 diffs)
• plugins/themeEditor/locales/en/help/help.html (modified) (2 diffs)
• plugins/themeEditor/locales/fr/help/help.html (modified) (2 diffs)
• plugins/themeEditor/style.css (modified) (1 diff)

locales/en/plugins.po

@@ -1965 +1965 @@
 msgid "Drag widgets here to remove."
 msgstr ""
+
+msgid "From theme:"
+msgstr ""
+
+msgid "From parent:"
+msgstr ""
+
+msgid "From template set:"
+msgstr ""

locales/fr/plugins.po

@@ -1983 +1983 @@
 msgstr "Glisser les widgets ici pour les retirer du volet."
 
+msgid "From theme:"
+msgstr "Du thème :"
+
+msgid "From parent:"
+msgstr "Du thème parent :"
+
+msgid "From template set:"
+msgstr "Du jeu de template :"
+
 #~ msgid "RSS or Atom feed subscription links"
 #~ msgstr "Liens d'abonnement aux flux RSS ou Atom"

plugins/themeEditor/class.themeEditor.php

@@ -47 +47 @@
      }
 
-     public function filesList($type,$item='%1$s')
+     public function filesList($type,$item='%1$s',$split=true)
      {
           $files = $this->getFilesFromType($type);
@@ -56 +56 @@
 
           $list = '';
-          foreach ($files as $k => $v)
-          {
-               if (strpos($v,$this->user_theme) === 0) {
-                    $li = sprintf('<li class="default-file">%s</li>',$item);
-               } elseif ($this->parent_theme && strpos($v,$this->parent_theme) === 0) {
-                    $li = sprintf('<li class="parent-file">%s</li>',$item);
-               } else {
-                    $li = sprintf('<li>%s</li>',$item);
-               }
-               $list .= sprintf($li,$k,html::escapeHTML($k));
+          if ($split) {
+               // First part for current theme
+               $list_theme = '';
+               foreach ($files as $k => $v)
+               {
+                    if (strpos($v,$this->user_theme) === 0) {
+                         $li = sprintf('<li class="default-file">%s</li>',$item);
+                         $list_theme .= sprintf($li,$k,html::escapeHTML($k));
+                    }
+               }
+               // Second part for parent theme if any
+               $list_parent = '';
+               foreach ($files as $k => $v)
+               {
+                    if ((strpos($v,$this->user_theme) !== 0) && ($this->parent_theme && strpos($v,$this->parent_theme) === 0)) {
+                         $li = sprintf('<li class="parent-file">%s</li>',$item);
+                         $list_parent .= sprintf($li,$k,html::escapeHTML($k));
+                    }
+               }
+               // Third part for template set
+               $list_tpl = '';
+               foreach ($files as $k => $v)
+               {
+                    if ((strpos($v,$this->user_theme) !== 0) && ($this->parent_theme && strpos($v,$this->parent_theme) !== 0)) {
+                         $li = sprintf('<li>%s</li>',$item);
+                         $list_tpl .= sprintf($li,$k,html::escapeHTML($k));
+                    }
+               }
+               $list .= ($list_theme != '' ? sprintf('<li class="group-file">'.__('From theme:').'<ul>%s</ul></li>',$list_theme) : '');
+               $list .= ($list_parent != '' ? sprintf('<li class="group-file">'.__('From parent:').'<ul>%s</ul></li>',$list_parent) : '');
+               $list .= ($list_tpl != '' ? sprintf('<li class="group-file">'.__('From template set:').'<ul>%s</ul></li>',$list_tpl) : '');
+          } else {
+               foreach ($files as $k => $v)
+               {
+                    if (strpos($v,$this->user_theme) === 0) {
+                         $li = sprintf('<li class="default-file">%s</li>',$item);
+                    } elseif ($this->parent_theme && strpos($v,$this->parent_theme) === 0) {
+                         $li = sprintf('<li class="parent-file">%s</li>',$item);
+                    } else {
+                         $li = sprintf('<li>%s</li>',$item);
+                    }
+                    $list .= sprintf($li,$k,html::escapeHTML($k));
+               }
           }
 

plugins/themeEditor/locales/en/help/help.html

@@ -17 +17 @@
 </ul>
 
+<p>Each section of files may have three sub-sections :</p>
+<ul>
+     <li>Files belonging to the current theme</li>
+     <li>Files belonging to the parent theme</li>
+     <li>Files belonging to the template set</li>
+</ul>
+
 <p>The bullet in front of each file can have three different colors:</p>
 <ul>
-<li><strong>Black</strong> means it is in the <code>default</code> theme.</li>
-<li><strong>Yellow </strong> means it belongs to the current theme.</li> 
+<li><strong>Yellow </strong> means it belongs to the current theme.</li>
 <li><strong>Red </strong> means it belongs to a parent theme.</li>
+<li><strong>Black</strong> means it is in the template set.</li>
 </ul>
+
+<p>The files belonging to the current theme are also in bold.</p>
 
 <h4>File mofidication</h4>
@@ -29 +38 @@
 <p>Be careful: Modifications are applied immediately after saving.</p>
 
-<p>If you modify a file that comes from the <code>default</code> theme, the file will be copied into your current theme.</p>
+<p>If you modify a file that comes from the template set or the parent theme, the file will be copied into your current theme.</p>
 
 <h4>Syntax highlighting</h4>

plugins/themeEditor/locales/fr/help/help.html

@@ -13 +13 @@
   <li>fichiers du thème&nbsp;: les templates,</li>
   <li>fichiers CSS&nbsp;: les feuilles de style,</li>
-  <li>fichiers JavaScript.</li>
+  <li>fichiers JavaScript,</li>
   <li>fichiers de traduction.</li>
+</ul>
+
+<p>Chaque groupe de fichiers peut contenir trois sous-groupes :</p>
+<ul>
+     <li>fichiers du thème actif,</li>
+     <li>fichiers du thème parent,</li>
+     <li>fichiers du jeu de "template".</li>
 </ul>
 
 <p>A la gauche de chaque fichier, chaque puce se voit attribuée une couleur :</p>
 <ul>
-<li><b>noire</b> indique qu'il se trouve dans le thème "default".</li>
-<li><b>jaune</b> indique qu'il fait partie du thème actif.</li> 
-<li><b>rouge</b> indique qu'il fait partie d'un thème parent.</li>
+<li><b>jaune</b> indique qu'il fait partie du thème actif.</li>
+<li><b>rouge</b> indique qu'il fait partie du thème parent.</li>
+<li><b>noire</b> indique qu'il se trouve dans le jeu de "template".</li>
 </ul>
+
+<p>Les fichiers du thème actif sont également repérés en gras.</p>
 
 <h4>Modification des fichiers</h4>
@@ -31 +40 @@
 <p>Les modifications sont immédiatement appliquées lors de la sauvegarde, soyez vigilants.</p>
 
-<p>A noter que si vous modifiez un fichier se trouvant dans le thème "default", celui-ci sera copié dans votre thème en cours d'utilisation.</p>
+<p>A noter que si vous modifiez un fichier se trouvant dans le jeu de "template" ou dans le thème parent, celui-ci sera copié dans votre thème en cours d'utilisation.</p>
 
 <h4>Coloration syntaxique</h4>

plugins/themeEditor/style.css

@@ -21 +21 @@
 #file-chooser li.default-file {
      color: #f90;
+     font-weight: bold;
 }
 #file-chooser li.parent-file {
      color:#c00;
 }
+#file-chooser li.group-file {
+     list-style: none;
+     margin-bottom: 1em;
+}
+#file-chooser li.group-file ul {
+     margin-top: .5em;
+}
 textarea {
      font: 1em Monaco,"Courier New",Courier,monospace;