Projet

Général

Profil

Development #25259

nouveau thème Villeurbanne

Ajouté par Serghei Mihai il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 juillet 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

A partir de la maquette fournie par Novactive: http://perso.entrouvert.org/~smihai/villeurbanne/sample.php.


Fichiers

0001-villeurbanne-2018-initial-theme-25259.patch (43,8 ko) 0001-villeurbanne-2018-initial-theme-25259.patch Serghei Mihai, 12 juillet 2018 23:15
0001-villeurbanne-2018-initial-theme-25259.patch (43,7 ko) 0001-villeurbanne-2018-initial-theme-25259.patch Serghei Mihai, 12 juillet 2018 23:35
form.ong (121 ko) form.ong Serghei Mihai, 12 juillet 2018 23:38
accueil.png (201 ko) accueil.png Serghei Mihai, 12 juillet 2018 23:38
profil.png (169 ko) profil.png Serghei Mihai, 12 juillet 2018 23:38
Mes demandes.png (514 ko) Mes demandes.png Serghei Mihai, 12 juillet 2018 23:38
0001-villeurbanne-2018-initial-theme-25259.patch (43,9 ko) 0001-villeurbanne-2018-initial-theme-25259.patch Serghei Mihai, 13 juillet 2018 10:17
0001-villeurbanne-2018-initial-theme-25259.patch (43,8 ko) 0001-villeurbanne-2018-initial-theme-25259.patch Serghei Mihai, 18 juillet 2018 10:01
un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_.png (95,4 ko) un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_.png Anonyme, 21 août 2018 15:10
un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_(Galaxy S5).png (234 ko) un-connexion-authentic.dev-eshowk.ddns.entrouvert.org_accounts_edit_(Galaxy S5).png Anonyme, 21 août 2018 15:10
un-user-combo.dev-eshowk.ddns.entrouvert.org_.png (183 ko) un-user-combo.dev-eshowk.ddns.entrouvert.org_.png Anonyme, 21 août 2018 15:14
un-user-combo.dev-eshowk.ddns.entrouvert.org_(Galaxy S5).png (186 ko) un-user-combo.dev-eshowk.ddns.entrouvert.org_(Galaxy S5).png Anonyme, 21 août 2018 15:14
0001-villeurbanne-2018-initial-theme-25259.patch (43,9 ko) 0001-villeurbanne-2018-initial-theme-25259.patch Serghei Mihai, 21 août 2018 15:38
editer-compte.png (165 ko) editer-compte.png Serghei Mihai, 21 août 2018 15:39
0001-villeurbanne-2018-initial-theme-25259.patch (43,9 ko) 0001-villeurbanne-2018-initial-theme-25259.patch Serghei Mihai, 23 août 2018 17:47

Demandes liées

Lié à Intégrations graphiques Publik - Development #25260: ajouter la police BitterFermé13 juillet 2018

Actions

Révisions associées

Révision ad553c63 (diff)
Ajouté par Serghei Mihai il y a plus de 5 ans

villeurbanne-2018: initial theme (#25259)

Historique

#2

Mis à jour par Serghei Mihai il y a plus de 5 ans

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.

#3

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.

#4

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Tracker changé de Support à Development
#5

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)

#6

Mis à jour par Serghei Mihai il y a plus de 5 ans

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)

#7

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.

#8

Mis à jour par Serghei Mihai il y a plus de 5 ans

#9

Mis à jour par Serghei Mihai il y a plus de 5 ans

Ok.
Police "Bitter" déclarée, mais chargeant le fichier de la police grasse. Ça fonctionne.
Dependance du ticket #25260.

#10

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.

#11

Mis à jour par Serghei Mihai il y a plus de 5 ans

Avec la déclaration correcte de la police Bitter dans #25260, plus besoin de rajouter une déclaration ici.

#12

Mis à jour par Anonyme il y a plus de 5 ans

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 ?)
    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.
#13

Mis à jour par Serghei Mihai il y a plus de 5 ans

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.

#14

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

Ç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)

Chromium.
Ça vient surement de l'utilisation d'un padding en "ex" sur h2:first-child. Côté code :
  • 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.
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;
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 258 box-shadow: 0 0 10px 0 silver; ?
#15

Mis à jour par Serghei Mihai il y a plus de 5 ans

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.

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 258 box-shadow: 0 0 10px 0 silver; ?

Yep, corrigé.

#16

Mis à jour par Anonyme il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Go

#17

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)
#18

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

Formats disponibles : Atom PDF