Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

[Roadmap] Sortir le JS du code serveur #128

Merged
merged 15 commits into from
Mar 5, 2016
Merged

[Roadmap] Sortir le JS du code serveur #128

merged 15 commits into from
Mar 5, 2016

Conversation

prytoegrian
Copy link
Member

Cf. Roadmap #103

Hors petit js inline, il n'y a désormais plus de javascript au sein du code serveur ; dans le but d'éclater les responsabilités des fichiers et améliorer la lisibilité, c'était nécessaire. Il n'y a plus non plus de contenu JS sous la forme de fichier php afin de permettre la mise en cache par le navigateur et les inclusions des ressources sont toutes déportées dans les vues et uniquement là-bas.
De plus, les ressources externes que nous ne pouvons pas faire gérer par composer ont subi le même traitement d'organisation.

Je suis donc parti sur une forme d'arborescence qu'il serait sympa d'avoir dans le futur de l'appli, tu me diras ce que tu en penses.

L'organisation des ressources se retrouve donc comme suit :

  • Bibliothèques externes (jQuery, font-awesome, etc.) sont chacun dans leur propre répertoire mais désormais dans Public\Assets\nomDeBibliothèque
  • Création des répertoires Public\Assets\Js, Public\Assets\Css et Public\Assets\Img
  • Création de la future arbo applicative App\...

Comme tu peux le voir, il n'y plus que les vues dans /template, on s'approche donc :-)

Puisque j'en avais besoin et que c'est l'objet d'un ticket quelques part, j'ai créé une fonction de génération de Datepicker, utilisable où l'on veut et avec les paramètres qu'on veut. Manque plus qu'à lui dire de travailler avec n'importe quel input mais comme c'était pas le sujet, j'ai pas plus avancé dans cette voie.

Pour un besoin lié au déplacement du JS, j'ai dû amorcer la gestion de la testabilité et ajouter des fichiers de configuration pour l'outil de tests unitaires, atoum.

Je n'ai par contre pas trop su quoi faire des fichiers app.css, app.less et de ce qu'il y a dans scss

@prytoegrian prytoegrian mentioned this pull request Feb 23, 2016
18 tasks
@wouldsmina
Copy link
Member

revue ok.
test ok.

@wouldsmina
Copy link
Member

Mince! J'ai fait un merge en ligne de commande, mais j'ai pas tes commits!

@prytoegrian
Copy link
Member Author

ah oui tiens ! je sais pas comment tu as fait ça

@prytoegrian
Copy link
Member Author

Je merge uniquement pour qu'on soit carré, les commits étant déjà intégré dans develop

prytoegrian added a commit that referenced this pull request Mar 5, 2016
[Roadmap] Sortir le JS du code serveur
@prytoegrian prytoegrian merged commit ceaa6e8 into libertempo:develop Mar 5, 2016
@prytoegrian prytoegrian deleted the topic/assets branch March 5, 2016 19:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants