Projet

Général

Profil

Development #6184

Ajouter une option pour charger des javascripts

Ajouté par Jérôme Schneider il y a plus de 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Jérôme Schneider
Catégorie:
-
Version cible:
-
Début:
19 décembre 2014
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Ajouter une options dans les mappers qui permettre de charger un ou plusieurs js dans les headers. Comme par exemple :

{
   'path': r'/',
   'method': 'GET',
   'scripts': ['js/test.js', 'toto.js']
},

Fichiers

Révisions associées

Révision d7e0fffb (diff)
Ajouté par Jérôme Schneider il y a environ 9 ans

add option 'scripts' to mapper which allows to load javascripts in header

Closes #6184

Historique

#1

Mis à jour par Jérôme Schneider il y a plus de 9 ans

#2

Mis à jour par Frédéric Péters il y a plus de 9 ans

Si jamais le <head> avait des attributs, ils seraient perdus; ça ne m'a pas l'air commun mais comme tu y fais attention dans la regex <head.*?>, je me dis que ça se rencontre quand même.

#3

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 80 à 100

Appliqué par commit commit:303fca868945ff2986088e8bab887c0441ffeb6b.

#4

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Statut changé de Résolu (à déployer) à En cours
  • % réalisé changé de 100 à 60
#5

Mis à jour par Frédéric Péters il y a plus de 9 ans

Quid des attributs du <head> ?

#6

Mis à jour par Jérôme Schneider il y a plus de 9 ans

Nouveau patch qui gère ce cas.

#7

Mis à jour par Frédéric Péters il y a environ 9 ans

Ça marche. (je me dis qu'il y a peut-être encore des sites qui mettent leurs <HEAD> en majuscules mais qu'on peut ignorer ça pour le moment).

#8

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

Frédéric Péters a écrit :

Ça marche. (je me dis qu'il y a peut-être encore des sites qui mettent leurs <HEAD> en majuscules mais qu'on peut ignorer ça pour le moment).

Un petit re.IGNORECASE ça mange pas de pain..

#9

Mis à jour par Jérôme Schneider il y a environ 9 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 60 à 100

Appliqué par commit commit:d7e0fffb124f8c06a67d39e3ae2b78336580f79e.

#10

Mis à jour par Jérôme Schneider il y a environ 9 ans

J'ai bien rajouté le re.IGNORECASE dans le commit.

#11

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF