search

themesdefrance / Galopin

1 stars 2 forks source link

Responsive à améliorer #7

Closed themesdefrance closed 10 years ago

themesdefrance commented 10 years ago

capture d ecran 2014-09-15 a 08 26 30

y-lohse commented 10 years ago

C'est en masonry que ca t'arrive? Parce que si oui c'est probablement une duplication de #6.

themesdefrance commented 10 years ago

Oui, c'est ça. Du coup je clos le ticket.

themesdefrance commented 10 years ago

En fait non, le rendu n'est pas top pour les pages avec sidebar : capture d ecran 2014-10-21 a 14 49 05

themesdefrance commented 10 years ago

Avec l'image d'accueil en grand, on a ça sur mobile : capture d ecran 2014-10-21 a 16 51 37

y-lohse commented 10 years ago

Je ne vois pas de problème avec les 2 images.

Sur la première,la colonne est un peu étroite mais entre le avec/sans colonne et les différents breakpoints, c'est le meilleur compromis je pense.

La seconde est juste normale à mon sens.

themesdefrance commented 10 years ago

Yop, Pour la première, la sidebar est effectivement un peu trop étroite. Sur Étendard ça passe beaucoup mieux je trouve. L'idéal serait de rajouter un breakpoint intermédiaire je pense.

Il faudra que je mette la miniature en height auto car c'est pété.

En format mobile, je pense qu'il serait mieux de mettre le menu en haut à gauche et le titre au milieu comme en version normale. Je vais faire ça ce soir ou demain.

themesdefrance commented 10 years ago

Height auto miniatures ea8fd86

themesdefrance commented 10 years ago

Optimisations du menu 45e8a68 Par contre, le menu en stuck merde un peu pour bien coller à la barre d'admin de WP en responsive. J'ai mis les medias queries mais ça ne marche pô

y-lohse commented 10 years ago

Je ne suis d'accord avec aucune des 2 modifs^^.

Pour la première, si on ajoute un breakpoint supplémentaire plus large, l'affichage sans sidebar sera dégueu avec des lignes ultra longues.Ta capture d'écran est faite dans la zone des 1000px de large, qui est effectivement la pire. Au dessus de 1000 ca redeviens bien, et en dessous de 800 c'est tout à fait raisonnable.

Pour le menu en responsive, je suis meme totalement opposé aux changements. D'une part parce que l'affiche courant est très bien comme il est: Il est cohérent avec les pages ou il n'y a pas de hero image et il ne force pas à scroller jusqu'en haut pour avoir accès au bouton menu. A coté de ca, le css de la partie hero + menu + responsive est déja suffisament complexe, ajouter des comportements différents selon les résolutions va rendre le truc inmaintenable (cf. 45e8a685cb9138e7601054316762633b6b1ead5f et c'est qu'un début).

themesdefrance commented 10 years ago

Re, J'ai zappé de répondre à ce ticket :/

Pour te répondre, je trouve que pour la zone entre les 1000 et 800, 500px de large pour la totalité du contenu ça fait un peu short. C'est trop tard pour la release de la v1, mais je regarderai pour améliorer ça ensuite.

Je vois ce que tu veux dire pour le menu mais avoir les deux items au milieu ça fait bizarre. Le top serait d'avoir le bouton du menu en haut à gauche (comme sur tous les thèmes en fait) et le lien texte au milieu de l'écran comme en version desktop.

Du coup on se rapprocherait plus de ce qu'il y avait à la base (le bouton de menu change juste de place et le lien est centré).

Pas grave si ce n'est pas pour la V1

y-lohse commented 10 years ago

Comme vu en réu,dans l'immédiat on recentre l'icone et le lien;dans une prochaine release on pourra essayer de mettre l'icone du menu en haut et centrer le lien.

Pour le premier point, voir si simplement ajouter un breakpoint ne cause pas de problemes collateraux.

themesdefrance commented 10 years ago

:+1:

y-lohse commented 10 years ago

Ok pour le breakpoint en plus. Je crée un nouveau ticket pour le header.

themesdefrance commented 10 years ago

Super :)