code lifting

mardi 28 avril 2009 à 22h56
par gilcot

Logo de gilcot

je suis entrain de regarder la noisette recherche et je constate que le copier-coller a été trop efficace (ou inachevé/incomplet, c’est selon) : l’attribut "for" de la balise "label" dans les fichiers inc_recherche_MOTEUR.html a toujours pour valeur "recherche".. il faudrait les "input" aient pour "id" également recherche pour que ce soit correct ! mais comme le formulaire de recherche de spip a déjà cet ID et que ces identifiants doivent être unique dans la page, je conseille de mettre comme "id" la même valeur que le "name" (quoique, je me demande s’il ne faut pas mettre recherche-MOTEUR pour éviter tout risque d’interaction avec d’autres formulaires à venir dans la page) et de mettre le "for" à jour en conséquence


Ce sujet est verrouillé : vous pouvez consulter son fil de discussion mais vous ne pouvez plus y répondre.

Réponses

28 avril 2009 à 23h51

toujours concernant les noisettes de recherche...

je suis entrain d’en réaliser un et je constate un étrange effet sur les menus déroulants : toutes les options sont sur la même ligne ! je pense (suis entrain de tester via perso.css) que le responsable est ceci dans base.css :

/* - Formulaires de recherche */
.formulaire_recherche {margin: 0 0 0 10px; padding: 0; background: none; display: inline; width : 158px; }
.formulaire_recherche * { display: inline; vertical-align: middle; }

  /* ... */

/* -- specificite du formulaire dans la noisette recherche */
.contenu .formulaire_recherche {display : block; padding: 0 0 5px 0;}

je ne vois pas trop pourquoi tous les éléments sont déclarés comme étant en ligne :-/ il faudrait remettre (selon moi) :

.formulaire_recherche * {vertical-align: middle; }

mais s’il y a une raison valable pour ne pas changer cela, il faudrait alors rajouter (ce n’est visiblement pas être hérité) :

/* -- specificite du formulaire dans la noisette recherche */
.contenu .formulaire_recherche, .contenu .formulaire_recherche select, .contenu .formulaire_recherche textarea {display : block; padding: 0 0 5px 0;}
29 avril 2009 à 09h53

Salut gilcot

Effectivement, ces display inline n’ont pas lieu d’être puisqu’ils sont systématiquement surchargés par la suite par des display block

Meme le vertical-align : middle, je doute de son utilité

Je me souviens avoir un peu galéré avec les css des formulaires de recherche. Ils doivent provenir des multiples essais que j’ai pu faire et que je n’ai pas viré.
Par contre, je ne vois pas en quoi cela peut influencer les menus déroulants. Si tu peux nous en dire plus, je suis preneur.

Xavier

29 avril 2009 à 10h36

Bien vu Gilcot pour l’attribut "for" de la balise "label" dans les fichiers inc_recherche_MOTEUR.html

C’est corrigé et commité

29 avril 2009 à 12h40

bonjour SarkARchimeD (Xavier)

si je ne rentre pas tard ce soir (donc avant 21h), je passerai par la shoutbox et si tu es là alors je vais virer mon perso.css pour que tu puisse contempler l’effet des menus/listes déroulant/es : il s’affichent carrément sur une ligne avec mon FF3/Linux :o (c’est Firebug qui m’a permis de traquer la classe fautive)

29 avril 2009 à 12h45

je viens de comprendre : tu as compris menu ..des rubriques..
en fait je faisait allusion aux balises SELECT (donc j’aurais du dire liste déroulante..) dans le/la formulaire/noisette (j’avais menu en tête car dans ma personnalisation, la recherche fonctionne comme un filtre ..et donc est un sorte de menu alternatif et non une vraie recherche)

7 mai 2009 à 16h02

hello la SarkaTeam

je reviens sur les formulaires de recherche... toujours au niveau des css, il y a une déclaration incongrue :

.formulaire_recherche label {display: none !important; }

ça fonctionne.. Mais : concernant ma remarque sur l’attribut "for" de cette balise ; elle n’est pas obligatoire dans un cas particulier mais valide :

<label><:info_rechercher_02:>
<input type="text" name="p" class="text" size="10" maxlength="255" value="<:sarkaspip:requete:>" onfocus="this.value='';" />
</label>

(exemple de réécriture de inc_recherche_yahoo)
or dans ce cas, que se passe t-il ?
par ailleurs, la balise "label" n’est pas obligatoire (et effectivement elle n’est pas utilisé dans inc_recherche_voila) mais recommandé (car augmente l’acceptabilité) pour les titres associés aux champs : c’est donc une information utile quand elle est présente, donc elle ne devrait pas être caché (ou alors on ne la fait pas figurer puisque ce n’est aucunement obligatoire) :/