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

[beta][v16][rc1] L'affichage d'une partie d'un contenu public est cassé #3315

Closed
firm1 opened this issue Jan 21, 2016 · 11 comments
Closed

[beta][v16][rc1] L'affichage d'une partie d'un contenu public est cassé #3315

firm1 opened this issue Jan 21, 2016 · 11 comments
Labels
C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant

Comments

@firm1
Copy link
Contributor

firm1 commented Jan 21, 2016

Version : 16-RC1

Prod

lien

https://zestedesavoir.com/tutoriels/757/la-fabrication-numerique/1079_arduino/

screen

good-beta

Beta

lien

https://beta.zestedesavoir.com/tutoriels/714/la-fabrication-numerique/1079_arduino/

screen

bad-beta

Comme on peut le voir sur la version beta, l'affichage ne va pas en s’arrangeant.

It's smell like a regression

@firm1 firm1 added C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant labels Jan 21, 2016
@sandhose
Copy link
Contributor

Problème côté templates, le markup est pas le même: http://imgur.com/a/crqcP

A priori, c'est ce commit le fautif (à vérifier, c'est vraiment chelou): 2238c21

@pierre-24
Copy link
Member

Tordu, j'était convaincu d'avoir QA ça correctement (mais c'était bien une PR d'artragis)

@artragis
Copy link
Member

ça va être chaud à corriger ça.

@artragis
Copy link
Member

Comme on peut le voir sur la version beta, l'affichage ne va pas en s’arrangeant.

Bon, pour revenir sur cette petite phrase je voudrai quand même qu'on réfléchisse un peu à ce uqe c'est que "s'arranger".

Actuellement passer de la page "sommaire général" à la page "partei" impose un changement de style fort dans la présentation des titres. Sur la béta on a une certaines cohérence.

Attention, je ne nie pas qu'il y a un bug : les titres sont trop gros. Mais là je trouve qu'on a été dans le sens de l'unification et non d'une régression. Pire c'est exactement ce vers quoi on voulait aller quand j'ai pris en charge ce bug.

Perso j'enlèverai le tag "régression" et mettrais "bug". Je vais tenter de voir avec @sandhose pour voir pourquoi alors qu'on a tout fait pour unifier les tags html on se retrouve avec des trucs qui ont des tailles différentes.

En attendant @GerardPaligot et @pierre-24 vous qui êtes les nouveaux chefs de projets, pensez vous qu'on soit dans une régression du point de vue UX (pdv firm1) ou dans une unification buggée (pdv moi même)

@pierre-24
Copy link
Member

Pire c'est exactement ce vers quoi on voulait aller quand j'ai pris en charge ce bug.

Sans vouloir défendre Firm1, c'est effectivement ce qu'on voulait pour le sommaire des moyen-tutos, or le screen est issus du sommaire d'un big-tuto (pas de celui de la partie d'un big tuto), et du coup ... L'ancien affichage était bien le bon, c'est donc bien une régression :)

Il serait juste bon de faire des rapport de bugs plus explicites :)

@GerardPaligot
Copy link
Member

Hormis le problème de front évident dans les titres et les liens soulignés d'office, ta question @artragis est légitime. Par contre, j'ai peur que tu ne proposes pas de revenir à un affichage prod quand on consulte un big tutoriel.

Grillé par @pierre-24. :)

@artragis
Copy link
Member

D'où ma question. Si vous pensez qu'on doit garder l'affichage prod pour
les big tuto on agira en conséquence, j'ai juste pas envie de me casser la
tête ou d'embêter les devs front pour développer un truc dont on n'a pas
envie à la fin.

Le 28 janvier 2016 à 14:04, Gérard Paligot notifications@github.com a
écrit :

Hormis le problème de front évident dans les titres et les liens soulignés
d'office, ta question @artragis https://github.com/artragis est
légitime. Par contre, j'ai peur que tu ne proposes pas de revenir à un
affichage prod
https://cloud.githubusercontent.com/assets/6066015/12481973/17738320-c04d-11e5-966a-379d0585f3d3.PNG
quand on consulte un big tutoriel.

Grillé par @pierre-24 https://github.com/pierre-24. :)


Reply to this email directly or view it on GitHub
#3315 (comment)
.

@GerardPaligot
Copy link
Member

Je te confirme. Pour les bigs tutos, on garde le visuel actuel de la prod. Pour les moyens tutos, on propose le visuel que tu proposes de la bêta.

@sandhose
Copy link
Contributor

Ca me parait assez incohérent niveau UX comme choix, en plus du fait que AMHA, la visualisation d'un conteneur reste une vue très peu utilisée par les utilisateur (ça sert en gros dans le fil d'ariane ; je pense pas qu'un utilisateur lambda clique sur une partie directement au lieu de directement sur le chapitre)

@SpaceFox
Copy link
Contributor

Personnellement je préfère la cohérence de l'expérience utilisateur à un affichage iso-prod.

@firm1
Copy link
Contributor Author

firm1 commented Jan 30, 2016

J'ai l'impression que tout le monde ne comprend pas le problème. Si je fais
un récap ça donne :

  • Aujourd'hui en prod on a :
    • Pour les big tuto :
      • le sommaire du tuto s'affiche sur 2 colonnes
      • le sommaire d'une partie s'affiche sur 2 colonnes [c'est ce
        cas qui est concerné par mon ticket]
      • le sommaire du chapitre est dans l'introduction.
    • Pour les moyens tuto:
      • le sommaire du tuto s'affiche sur 1 colonnes ('fin c'est
        censé etre ainsi car il y'a un bug dessus pour les tutos online)
      • le sommaire d'un chapitre s'affiche dans l'introduction
    • Pour les minitutos:
      • le sommaire du tuto s'affiche dans l'introduction.

Le truc bien avec l'affichage du sommaire sur 2 colonnes c'est qu'on n'a
pas besoin de scroller sur les petits écrans et c'est pour cela que pour
moi le comportement idéal serait de rester sur 2 colonnes. Pour moi,
uniformiser l'UX signifie rester sur 2 colonnes pour les sommaires. Les
sommaires sur 1 colonnes (dont j'ai du mal a voir l'intéret) sont apparus
avec l'arrivée des moyens tuto (jusqu'ici ça ne faisait pas parti de l'UX)

Le sam. 30 janv. 2016 17:08, SpaceFox notifications@github.com a écrit :

Personnellement je préfère la cohérence de l'expérience utilisateur à un
affichage iso-prod.


Reply to this email directly or view it on GitHub
#3315 (comment)
.

gustavi pushed a commit that referenced this issue Feb 9, 2016
Corrige le sommaire des conteneurs de parties (fix #3315)
@gustavi gustavi closed this as completed Feb 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

No branches or pull requests

7 participants