Development #46464
ajout des templates Solis RSA
100%
Description
Conformes aux sorties du connecteur Passerelle
Fichiers
Sous-tâches
Demandes liées
Révisions associées
somme-cd80: add json cells configuration in config.json (#46464)
somme-cd80: add apa-links json cell in config.json (#46464)
Historique
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Déjà fonctionnellement validé.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
Thomas Noël a écrit :
Déjà fonctionnellement validé.
En fait non, donc on attend un peu et un prochain patch arrivera si besoin.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch ajouté
- Statut changé de En cours à Solution proposée
Voici la version quasi finale (reste un seul détail sur "rsa-actions" qui sera finalisé mardi prochain avec le département). Si on peut l'intégrer maintenant dans la recette c'est bien.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch ajouté
Version validée.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch ajouté
Avec le templates/combo/json/rsa-interlocuteur.html oublié (merci Mik pour le coup d'oeil)
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier Screenshot_2020-09-15 Portail usager - Situation bénéficiaire RSA.png Screenshot_2020-09-15 Portail usager - Situation bénéficiaire RSA.png ajouté
- Fichier Screenshot_2020-09-15 Portail usager - Fiche bénéficiaire RSA.png Screenshot_2020-09-15 Portail usager - Fiche bénéficiaire RSA.png ajouté
Mis à jour par Thomas Jund il y a plus de 3 ans
- beaucoup de styles embarqués
- `style="text-align: center;" (typographiquement ne se justifie pas toujours je trouve).
- l'utilisation de la class "infonotice" : cela fixe le design de ces blocs sans avoir la possibilité d'en changer
- ** les `<p> </p>` et `<br></br> pour générer des espaces blancs sont une bien mauvaise idée.
- sémantiquement, les balises dl > dt + dd pourraient être utiles
Sinon, voici les erreurs de code relevées
- rsa-actions.html, rsa-indus.html
- ligne 10 : retirer la balise <p> qui encadre la balise <table>. non valide. (Chaque fois s'uil y a un tableau j'ai l'impression qu'il est entouré par un p… à vérifier).
- rsa-situation.html
- ligne 92, pas de ul dans un p. Non autorisé, non valide.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch ajouté
Hop hop, j'ai corrigé ce que j'ai pu.
Thomas Jund a écrit :
- `style="text-align: center;" (typographiquement ne se justifie pas toujours je trouve).
J'ai choisi de laisser ça, la typographique j'imagine a été voulue par le client. Si un autre client RSA désire une autre présentation (gros doute) on modifiera et on ajoutera des classe CSS dédiées du genre rsa-actions-du-departement (misère...).
- l'utilisation de la class "infonotice" : cela fixe le design de ces blocs sans avoir la possibilité d'en changer
Oups, bien vu : j'ai remplacé par la nouvelle et officielle classe pk-information (le rendu est identique).
- ** les `<p> </p>` et `<br></br> pour générer des espaces blancs sont une bien mauvaise idée.
Ca veut dire "un paragraphe vide" ou "un saut de ligne", ça me va bien :-) Blague à part, je ne sais pas du tout quoi mettre à la place...
Sinon, voici les erreurs de code relevées
- rsa-actions.html, rsa-indus.html
- ligne 10 : retirer la balise <p> qui encadre la balise <table>. non valide. (Chaque fois s'uil y a un tableau j'ai l'impression qu'il est entouré par un p… à vérifier).
- rsa-situation.html
- ligne 92, pas de ul dans un p. Non autorisé, non valide.
Cool, merci . J'ai corrigé tout ça (et quelques autres <p><table> ailleurs) et visuellement rien n'a bougé, parfait.
Mis à jour par Thomas Jund il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
- Assigné à mis à Thomas Jund
Mis à jour par Thomas Jund il y a plus de 3 ans
- Lié à Development #46875: Pouvoir utiliser les class .pk-table-* avec @extend ajouté
Mis à jour par Thomas Jund il y a plus de 3 ans
- Fichier 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch 0001-combo-add-templates-for-passerelle-Solis-RSA-cells-4.patch ajouté
- Statut changé de En cours à Solution proposée
Modifs apportées :
- remplacement des class .warningnotice par .pk-attention
- suppression des styles embed text-align: center par un div.rsa-cell--introduction qui centre son contenu (voir patch css pour theme).
- suppression des styles embed with:100% sur les tableau (appliqué via div.rsa-cell--datas voir patch css pour theme)
- supression des balises vides pour créer des espaces (compensé en css, voir patch css theem)
- cell rsa-help-links:
- suppression de l'url vers le fomulaire de la somme écrit en dur, compensé par ajout d'un champ "URL du formulaire de contact par défaut" dans la définition de la cellule.
- suppresion de la class .pk-information, il est préférable de l'**appliquer directement sur la cellule**. **
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Complètement oublié qu'aujourd'hui les « {% load combo %} » ne sont plus nécessaire (on charge "combo" et "humanize" par défaut). Si tu peux les virer avant de pousser...
Et merci !
Mis à jour par Thomas Jund il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 380c99bc1fd4c2be8119237e5333fc85267a5ab7 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Tue Sep 8 15:23:17 2020 +0200 combo: add templates for passerelle Solis RSA cells (#46464)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
combo: add templates for passerelle Solis RSA cells (#46464)
to be used with these settings.JSON_CELL_TYPES: