Nouvelle page de profil

[A-312]

Suite de : #5025

Prêt à R/Q.

[A-312]

[coveralls]

Coverage decreased (-15.5%) to 58.819% when pulling c6881d0 on A-312:profilv2 into 8357aed on zestedesavoir:dev.

[A-312]

Vous pouvez Q/A ;)

[A-312]

[A-312]

j'ai oublié :

[A-312]

Je n'arrive pas à reproduire la situation ci-dessus

[A-312]

https://zestedesavoir.com/forums/sujet/9222/nouvelle-page-de-profil/?page=3#p199540

• Ajout margin top
• Fusionner les boutons de suivi.

[A-312]

Je ne comprends pas : https://travis-ci.org/zestedesavoir/zds-site/jobs/503941521

J'ai simplement déplacé les hats sans les enlever.

[artragis]

ça vient de là : https://github.com/zestedesavoir/zds-site/pull/5284/files#diff-f7c70450a4d0c15961dfe9554ad685a2L113

[A-312]

La barre de recherche n'a pas le bon icône

Le sprite est décalé il me semble. Normalement, c'est l'icone loupe de la page d'accueil

Nos éléments Markdown ne s'affichent pas correctement dans la biographie

Pareil que le premier point

Les boutons S'abonner et S'abonner par courriel sont en vert au survol. Je ne trouve pas ça intuitif ou beau, je propose pour qu'on mette du bleu, bien entendu une nuance différence que celle de base

Il faut les modifier comme Armaury l'a proposé

Le bouton Modifier le profil ne change pas de couleur de fond au survol. Je pense qu'il faut une deuxième nuance de bleu

Idem voir commentaire d'armaury, il manque certaine chose

Je trouve dommage que l'avatar disparaisse en version mobile. Pareil pour les boutons S'abonner et S'abonner par courriel

C'est pour rester cohérent avec la bibliothèque

#5284 (comment)

[artragis]

On en est où

[Situphen]

@artragis J'ai fait une PR sur le dépôt d'@A-312 (A-312#14) avec plusieurs modifications. Avec cette PR, je pense que du point de vue fonctionnel on est pas mal ! Au niveau du code source, j'ai des choses à dire sur le code SCSS mais je manque de temps pour faire les modifications. :/ Je vais essayer de dégager du temps et pour à minima en discuter avec @A-312 mais si je n'y arrive pas alors on pourra merger (cette PR commence à dater). Par contre, les tests back ne passent pas donc il faut corriger ça.

[artragis]

ok @A-312 si tu bloques sur les tests backend préviens-moi je ferai une petite investigation

[A-312]

Je pensais que ça allait garder tes commits en entier et ton pseudo. :/ Je pensais avoir choisi le bon choix dans la selectbox quand j'ai merge la pr

[Situphen]

Si mes commits ont été gardés, il y a juste un commit de merge qui a été rajouté. :)

[A-312]

Je viens de les voir. Je n'avais pas scrollé assez haut. Quel comble pour un avion. 😭

[artragis]

Du coup tu t'en sors?

[A-312]

Je pensais faire quelques choses, je n'ai rien fais et je ne peux pas m'en occuper dans les prochains jours. 😞

[artragis]

On en est où?

[artragis]

On en est où ici? Avez vous besoin d'aide?

[A-312]

je n'ai plus d'instance de dev

[Situphen]

Je m'en occupe cette semaine.

[Situphen]

Étant donné que @A-312 n'a plus d'instance de dev et que j'ai des nouvelles modifications à apporter, je me suis permis de créer une nouvelle PR (en gardant les commits actuels) : #5350