Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embed vidéos #37

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Embed vidéos #37

wants to merge 3 commits into from

Conversation

thom4parisot
Copy link
Contributor

Première ébauche, pour envisager un meilleur rendu.

screen shot 2015-09-23 at 00 08 13
screen shot 2015-09-23 at 00 08 28

close #36

@DirtyF
Copy link
Contributor

DirtyF commented Sep 23, 2015

@oncletom merci, c'est cool. Pour nous simplifier la vie je suis d'avis de ne pas afficher les vidéos sur les résolutions ou ça casse. En gros pour le moment on le laisse que sur Desktop, y'a les liens vers les vidéos par défaut.

https://www.dropbox.com/s/nmwa7rol3wfh55x/embed-videos-rwd.png?dl=0

Pour le rendu, peut-être cela implique t-il de penser ou utiliser un système de grille ou à minima de mieux utiliser Flexbox ? Une mission pour @Twikito ?

@Twikito
Copy link

Twikito commented Sep 23, 2015

@oncletom @DirtyF > Vous aider, aucun souci, mais il faudrait que je sois au courant du problème (je n'étais pas là dès le début) ;-)

@thom4parisot
Copy link
Contributor Author

@Twikito cf. les screenshots au-dessus : c'est un premier jet pour intégrer les vidéos au site. Preneur de conseils en terme de disposition (où et quelle(s) taille(s) – responsive in mind).

@Twikito
Copy link

Twikito commented Sep 26, 2015

@oncletom J'avoue que je ne suis pas très fan de la disposition des informations, mais ce sera suffisant pour un premier jet, et surtout facile à responsiver. Ceci dit, est-ce vraiment nécessaire de mettre les vidéos sur le site vu que les liens y sont ?
Comme je le disais à @DirtyF sur Slack, peut-être est-ce une bonne occasion de revoir le site en entier ? Je pourrai m'occuper de l'intégration bien sûr.

@thom4parisot
Copy link
Contributor Author

@Twikito commence déjà par une redisposition des infos - ça sera toujours plus rapide que refondre le site ;-) (et si on refond, c'est itérativement, pas entièrement d'un coup).

Et j'ai jeté les vidéos comme ça, sans toucher au reste — on peut virer les liens, les placer autrement : c'est pas à merger en l'état, c'est pas esthétique. C'est également plus rapide de proposer des améliorations juste pour ce scope ci, et pas refondre complètement un ensemble de page.

Petit puis grand ensuite.

@Twikito
Copy link

Twikito commented Sep 28, 2015

@oncletom > Je comprends bien sûr l'intention de procéder en itératif, mais je dois dire que je ne suis pas d'accord avec toi. De mon humble avis, c'est justement le meilleur moment pour repenser et travailler sur une nouvelle version du site : entre le post-event 2015 et le pré-event 2016.
Nous sommes d'accord qu'il faut améliorer la communication de ses côtés. Pourquoi ne pas redorer l'image de Sud Web en commençant par le site ?

@DirtyF
Copy link
Contributor

DirtyF commented Sep 28, 2015

same old story... ça part en hors sujet 😄

Tu as raison @Twikito nous aurons besoin de revoir les fondations pour le prochain, le système actuel est pas assez souple notamment en terme de mise en page, de grille (la preuve avec cette PR), de documentation (style guide). C'est le bon moment pour s'attaquer à des chantiers de fond, sans pour autant perdre les avantages du système actuel (Github pages, scripts npm). Mais c'est une autre PR ça 😄

Là présentement, le besoin était juste de voir si on peut améliorer la présentation des vidéos.
Si la réponse est non, pas avec le système actuel avec lequel @Twikito n'est pas familier, OK.

@thom4parisot
Copy link
Contributor Author

Yep, pour le moment l'idée est d'améliorer ce bloc de présentation (des patterns à suggérer ? des erreurs ou pièges responsive ?)

On est preneur d'améliorations — réagir juste en négation ça fait pas avancer le shmilblick.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lazyload
3 participants