We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 8790
    • 526 Posts
    je confirme :

    Il faut revoir les css fournis si on veut les utiliser tq ou même pour les customiser.

    A priori, virer les def des span tous seuls.
    et sûrement redéfinir les "a" avec leurs chainage d’héritage et non pas tous seuls.

    évidemment ça peut quand même marcher si ton site n’a pas dans ses css "normaux" ni def de span, ni def de "a", mais pour au moins ce dernier cas..ce serait très surprenant.

    Ca ne règle pas le pb d’insertion de balises styles en plein milieu du body.

    Il est donc indispensable d’appliquer la méthode de contournement exposée par FuryDE, ET de ne pas passer par un appel de chunk css ET d’utiliser ses propres link css.

    Mon analyse est-elle correcte?
      Schtroumpf Grognon - Grouchy Smurf
      ---------------------------------
      Faites pas attention.. - Don't pay attention
      http://www.dzi-neo.net
      • 13320
      • 245 Posts
      Bonjour,

      J’ai corrigé le problème de l’insertion des feuilles de styles.

      Je suis pas malheuresement le meilleur codeur CSS, j ’ai pu créer de gros span pas propre, mais dites moi et je corrige les feuilles d’expemple de l’archive.

      Effectivement, les styles sont dans le body et c’est pas très propre, je pensais vous mettre des placeholder poru que les mettiez ou bon vous semble, vous en pensez quoi ?

      Pour ton premier post ce matin, le dernière version de CALx (pas encore sortie) gère les osus répertoire tu pourras donc créer plein de répertoire en dessous ^^

      Désolé, CALx a encore des bugs, et c’est avec plaisir que je prends vos remarques en compte pour le corriger.

      Dites moi bien ce que vous changez dans le code pour que je fasse pareil, sinon vous allez avoir des surpises en reprenant la nouvelle version :/
        [b]D
        • 6726
        • 7,075 Posts
        Il n’est pas question de critiquer un code déjà complexe, tu as fait un excellent boulot !
        A nous de suggérer des améliorations smiley

        Avoir des chunks servant de template et des placeholders serait la cerise sur le gateau grin
          .: COO - Commerce Guys - Community Driven Innovation :.


          MODx est l'outil id
          • 13320
          • 245 Posts
          J’ai déjà ajouté deux place holder pour les liens vers l’année complète et les évènements du mois.

          Je réfléchis régulmièrement à un CALx "chuncker" wink
            [b]D
            • 1876
            • 835 Posts
            Bonjour,

            Je pense que elz64 se fera un plaisir de t’envoyer ses corrections suite à ses tests.

            Aour
              • 8790
              • 526 Posts
              Merci Jabber.

              Effectivement, les styles sont dans le body et c’est pas très propre, je pensais vous mettre des placeholder poru que les mettiez ou bon vous semble, vous en pensez quoi ?

              Je ne suis pas un codeur comme d’autres, je ne vois donc pas forcément les choses sous le même angle . Perso, quel est l’intérêt de passer par ce mécanisme de gestion des styles ?
              Les styles doivent êtres insérés en <head>. Pas de choix possible.

              Je préfère donc largement faire appel à une css externe placée normalement dans le/css du template. Le fait que ce soit en chunk ne me parait pas indispensable (voire même contraignant). De la sorte on link dans l’ordre adequat et basta.
              Si le besoin de styler de manière différente, dans le même site différents appels de CALx étaient nécessaire, je suggère alors d’enrober sa sortie dans un div id="bloc-CALx.." ce nom pouvant être effectivement un des paramètres à passer.

              Dans cet ordre d’idée, afin de pouvoir facilement styler la liste des évènements (qui appelle toujours la même page en l’écrasant à la volée et la recréer uniquement sur la base du default template du site, ce qui est assez contraignant) je suggère aussi comme améliorations :


              • d’éviter de casser la définition du modèle de cette page (si c’est possible). Une fois que c’est défini dans le manager ca ne devrait pas être écrasé à chaque appel). Ou que le template de page associé à la page de liste soit aussi un paramètre.

                Que la sortie multievents ne soit pas directement insérée dans le content, mais dans un div identifié CALx de telle sorte à permettre de styler DEDANS cas la liste n’a pas de class particulière.
              (j’ai mis ca dans run() dans l’outpu:
              	if($this->getTypeProcess=='showMultiEvent'){
              
              			//Appel pour la version avec les dates de publications
              
              			if($this->useTV=='false'){
              
              				$output = $this->createLinkMultiEventPage($_GET['jour']);
              
              			}else{
              
              				$output = $this->createLinkMultiEventPageTV($_GET['jour']);
              
              			}
              // on enrobe la sortie ds un div classé pour permettre le styling dedans
              
              			return utf8_encode( '<div id="list-CALx">'.$output.'</div>');
              
              		}



              la dernière version de CALx (pas encore sortie) gère les osus répertoire
              Cool !
              Bravo

              CALx a encore des bugs, et c’est avec plaisir que je prends vos remarques en compte pour le corriger.
              Pas la peine de t’excuser ! (manquerai plus que ca)
              Tu donnes un outils gratos, tu le supportes en réagissant immédiatement, (et c’est pas si courant). On parle juste de boulot là pour faire avancer le truc .
              Tiens moi au courant de tes avancées, je pense pouvoir continuer à le tester dan sles prochaines semaines.

              Cordialement.


              rem pour Aour :
              J’interprète peut-être mal, mais quand je pose une question sans trop chercher on me jette, quand j’analyse un pb de manière un peu fouillée parce je bosse vraiment dessus, on semble m’accuser avec ironie de critiquer gratuitement, je suis sans doute pas très bon, soit, aussi n’ai-je donc pas le droit de remonter mon analyse, c’est ca ? undecided bof





                Schtroumpf Grognon - Grouchy Smurf
                ---------------------------------
                Faites pas attention.. - Don&#39;t pay attention
                http://www.dzi-neo.net
                • 23050
                • 1,842 Posts
                Quote from: elz64 at Sep 12, 2007, 02:54 AM

                Si le besoin de styler de manière différente, dans le même site différents appels de CALx étaient nécessaire, je suggère alors d’enrober sa sortie dans un div id="bloc-CALx.." ce nom pouvant être effectivement un des paramètres à passer.

                Je rejoins la proposition d’elz. Je n’ai pas encore pu tester CALx (honte à moi lipsrsealed) mais je trouve très pratique quand les output d’un snippet sont contenus dans un élément html qui contient un #id propre à lui. Ca permet de styler ensuite très facilement les enfants du bloc.

                Je propose #calx wink

                PS : Bon et puis sinon, je vais bientôt installer CALX... si si ! grin
                  • 1876
                  • 835 Posts
                  rem pour Aour :
                  J’interprète peut-être mal, mais quand je pose une question sans trop chercher on me jette, quand j’analyse un pb de manière un peu fouillée parce je bosse vraiment dessus, on semble m’accuser avec ironie de critiquer gratuitement, je suis sans doute pas très bon, soit, aussi n’ai-je donc pas le droit de remonter mon analyse, c’est ca ? Undecided bof

                  Si tu lis la réponse de Jabber, il est à deux doigts de s’excuser de son code. Donc c’est à cela que nous répondons.
                  Je ne vois pas de critique dans mon post ni dans celui de David. pour ma part, je lui indique juste que tu lui fera un retour de tes tests avec des pistes d’améliorations.

                    • 8790
                    • 526 Posts
                    Salut Perrine,


                    Pour les conventions de nommage, il faut aussi prendre en compte les enrobages qui existent déjà dans d’autres sections du snippet.

                    A ma connaissance, Jabber avait déjà fait ca pour la sortie du calendrier lui-même.
                    Le bloc est un div id="CALxContent"

                    note: ce nom est bien pour un bloc très générique.

                    Voir si il suffit à tous les cas possibles d’enrobage. (c-à-d pour les listes texte multi events aussi)

                    Complément à mes modif du php :

                    J’ai oublié de préciser tout à l’heure que j’ai aussi tapé dans la création du tableau de calendier :
                    Il faut en effet scinder la 1ere ligne (normalement devrait être un <thead>, mais bon, jep ense qu’il faut juste IDentifier le 1er row dans BuildClax() :
                    	//Creation du tableau inferieur
                    
                    			$output.= '<div class="CALx"><table class=\'CALxTable'.$this->chunkCSSID.'\'>'."\n";
                    
                       		$output.='<tr class="CALx-jours">';


                    Ainsi j’ai pu styler la ligne des jours (autre couleur, capitalisé par ex.)

                    J’aime pô les tables parce que je suis assez nul pour les styler et j’arrive pas à faire tout ce que je voudrais mais ca .. grin


                      Schtroumpf Grognon - Grouchy Smurf
                      ---------------------------------
                      Faites pas attention.. - Don&#39;t pay attention
                      http://www.dzi-neo.net
                      • 8790
                      • 526 Posts
                      ah, ok Aour désolé, la parano à la vie dure !

                      juste une question de ton alors .
                      smiley
                        Schtroumpf Grognon - Grouchy Smurf
                        ---------------------------------
                        Faites pas attention.. - Don&#39;t pay attention
                        http://www.dzi-neo.net