Produits Entr'ouvert » Intégrations graphiques Publik

Quelques retours perso après avoir installé le thème:

• Je trouve assez inutile de surcharger page-template.html et forker le code du block user-info juste pour ajouter un bonjour et 2 icônes. On pourrait s'en rapprocher avec le code existant.
• Le header desktop est trop volumineux en hauteur. Aucune obligation à respecter la hauteur de celui du site du département.
• Le main est assez austère. En particulier les pages wcs. Pourquoi ne pas reprendre les lignes grises utilisées en background sur le site du département ?
• Je ne partirais pas sur une disposition de cellules avec une grille float, la grille casse trop vite. Plutôt un template en 3 cols.
• augmenter $width (le site du département est en 1260px).
• Je baisserais la taille des titres des catégories, surtout si les titres sont forcés en capitale.
• Trouver un :hover plus visible sur les liens dans les cellules ?
• Je chercherais une unité graphique entre les titres des pages et le titre d'une démarche dans les pages wcs.
• Dans le header, je ne suis pas fan du titre "Mes démarches" en blanc sur fond bloc rouge. Mauvaise idée que de reprendre le design du champ de recherche. Bien meilleur impact: reprendre simplement 'Mes démarches' en rouge et bold ?
• différencier les style de $button et $cancel-button

Je ne doute pas que ce que pointe Thomas fasse partie des bonnes pratiques à respecter et c'est utile d'avoir tout ça, mais pour rassurer Nico tout de même c'est un POC, il n'y aura pas un niveau d'exigence élevé.

La date pour le ticket Gers c'est le 13 janvier.

Aussi, en mobile l'icone "Mes démarches" n'a pas vraiment de sens à mon avis et porte confusion à côté du menu mobile.

Je suis également d'avis de ne pas surcharger les templates juste pour un Bonjour <user>.

En fait je m'en moque de ça. Je réalise que j'aurais du décrire la spécificité de ce thème.

• en desktop, quand on rétrécie la page, la largeur du bloc de droite du header reste inchangée, et la hauteur du header est définie par la hauteur de l'image requise pour qu'elle s'affiche entièrement dans la largeur restante.
• en tablette et en mobile le bloc de droite passe au dessus, même comportement pour la hauteur de l'image.

... plutôt fixer une hauteur et caller l'image de la page dessus.

Donc, le comportement décris ci-dessus ne convient pas.
Il faut que je revois ma copie avec une intégration graphique "vraiment" standard (un header pas responsive).
Ça t'ira Pierre ?
Personnellement, je ne le visualise pas.

Je viens de discuter avec Nicolas qui ne peux pas faire les modifs demandées et mettre ça en recette pour lundi, date d'échéance du ticket Gers.

Je lui ai donc demandé de mettre sur la recette ce qu'il avait, les modifs viendront plus tard. Et s'il y a régression, j'expliquerai.

Il faut que je revois ma copie avec une intégration graphique "vraiment" standard (un header pas responsive).

Alors clairement toutes les complications/spécificités mises en place pour une intégration graphique sont des trucs qui pèsent derrière lors de de la maintenance, et c'est pour ça qu'on essaie de les éviter, mais comme rappelé plus haut on est ici sur un POC, on peut se dire que ça aura une durée de vie limitée et moins s'en soucier. (?).

Aussi, je n'ai absolument pas cherché à comprendre l'explication de pourquoi c'était nécessaire de taper des gabarits spécifiques; si quelque chose d'utile ne peut pas être fait de manière simple avec les gabarits de base, exprimons ça dans un ticket dédié pour améliorer les gabarits de base.

Je pense qu'il n'était pas plus compliqué pour Nicolas de faire les choses d'emblée de manière standard, il y a donc juste un problème de communication / formation entre devs.

Alors oui, POC, ça pourrait ne pas durer. Mais je vais essayer de faire financer à un moment une vraie intégration graphique qui permettrait de revenir dans les clous.

Merci Thomas, je regarde.

si quelque chose d'utile ...

non, c'est une catastrophe ergonomique, à cause de la taille du/des bandeau(x)

Encore une fois, j'ai fais ça parce que je n'étais pas du tout inspiré et que j'ai donc plagié leur site.
Je n'avais pas compris que l'intégration graphique n'avais vraiment aucune importance sur ce projet, je ne sais pas pourquoi je n'ai pas imprimé que c'était un POC.

Pour ma défense (après coup), j'ai l'impression que cette intégration "inutilisable" correspondrait à leur premières attentes.
mail de SEDDAOUI Karim :

Pour le moment, ce n'est pas responsive.

Je pense qu'il n'était pas plus compliqué pour Nicolas de faire les choses d'emblée de manière standard, il y a donc juste un problème de communication / formation entre devs.

Il y a toujours, et chez tout le monde, une facilité à copier/coller plutôt que faire des tickets d'évolutions globales. Sur le cas précis de la zone connexion/inscription, je viens de créer #38856.

_~~

  {% if idp_account_url %}
  <a class="account" href="{{idp_account_url}}" title="Accéder à mon compte">
  {% endif %}

mais euh, ce <a> est pas fermé.