Amélioration de la lisibilité du chat

Lesterpig / openparty-mafia

An openparty extension for Mafia role game

GNU General Public License v3.0

4 stars 4 forks source link

Amélioration de la lisibilité du chat #10

Closed HemnesLGeL closed 9 years ago

HemnesLGeL commented 9 years ago

Ajout de phrases pour le tour des spés Ajout de boites pour le tour des spés Modification des votes

Lesterpig commented 9 years ago

Hum ça ne fonctionne pas (nombreuses erreurs de syntaxe) :/ Pourrais tu tester et corriger en local ? Je vais pas tarder à poster une documentation plus complète pour l'installation et la configuration.

HemnesLGeL commented 9 years ago

Je vais regarder ça dans l'après-midi :)

HemnesLGeL commented 9 years ago

Corrigé, c'était une bête erreur de guillemets ^^

Lesterpig commented 9 years ago

Je regarde ça demain en détails :) Par contre plusieurs petits trucs :

Pourrais tu regarder comment s'utilise git ? Tu dois faire les modifications depuis le site web, mais du coup ça alourdit pas mal la pull request et c'est difficile à relire ...
ça se rapproche trop de LGeL niveau style :p pas besoin de mettre des "Privés" par exemple
les messages "Les vigiles se réveillent" n'ont de sens que si des vigiles sont présents dans la partie. Du coup ça rejoint un peu l'issue #16 on peut utiliser ça pour rappeler la composition :)

HemnesLGeL commented 9 years ago

J'ai utilisé git correctement ? :)

Mise en valeur de la lisibilité du chat (boites pour la mafia, modification du style des votes, modification du style de l'annonce de la mort)
Rappel de la composition au lancement de la partie
Ajout de phrases lorsque la mafia et le village n'éliminent personne.

Lesterpig commented 9 years ago

Pas mal :) J'ai bien mis à jour la version officielle avec ton patch (manuellement) : https://github.com/Lesterpig/openparty-mafia/commit/5e17a1d8d03b2c9a604fa1466d674dc6c2678a26

Super boulot ! Quelques remarques :

Essaye de configurer ton éditeur de texte pour qu'il remplace les tabulations par deux espaces (notepad++ le fait très bien par exemple). Sinon ça décale le code sur certains éditeurs (comme le mien ^^ )
Ta branche de git semble "désynchronisée". Ca m'a donné un peu de travail supplémentaire, mais rien de bien grave. Ca devient assez technique, mais si ça t'intéresse tu peux toujours regarder dans la documentation GitHub pour refaire une synchronisation : https://help.github.com/articles/syncing-a-fork/

Merci encore !