Refacto du template user-info.html permettant les surcharges (#72598) #85

Merged
tjund merged 8 commits from wip/72598-user-info-refacto into main 2023-01-05 10:34:10 +01:00
Owner
No description provided.
tjund changed title from WIP: Refacto du template user-info.html permettant les surcharges (#72598) to Refacto du template user-info.html permettant les surcharges (#72598) 2022-12-21 11:57:51 +01:00
Author
Owner

Beaucoup de commit, mais tous petits.

J'en ai profité pour supprimer les templates de label qui deviennent inutiles.
(mais ça peut être dans une autre PR).
Possible aussi de tout stasher.

@Serghei, pour la MEL, on perd la condition sur l'image. Dis moi si c'est problématique.

Beaucoup de commit, mais tous petits. J'en ai profité pour supprimer les templates de label qui deviennent inutiles. (mais ça peut être dans une autre PR). Possible aussi de tout stasher. @Serghei, pour la MEL, on perd la condition sur l'image. Dis moi si c'est problématique.
Owner

tjund a modifié le titre de WIP: Refacto du template user-info.html permettant les surcharges (#72598) à Refacto du template user-info.html permettant les surcharges (#72598) il y a 5 minutes

Attention juste retirer le WIP ne fait pas évoluer le statut côté redmine.

> tjund a modifié le titre de WIP: Refacto du template user-info.html permettant les surcharges (#72598) à Refacto du template user-info.html permettant les surcharges (#72598) il y a 5 minutes Attention juste retirer le WIP ne fait pas évoluer le statut côté redmine.
Owner

(sur les intitulés de commits, il y aurait tpl → template).

J'en ai profité pour supprimer les templates de label qui deviennent inutiles.

Je trouve qu'on perd à ne plus avoir ces fichiers séparés, on passe d'un fichier où juste écrire "Je me connecte" à un fichier où écrire

{% extends "includes/user-info.html" %}

{% block user-info-login-label %}Je me connecte{% endblock %}
(sur les intitulés de commits, il y aurait tpl → template). > J'en ai profité pour supprimer les templates de label qui deviennent inutiles. Je trouve qu'on perd à ne plus avoir ces fichiers séparés, on passe d'un fichier où juste écrire "Je me connecte" à un fichier où écrire ``` {% extends "includes/user-info.html" %} {% block user-info-login-label %}Je me connecte{% endblock %} ```
Owner

@Serghei, pour la MEL, on perd la condition sur l'image. Dis moi si c'est problématique.

Non, ce n'est pas un souci.

> > @Serghei, pour la MEL, on perd la condition sur l'image. Dis moi si c'est problématique. > Non, ce n'est pas un souci.
fpeters reviewed 2022-12-21 12:12:45 +01:00
@ -152,2 +152,4 @@
{% skeleton_extra_placeholder top-links %}
{% include 'includes/user-info.html' %}
{% end_skeleton_extra_placeholder %}
{% endblock %}
Owner

Indentation ratée; je pensais que djhtml gérerait.

Indentation ratée; je pensais que djhtml gérerait.
Author
Owner

C'est djhtml qui m'inpose cette indentation :/

C'est djhtml qui m'inpose cette indentation :/
Author
Owner

voir ligne 161 où djhtml a déjà indenté de la sorte :

{% block body-bottom %}
  {% skeleton_extra_placeholder service-worker %}
  {% if page.pk %}  {# limit this to concrete pages on combo (ex: not 404) #}
    {% if pwa_display == "standalone" %}
      <script src="/service-worker-registration.js"></script>
    {% endif %}
  {% endif %}
{% end_skeleton_extra_placeholder %}
{% endblock %}
voir ligne 161 où djhtml a déjà indenté de la sorte : ```html {% block body-bottom %} {% skeleton_extra_placeholder service-worker %} {% if page.pk %} {# limit this to concrete pages on combo (ex: not 404) #} {% if pwa_display == "standalone" %} <script src="/service-worker-registration.js"></script> {% endif %} {% endif %} {% end_skeleton_extra_placeholder %} {% endblock %} ```
Owner

Attention il y a aussi à gérer dans imio-publik-themes.

Attention il y a aussi à gérer dans imio-publik-themes.
Author
Owner

Attention il y a aussi à gérer dans imio-publik-themes.

Oui j'ai regardé. J'attendais de voir si le code était ok de ce côté.

(sur les intitulés de commits, il y aurait tpl → template).

C'est pour pas dépasser parfois les 80 caractères :)

Je trouve qu'on perd à ne plus avoir ces fichiers séparés, on passe d'un fichier où juste écrire "Je me connecte" à un fichier où écrire

Pas faux. C'est dans l'idée de revenir à une cohérence avec les autres templates, où jamais on n'utilise d'include pour juste un label, mais des blocks.
1 seul fichier reprend l'avantage lorsqu'il y a plusieurs labels a changer et je trouve ça mieux en lecture.

> Attention il y a aussi à gérer dans imio-publik-themes. Oui j'ai regardé. J'attendais de voir si le code était ok de ce côté. > (sur les intitulés de commits, il y aurait tpl → template). C'est pour pas dépasser parfois les 80 caractères :) > Je trouve qu'on perd à ne plus avoir ces fichiers séparés, on passe d'un fichier où juste écrire "Je me connecte" à un fichier où écrire Pas faux. C'est dans l'idée de revenir à une cohérence avec les autres templates, où jamais on n'utilise d'include pour juste un label, mais des blocks. 1 seul fichier reprend l'avantage lorsqu'il y a plusieurs labels a changer et je trouve ça mieux en lecture.
tjund force-pushed wip/72598-user-info-refacto from bd7ff62c6b to 77fa01266d 2023-01-02 17:16:12 +01:00 Compare
Author
Owner

Pour avancer sur ce qui m'intéresse, j'ai supprimé les commits relatifs aux labels. Les includes labels sont restaurés.

Pour avancer sur ce qui m'intéresse, j'ai supprimé les commits relatifs aux labels. Les includes labels sont restaurés.
smihai approved these changes 2023-01-05 10:24:39 +01:00
tjund merged commit ab55513dd4 into main 2023-01-05 10:34:10 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#85
No description provided.