Project

General

Profile

Evolution #3460

Compléter le filtre vider_attribut ou ajouter des filtres pour avoir un équivalent de addClass / removeClass

Added by erational 👺鬼 over 5 years ago. Updated about 22 hours ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
-
Target version:
3.3
Start date:
06/04/2015
Due date:
% Done:

0%

Resolution:
fixed

Description

Les filtres vider_attribut, inserer_attribut, extraire_attribut répondent bien au besoin sauf dans le cas d'attribut avec plusieurs valeurs typiquement
class="a b c"

Il faut peut-être pouvoir supprimer "b" ou ajouter "d" simplement à la facon de addClass / removeClass en jQuery

faut-il:
- créer de nouveaux filtres ?
OU
- ajouter un argument à la fonction vider_attribut vider_attribut{class,b}
Dans ce cas, il faudra préciser la syntaxe précise de inserer_attribut.
Pour l’instant, le inseret_attribut écrase la valeur précédente pour la nouvelle et ne permet pas l'ajout.

History

#1 Updated by b b over 5 years ago

  • Description updated (diff)

#2 Updated by erational 👺鬼 over 5 years ago

  • Target version changed from 3.1 to 3.2

3.2 pour ne pas retarder la release 3.1

#3 Updated by marcimat 🌻 about 4 years ago

Ah oui ça me parle ça. J'ai déjà eu ce soucis avec des classes.
On pourrait proposer "ajouter_classe" et "enlever_classe" en filtre. C'est avant tout l'usage le plus courant.

Sinon "completer_attribut" et "tronquer_attribut", mais la gestion des espaces pourrait être ennuyante (en ajoutant une classe on sait forcément qu'il y a un caractère espace en délimiteur, alors que pour un autre attribut, peut être qu'il y a rien, par exemple si on veut compléter le 'title').

#4 Updated by cedric - 16 days ago

  • Target version changed from 3.2 to 3.4

#5 Updated by cedric - 6 days ago

  • Status changed from Nouveau to En cours
  • Target version changed from 3.4 to 3.3

J'ai donc fait une proposition ici https://git.spip.net/spip/spip/pulls/127
puisque ça répond aussi au besoin de #4552

#6 Updated by marcimat 🌻 6 days ago

Cool :)

#7 Updated by erational 👺鬼 6 days ago

Super. Je teste cela la semaine prochaine :)

#8 Updated by erational 👺鬼 3 days ago

J'ai testé.
Tout fonctionne parfaitement. Génial !

Remarques de détails:
- on peut avoir un class vide mais c'est valide HTML5 <p class=''>
- suite à une opération on peut avoir un espace blanc mais bon ce n'est génant <p class=' maclasse'>
- bascule_class est surprenant mais efficace.

#9 Updated by cedric - about 22 hours ago

  • Status changed from En cours to Fermé
  • Resolution set to fixed

Intégré, en renommant basculer_class en commuter_class

Also available in: Atom PDF