Development #25259
nouveau thème Villeurbanne
0%
Description
A partir de la maquette fournie par Novactive: http://perso.entrouvert.org/~smihai/villeurbanne/sample.php.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-initial-theme-25259.patch 0001-villeurbanne-2018-initial-theme-25259.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Téléchargement du template de base à partir de la maquette, le temps qu'il y ait une vraie recette à Novactive à partir de laquelle récupèrer le template à jour.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Tu utilises roboto mais ne déclare pas la police.
Mis à jour par Frédéric Péters il y a plus de 5 ans
content = re.sub('<script src=.*jquery.min.js.*', '', content)
mais pourtant pas présent dans leur modèle.
Tu sembles par contre y laisser le chargement de Bitter/Roboto depuis le CDN Google.
(Ne pas oublier de capture d'écran pour qu'un CPF puisse commenter)
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-initial-theme-25259.patch 0001-villeurbanne-2018-initial-theme-25259.patch ajouté
- Fichier accueil.png accueil.png ajouté
- Fichier form.ong form.ong ajouté
- Fichier Mes demandes.png Mes demandes.png ajouté
- Fichier profil.png profil.png ajouté
Frédéric Péters a écrit :
[...]
mais pourtant pas présent dans leur modèle.
Un mauvais copier-coller. Viré.
Tu sembles par contre y laisser le chargement de Bitter/Roboto depuis le CDN Google.
Oui. Et la raison à cela est la fait que les 2 polices sont utilisées dans les styles du menu du site de la ville.
Si je les retire et j'inclut notre Roboto packagé et Bitter (que j'ai rajouté dans le thème localement) le rendu du menu du site n'est pas correct.
Par exemple:
font-family: "Bitter"; font-weight: 700;
et cela correspond à la police grasse.
Mais le fichier et le nom pour la police grasse de Bitter (https://www.fontsquirrel.com/fonts/bitter) est "Bitter-Bold".
Utiliser:
font-family: "Bitter"; font-weight: 700;
donc ne met pas le texte plus gras.
J'ai préféré éviter les prises de tête sur le rendu du menu du site et laissé l'import du CDN Google (pour l'instant).
(Captures rajoutées)
Mis à jour par Frédéric Péters il y a plus de 5 ans
Mais le fichier et le nom pour la police grasse de Bitter (https://www.fontsquirrel.com/fonts/bitter) est "Bitter-Bold".
Tu peux déclarer un @font-face avec comme nom Bitter-Bold qui charge le .woff de la police grasse.
J'ai préféré éviter les prises de tête sur le rendu du menu du site et laissé l'import du CDN Google (pour l'instant).
Il n'y aura pas de ack sur les intégrations utilisant des ressources d'un CDN externe.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Lié à Development #25260: ajouter la police Bitter ajouté
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-initial-theme-25259.patch 0001-villeurbanne-2018-initial-theme-25259.patch ajouté
Ok.
Police "Bitter" déclarée, mais chargeant le fichier de la police grasse. Ça fonctionne.
Dependance du ticket #25260.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Comme dans #25260 il y a eu ajout de woff2/woff, il faudrait en profiter.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-initial-theme-25259.patch 0001-villeurbanne-2018-initial-theme-25259.patch ajouté
Avec la déclaration correcte de la police Bitter dans #25260, plus besoin de rajouter une déclaration ici.
Mis à jour par Anonyme il y a plus de 5 ans
- Fichier un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_.png un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_.png ajouté
- Fichier un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_(Galaxy S5).png un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_(Galaxy S5).png ajouté
- Fichier un-user-combo.dev-eshowk.ddns.entrouvert.org_.png un-user-combo.dev-eshowk.ddns.entrouvert.org_.png ajouté
- Fichier un-user-combo.dev-eshowk.ddns.entrouvert.org_(Galaxy S5).png un-user-combo.dev-eshowk.ddns.entrouvert.org_(Galaxy S5).png ajouté
- Des ressources en 404, venant de l'erreur (surement modifier prepare-template.py qui pour que les liens vers perso.entrouvert.org soient dans le répertoire statique du thème ?)
Cross-Origin Read Blocking (CORB) blocked cross-origin response https://perso.entrouvert.org/~smihai/villeurbanne/images/_aliases/bandeau/4/4/8/1/1844-2-fre-FR/tetiere_ma_ville.jpg
- un problème avec l'icone "2 barres vertes" de la ville sur les titres de blocs de cellules combo : elle est collée au titre (capture jointe)
- sur la page A2 /accounts/, mais aussi sur la page d'accueil avec les blocs de texte et de notification n'ont pas d'ombre à gauche, contrairement aux captures (captures 1 et 4 jointes)
- l'espace en haut de page c'est à dire les traits gris obliques, et zone blanche repoussent excessivement le contenu vers le bas (2e et 4e capture), voir .banner-top dans le banner.scss du site de la ville, mais probablement utile d'enlever le padding-top en mobile au moins.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-initial-theme-25259.patch 0001-villeurbanne-2018-initial-theme-25259.patch ajouté
- Fichier editer-compte.png editer-compte.png ajouté
Elias Showk a écrit :
Voici mes premières remarques avec les captures:
- Des ressources en 404, venant de l'erreur (surement modifier prepare-template.py qui pour que les liens vers perso.entrouvert.org soient dans le répertoire statique du thème ?)
[...]
Ça vient du template de base fourni par Novactive qui fait référence à des fichiers qu'ils ne m'ont pas fourni.
- un problème avec l'icone "2 barres vertes" de la ville sur les titres de blocs de cellules combo : elle est collée au titre (capture jointe)
Tu obtiens ça avec quel navigo? Je n'arrive pas à réproduire sous Firefox ni Chrome ? (exemple en capture)
- sur la page A2 /accounts/, mais aussi sur la page d'accueil avec les blocs de texte et de notification n'ont pas d'ombre à gauche, contrairement aux captures (captures 1 et 4 jointes)
Je colle le patch à jour ajoutant les ombres sur la gauche.
- l'espace en haut de page c'est à dire les traits gris obliques, et zone blanche repoussent excessivement le contenu vers le bas (2e et 4e capture), voir .banner-top dans le banner.scss du site de la ville, mais probablement utile d'enlever le padding-top en mobile au moins.
Le template fourni est ainsi. Comme il n'y a pas de menu de navigation en haut, le décalage créé par ces barres obliques grises n'est pas trop important. Je supprime quand même le padding de 10px du bloc avec le contenu en mode mobile.
Mis à jour par Anonyme il y a plus de 5 ans
Serghei Mihai a écrit :
Elias Showk a écrit :
Voici mes premières remarques avec les captures:
- Des ressources en 404, venant de l'erreur (surement modifier prepare-template.py qui pour que les liens vers perso.entrouvert.org soient dans le répertoire statique du thème ?)
[...]
Mais du coup, tu tires un trait sur l'image bloquée ? https://perso.entrouvert.org/~smihai/villeurbanne/images/_aliases/bandeau/4/4/8/1/1844-2-fre-FR/tetiere_ma_ville.jpg
Chromium.Ça vient du template de base fourni par Novactive qui fait référence à des fichiers qu'ils ne m'ont pas fourni.
- un problème avec l'icone "2 barres vertes" de la ville sur les titres de blocs de cellules combo : elle est collée au titre (capture jointe)
Tu obtiens ça avec quel navigo? Je n'arrive pas à réproduire sous Firefox ni Chrome ? (exemple en capture)
Ça vient surement de l'utilisation d'un padding en "ex" sur h2:first-child.
- je propose de revenir sur du classique em qui reste proportionnel à la taille de la police. (cf. https://www.w3.org/Style/Examples/007/units.fr.html)
h2:first-child { padding: 0.5em 0.5em 0.5em 1em; } &::before { top: 0.6em; }
- tu peux supprimer la css de h2:first-child qui est en doublon lignes 98 à 104.
- tu peux réunir les deux sections #tracking-code pour ne pas risquer d'avoir des rendu différents, et vérifier aussi que tu n'as pas des sections de CSS dispersées, ça aidera la relecture et les patchs futurs.
- je vois le menu <nav>, donc accueil s'affiche toujours chez moi et pas dans tes captures. J'ai donc un "Accueil" qui n'est pas en
display: none;
- ça ne va toujours pas, ligne 79, pourquoi c'est
box-shadow: 2px 3px 5px rgba(0, 0, 0, 0.28);
qui ne donne pas une ombre sur tout le tour, et pas l'ombre qui fonctionne bien ligne 258box-shadow: 0 0 10px 0 silver;
?
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-initial-theme-25259.patch 0001-villeurbanne-2018-initial-theme-25259.patch ajouté
Elias Showk a écrit :
Mais du coup, tu tires un trait sur l'image bloquée ? https://perso.entrouvert.org/~smihai/villeurbanne/images/_aliases/bandeau/4/4/8/1/1844-2-fre-FR/tetiere_ma_ville.jpg
Oui, ça va être corrigé par Novactive dans leur template.
>
Chromium.
Ça vient surement de l'utilisation d'un padding en "ex" sur h2:first-child.
- je propose de revenir sur du classique em qui reste proportionnel à la taille de la police. (cf. https://www.w3.org/Style/Examples/007/units.fr.html)
[...]
Yep, corrigé.
Côté code :
- tu peux réunir les deux sections #tracking-code pour ne pas risquer d'avoir des rendu différents, et vérifier aussi que tu n'as pas des sections de CSS dispersées, ça aidera la relecture et les patchs futurs.
Ce sont 2 choses différentes: d'un côté c'est la cellule de saisie du code de suivi, de l'autre l'affichage du code lors du remplissage d'un formulaire.
Remarque supplémentaire:
- je vois le menu <nav>, donc accueil s'affiche toujours chez moi et pas dans tes captures. J'ai donc un "Accueil" qui n'est pas en
display: none;
Il suffit que tu exclus de la navigation tes pages et il n'y aura pas de "nav".
Ombres:
- ça ne va toujours pas, ligne 79, pourquoi c'est
box-shadow: 2px 3px 5px rgba(0, 0, 0, 0.28);
qui ne donne pas une ombre sur tout le tour, et pas l'ombre qui fonctionne bien ligne 258box-shadow: 0 0 10px 0 silver;
?
Yep, corrigé.
Mis à jour par Anonyme il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Go
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ad553c63eb71ecdf42c5b274500a54958f5ca79a (origin/master, origin/HEAD) Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu Jun 7 11:55:10 2018 +0200 villeurbanne-2018: initial theme (#25259)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
villeurbanne-2018: initial theme (#25259)