search

Unitystation-fork / Unitystation-MillstoneSteam

Objectif Stream coop
GNU Affero General Public License v3.0
1 stars 1 forks source link

Issue 28 / Bouton retour en haut de page #38

Closed Warwiren closed 1 year ago

Warwiren commented 1 year ago

Mise en place d'un bouton retour en haut de page / avec utilisation d'une image

Fr-Dae commented 1 year ago

@Clemy-beep peux t'on merge ceci ?

Clemy-beep commented 1 year ago

@Fr-Dae Bah non, puisque y a pas d'indication que le code est approuvé. Quand ce sera le cas tu en seras notifié par github.

Fr-Dae commented 1 year ago

@Fr-Dae Bah non, puisque y a pas d'indication que le code est approuvé. Quand ce sera le cas tu en seras notifié par github.

que faut t'il pour appouver le code ?

Clemy-beep commented 1 year ago

Que je le review et l'approuve. Donc on ronge son frein et on PATIENTE. Pour info, je garde un oeil sur les pull requests donc ça sert a rien de demander "on peut merge" comme si j'avais oublié.

Clemy-beep commented 1 year ago

D'ailleurs c'est une pull request foirée. @Warwiren, regarde les commits et dis moi s'il manque pas un truc. Genre TON CODE ? Le sens de la pull me semble absurde aussi (de dev vers ta branche). Le nom de la pull est viiblement erroné.

Warwiren commented 1 year ago

Je viens de regarder la branche, alors le code que j'ai mis est directement mis dans app.vue (+ l'image dans le dossier image)

Clemy-beep commented 1 year ago

@Warwiren pourquoi c'est pas dans les commits ? S'il y a effectivement le code du bouton je peux pas le review. Cette PR est invalide. Elle sera close. Indice pour en faire une correcte : inverse le sens du merge.