Add card template for embedded contents #202
Conversation
| overflow: hidden; | ||
|
|
||
| flex: 1 0 $news-card-size; | ||
| flex-direction: column; |
There was a problem hiding this comment.
Pas trop sur de ça, j'en avais besoin pour que news-card__caption stack en dessous pour les display moyens - @Nhacsam ?
There was a problem hiding this comment.
Du coup, l'image se retrouve toujours seule sur sa ligne au format mobile ? Je trouve ça un peu dommage.
Ce que j'avais fait, c'est que je faisait descendre le texte à partir de 400px grâce au flex-wrap dans @media (max-width: 400px).
Dans tous les cas, tu dois pouvoir mettre ça dans le @media (max-width: $screen-xs-max).
Si ça rend vraiment pas les embeded content a gauche du texte, je pense que le mieux c'est que fasse une exception pour ce type de contenu avec un modifier genre :
.news-card--embed-content {
flex-direction: column;
}There was a problem hiding this comment.
Je n'avais pas vu que c'était le design souhaité. Je regarde demain, il faut juste un peu de CSS pour ajouter du padding sur l'iframe je crois.
Yes pour le BEM, je vais essayer de plus le respecter (et surtout lire la spec une bonne fois pour toutes). Par contre le lien que tu as mis contredit ton snippet, la spec dit d'utiliser un _ :p (.news-card_embed-content ?)
There was a problem hiding this comment.
J'utilise le Harry Roberts' style ^^
Mais je t'avouerai que je l'applique pas parfaitement sur nuit debout :/ Le Css n'est pas flat, bootstrap me gène et je suis obligé de faire des exceptions pour les partie du html qu'on ne contrôle pas (comme le contenu des posts ou les menus)
|
Ça en est où du coup ? Moyen de voir un screenshot ? |
|
Oui je le fais ce soir, il y a p-e un petit glitch sur les screens tablets. |
No description provided.