search

JConcept / CV-Kinepolis

:school: My CV based on webdesign of Kinepolis Belgium. This is an school project for HEPL in "Design Web" class.
0 stars 0 forks source link

Amélioration de mon projet (only in french, sorry) #2

Open JConcept opened 7 years ago

JConcept commented 7 years ago

Bonjour M. @dominiquevilain et Mme @myriamdupont,

J'ai pris une petite avance sur GitHub comme vous pouvez le voir.

J'ai effectué les modifications que vous m'aviez conseillées pour mon CV du kinepolis. Si vous avez d'autres remarques et améliorations à proposer pour mon projet, n'hésitez pas à m'en faire part ici.

J'ai bien entendu publié mon travail sur le serveur de l'école : http://hepl01.cblue.be/~user18/dw/cv/

Merci de m'aider dans ce projet, Cordialement, LEPOT Jean-Christophe

dominiquevilain commented 7 years ago

capture d ecran 2016-11-23 a 09 36 08

Une page de 8,1 M, c’est difficilement justifiable. Il faut optimiser les images et les utiliser dans leur bonne taille.

Il y a deux 404 dans la liste des ressources nécessaires (deux polices) -> à corriger

On pourrait sans doute diminuer le nombre de requêtes HTTP en utilisant des sprites et en combinant les deux feuilles de styles en une seule.

Les select sont encore dépendants du chrome du navigateur. Je sais que JS est nécessaire pour passer outre, mais c’est une occasion d’essayer. Vois ce que tu peux faire.

La photo de toi ne va pas dans le sens du concept. Tu devrais plutôt faire une sorte d‘affiche de film qui te met en scène. Tant qu’à faire, joue le concept à fond et fais nous une parodie de Last Action Hero où ta tête remplace celle de Schwarzie. ET du coup, utilise cette affiche comme ils le font sur le site du Kinépolis, comme un poster horizontal.

L’intégration des logos pour les technos pose le même problème. Je pense qu’on pourrait aller dans le même sens, assumer le concept et faire un poster de film pour chaque techno, à utiliser en vertical cette fois, comme sur le site de Kinepolis quand ils te font des suggestions de films. Dans le bloc « tuto », il y a un problème avec le padding du bloc. C’est un peu serré.

Voilà déjà quelques remarques, là rapidement.

JConcept commented 7 years ago

Ah oui en termes de performances je n'avais pas encore pensé à optimiser, ni à vérifier si toutes les requêtes aboutissaient correctement... Je l'ajoute au todo, mais que voulez-vous dire par "Il faut optimiser les images et les utiliser dans leur bonne taille." ? Je dois proposer des images différentes en fonction de si on se trouve sur mobile ou sur desktop ? On doit utiliser JS pour y arriver non ?

C'est noté, j'ajoute aussi les sprites au todo.

Je n'ai pas bien compris, les select ?

Les select sont encore dépendants du chrome du navigateur. Je sais que JS est nécessaire pour passer outre, mais c’est une occasion d’essayer. Vois ce que tu peux faire.

Bonne idée de pousser le concept à fond, je n'y avais pas du tout pensé... Cela va me faire un gros todo sur le travail des images.

Pour les technos ça va être chaud d'essayer de les transformer en affiche de film, mais avec un peu de créativité, il y a surement moyen d'y arriver.

Merci pour les remarques !