Méthode qui affiche un message

danielsandanielsan Member
décembre 2012 modifié dans Discussions générales
Bonjour,


est-ce possible de réaliser ces 2 petites modifs pour la fonction d'affichage de message (plxUtils::showMsg) ?


1/ cette petite modif' qui ne change rien en front mais qui améliore un poil la lecture du code source:
un retour à la ligne après l'affichage du message


2/ enlever le strong qui entoure le message.
ça impose un style et empêche de mettre en valeur un mot contenu dans le message
exemple de message "Le dossier export a bien été créé".
Après tout, les styles .error et .success ont leur règle, on peut rajouter "font-weight: bold" pour avoir le même résultat actuel.


AVANT modif' 
	/**
	 * Méthode qui affiche un message
	 *
	 * @param	msg			message à afficher
	 * @param	class		class css à utiliser pour formater l'affichage du message
	 * @return	stdout
	 **/
	public static function showMsg($msg, $class='') {

		if($class=='') echo '<p class="msg"><strong>'.$msg.'</strong></p>';
		else echo '<p class="'.$class.'"><strong>'.$msg.'</strong></p>';
	}


APRES modif': 
	/**
	 * Méthode qui affiche un message
	 *
	 * @param	msg			message à afficher
	 * @param	class		class css à utiliser pour formater l'affichage du message
	 * @return	stdout
	 **/
	public static function showMsg($msg, $class='') {

		if($class=='') echo '<p class="msg">'.$msg.'</p>'."\n";
		else echo '<p class="'.$class.'">'.$msg.'</p>'."\n";
	}


c'est pas grand chose je sais mais ça a son utilité dans certains cas.
Qu'en pensez-vous ?


Cordialement,
_____
D.San

Réponses

  • Jerry WhamJerry Wham Member
    C'est vrai que ça ne mange pas de pain et que ça peut faciliter la lecture du code source de la page générée.
    Et ça respecte un peu plus la sémantique.


    Au grand chef de trancher...
  • danielsandanielsan Member
    oui, et les strong sont plus gênants pour moi ...
  • StéphaneStéphane Member, Former PluXml Project Manager
    Pris en compte. Merci

    Consultant PluXml

    Ancien responsable et développeur de PluXml (2010 à 2018)

  • danielsandanielsan Member
    cool merci Stéphane
Connectez-vous ou Inscrivez-vous pour répondre.