Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Variables de configuration manquantes dans le formulaires de haie #443

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

pyDez
Copy link
Collaborator

@pyDez pyDez commented Oct 4, 2024

@pyDez pyDez requested a review from thibault October 4, 2024 08:26
Copy link
Collaborator

@thibault thibault left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai l'impression qu'on surcharge le template home juste pour pouvoir modifier les styles et scripts ? Ça me semble disproportionné ?

Juste pour savoir, as-tu considéré la possibilité de faire dépendre uniquement les templates _form_styles et _form_scripts de la moulinette ? Qu'est-ce qui as motivé ton choix ?

@pyDez
Copy link
Collaborator Author

pyDez commented Oct 7, 2024

@thibault

Juste pour savoir, as-tu considéré la possibilité de faire dépendre uniquement les templates _form_styles et _form_scripts de la moulinette ? Qu'est-ce qui as motivé ton choix ?

Lorsque je suis dans le monde html/js/css, je trouve ca plus agréable de pouvoir me déplacer de fichier en fichier, sans passer par des variables du contexte qui me redirige vers d'autres fichiers.

J'ai l'impression qu'on surcharge le template home juste pour pouvoir modifier les styles et scripts ? Ça me semble disproportionné ?

C'est vrai aussi :)

@thibault
Copy link
Collaborator

thibault commented Oct 8, 2024

@pyDez

Lorsque je suis dans le monde html/js/css, je trouve ca plus agréable de pouvoir me déplacer de fichier en fichier, sans passer par des variables du contexte qui me redirige vers d'autres fichiers.

Un argument valide :D

Au final j'ai l'impression qu'il n'y a plus vraiment de point communs entre les affichages des deux environnements, et qu'on s'embête à avoir une hiérarchie de templates pour rien. On verra si ça nécessite du nettoyage à l'occasion.

@pyDez pyDez merged commit bc4ae56 into main Oct 8, 2024
4 checks passed
@pyDez pyDez deleted the bug/js-errors branch October 8, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants