developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#812 closed enhancement (fixed)

master_admin pouvoir personnaliser l'info-box

Reported by: Lipki Owned by: Lipki
Priority: normal Milestone: jelix 1.1
Component: module:master_admin Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

De la même manière que dans le menu, il peut être utile d'ajouter des éléments dans l'infobox, sans pour autant réecrire main.tpl

Le principe

function onmasteradminGetMenuContent ($event) {
    $event->add(new masterAdminMenuItem('lien', 'Mon lien', jUrl::get('main~default:index'), 7, 'infobox'));
}

Comme pour le menu, tous les items dans infobox, s'ajoute dans infobox.

Attachments (10)

infobox.diff (8.2 KB) - added by Lipki 11 years ago.
infobox.2.diff (6.5 KB) - added by Lipki 11 years ago.
infobox.3.diff (6.3 KB) - added by Lipki 11 years ago.
infobox.4.diff (4.2 KB) - added by Lipki 11 years ago.
infobox.5.diff (9.1 KB) - added by Lipki 11 years ago.
infobox.6.diff (5.4 KB) - added by Lipki 11 years ago.
infobox.7.diff (5.0 KB) - added by Lipki 11 years ago.
sans le tpl zone
infobox.8.diff (5.0 KB) - added by Lipki 11 years ago.
sans le tpl zone
infobox_aveczone.diff (3.7 KB) - added by Lipki 11 years ago.
infobox_sanszone.diff (4.5 KB) - added by Lipki 11 years ago.

Download all attachments as: .zip

Change History (27)

Changed 11 years ago by Lipki

comment:1 Changed 11 years ago by Lipki

  • Owner set to Lipki
  • Status changed from new to assigned
  • Type changed from bug to enhancement

Changed 11 years ago by Lipki

Changed 11 years ago by Lipki

comment:2 Changed 11 years ago by laurentj

  • Milestone jelix 1.1 deleted
  • review changed from review? to review-
  • problèmes d'espaces ajoutés en fin de ligne
  • utiliser un autre évènement pour récupèrer les informations ça évitera ces tests sur le parentId

comment:3 Changed 11 years ago by laurentj

  • Component changed from modules to module:master_admin

comment:4 Changed 11 years ago by Lipki

Je savais que tu aller dire ça :) Ok je corrige tous de suite.

comment:5 Changed 11 years ago by Lipki

onmasteradminGetInfoBoxContent ??

comment:6 Changed 11 years ago by Lipki

pour ce menu je n'utilise pas selectedMenuItem et je n'ai pas non plus permis l'utilisation de sous menu.

Question :

  • Donc et ce que je remet selectedMenuItem ?
  • et ce que je remet le gestion des sous menu ?

Changed 11 years ago by Lipki

Changed 11 years ago by Lipki

Changed 11 years ago by Lipki

comment:7 Changed 11 years ago by Lipki

  • review changed from review- to review?

comment:8 Changed 11 years ago by Lipki

  • Version changed from 1.0RC2 to trunk

comment:9 Changed 11 years ago by laurentj

  • review changed from review? to review-
+++ lib/jelix-admin-modules/master_admin/templates/main.tpl	(working copy)


     <div id="info-box">
-        <div id="info-user">{@master_admin~gui.header.user@}
-        {ifacl2 'auth.user.view'}
-        <span id="info-user-login"><a href="{jurl 'jauthdb_admin~user:index', array('id'=>$user->login)}">{$user->login}</a></span>
-        {else}
-        <span id="info-user-login">{$user->login}</span>
-        {/ifacl2}
-        | <a href="{jurl 'jauth~login:out'}" id="info-user-logout">{@master_admin~gui.header.disconnect@}</a>
-        </div>
+        {$INFOBOX}
     </div>

Non, utilise le plugin de template zone, cela evitera de modifier la reponse admin, et donc de rester compatible avec ceux qui ont déjà installé le module master_admin

+++ lib/jelix-admin-modules/master_admin/templates/zone_admin_infobox.tpl	(revision 0)

@@ -0,0 +1,18 @@

+<div class="infobox-bloc" id="info-user">
+    <ul>
+        <li>
+            <strong>{@master_admin~gui.header.user@}</strong>

pas besoin d'une div, et l'id info-user devrait être sur le li des infos du user

+++ lib/jelix-admin-modules/master_admin/zones/admin_infobox.zone.php	
+    protected function _prepareTpl(){
+        $infobox = new masterAdminMenuItem('infobox', '', '');
+        
+        $items = jEvent::notify('masteradminGetInfoBoxContent')->getResponse();
+        
+        foreach ($items as $item) {
+            $infobox->childItems[] = $item;
+        }
+        
+        usort($infobox->childItems, "masterAdminItemSort");
+        $this->_tpl->assign('infoboxitems', $infobox);
+        $this->_tpl->assign('user', jAuth::getUserSession());
+    }
  • la classe masterAdminMenuItem n'est pas incluse dans _prepareTpl.
  • pourquoi mettre un à les items d'un tableau dans un autre tableau (qui est vide à l'origine ?
  • pourquoi d'ailleurs utiliser un item de menu "racine", puisqu'il ne sert à rien ?
+++ lib/jelix-scripts/templates/responses/adminHtmlResponse.class.php.tpl	(working copy)

@@ -27,6 +27,7 @@

         $this->title .= ($this->title !=''?' - ':'').' Administration';
         $this->body->assignIfNone('selectedMenuItem','');
         $this->body->assignZone('MENU','master_admin~admin_menu', array('selectedMenuItem'=>$this->body->get('selectedMenuItem')));
+        $this->body->assignZone('INFOBOX','master_admin~admin_infobox');
         $this->body->assignIfNone('MAIN','');

à annuler donc, voir plus haut

+++ lib/jelix-www/design/master_admin.css	(working copy)

@@ -4,7 +4,7 @@

     font-family: Verdana, Arial, Sans; 
     font-size:0.8em;
     margin:0; padding:0;
-    background-color:#eff4f6;
+    background-color:#002830;

mais pourquoi donc vouloir un fond tout noir ?

+#info-box ul {
+    margin: 0;
+}
+#info-box ul li  {
+    list-style-type:none;
+    display: inline;
+    margin-left: 10px;
+}

à modifier selon les modifs préconisée dans le template

 
+
 #admin-message {}

éviter de rajouter des lignes pour rien.

Et enregistre ton patch avec des sauts de lignes unix, et pas windows.

comment:10 Changed 11 years ago by Lipki

1 le plugin zone mouais tes sur ?

Après la 1.1 ce sera définitif pour un bon moment, alors pourquoi créer dès maintenant un décalage entre la création du menu et celle de l'infobox, juste pour respecter des RC, c'est dommage.

2 Ouaip

3 Tout à fait.

4 Pareil voir plus haut

5 A non pas noir ! Vert foncé.
C'est pour éviter d'avoir un blanc sous le footer dans les pages courante.
Et puis j'aime bien la page de connexion, sombre qui nous accueil, avec juste une porte d'entrée blanche, au centre, on a envie d'aller vers la lumière :).

6 Bien sûr.

7 L'esthétique du patch en prend un coup, c'est vrai !

Changed 11 years ago by Lipki

sans le tpl zone

Changed 11 years ago by Lipki

sans le tpl zone

comment:11 Changed 11 years ago by laurentj

je ne vois toujours pas l'interet de faire ce foreach... Et il y a encore cette classe qui traine sur le ul.

Après la 1.1 ce sera définitif pour un bon moment, alors pourquoi créer dès maintenant un décalage entre la création du menu et celle de l'infobox, juste pour respecter des RC, c'est dommage.

Tu es alors chargé de répondre à toutes les questions sur le problème en question lorsque les utilisateurs mettront à jour.

comment:12 Changed 11 years ago by Lipki

dit moi au moins que tu est d'accord sur le principe, sinon je veut bien répondre au question autant que possible.

Mais je vais faire l'autre patch (pas eu le temps hier) ru n'auras qu'a choisir.

A tiens un foreach... mais que fait-il là ? :)

Changed 11 years ago by Lipki

Changed 11 years ago by Lipki

comment:13 Changed 11 years ago by Lipki

  • review changed from review- to review?

c'est les deux dernier, c'est bon.

tu prend celui que tu veut !

comment:14 Changed 11 years ago by laurentj

  • Milestone set to jelix 1.1

comment:15 Changed 11 years ago by laurentj

  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from assigned to closed

Tu as oublié les modifs dans le fichier manifest, et 2-3 details.

trunk + 1.1.x. r1302

comment:16 Changed 11 years ago by laurentj

  • Documentation needed set
Note: See TracTickets for help on using tickets.