From 86656c49420c2d4c8232a7f21ccab6a2f90a77c3 Mon Sep 17 00:00:00 2001
From: Alberto Xamin <alberto@xamin.it>
Date: Wed, 16 Jun 2021 18:14:44 +0200
Subject: [PATCH 1/2] wait to close and other quality of life fixes

---
 backend/bang/game.py              | 14 +++++++-----
 frontend/src/App.vue              |  2 +-
 frontend/src/components/Lobby.vue | 38 +++++++++++++++----------------
 3 files changed, 28 insertions(+), 26 deletions(-)

diff --git a/backend/bang/game.py b/backend/bang/game.py
index 7c9de12..a964b2a 100644
--- a/backend/bang/game.py
+++ b/backend/bang/game.py
@@ -449,12 +449,14 @@ class Game:
         if self.disconnect_bot and self.started:
             player.is_bot = True
             if len([p for p in self.players if not p.is_bot]) == 0:
-                print(f'no players left in game {self.name}, shutting down')
-                self.shutting_down = True
-                self.players = []
-                self.spectators = []
-                self.deck = None
-                return True
+                eventlet.sleep(5)
+                if len([p for p in self.players if not p.is_bot]) == 0:
+                    print(f'no players left in game {self.name}, shutting down')
+                    self.shutting_down = True
+                    self.players = []
+                    self.spectators = []
+                    self.deck = None
+                    return True
             eventlet.sleep(15) # he may reconnect
             if player.is_bot:
                 if len(player.available_characters) > 0:
diff --git a/frontend/src/App.vue b/frontend/src/App.vue
index 305afdc..d556899 100644
--- a/frontend/src/App.vue
+++ b/frontend/src/App.vue
@@ -153,7 +153,7 @@ html {
 	#app {
 		margin: 4pt;
 		margin-top: -16pt;
-		zoom: 0.8;
+		zoom: 0.75;
 	}
 }
 h1,h2,h3,h4,p,span,b,label{
diff --git a/frontend/src/components/Lobby.vue b/frontend/src/components/Lobby.vue
index 9508ff2..3324af6 100644
--- a/frontend/src/components/Lobby.vue
+++ b/frontend/src/components/Lobby.vue
@@ -17,6 +17,25 @@
 			
 			<!-- <div class="players-table"> -->
 				<!-- <div style="position: relative;width:260pt;height:400pt;"> -->
+					<!-- :style="p.style"/> -->
+				<!-- </div> -->
+			<!-- </div> -->
+			<div v-if="!started">
+				<h3>{{$t("expansions")}}</h3>
+				<div v-for="ex in expansionsStatus" v-bind:key="ex.id">
+					<PrettyCheck @click.native="toggleExpansions(ex.id)" :disabled="!isRoomOwner" :checked="ex.enabled" class="p-switch p-fill" style="margin-top:5px; margin-bottom:3px;">{{ex.name}}</PrettyCheck>
+					<br>
+				</div>
+				<h3>{{$t('mods')}}</h3>
+				<PrettyCheck @click.native="toggleCompetitive" :disabled="!isRoomOwner" v-model="is_competitive" class="p-switch p-fill" style="margin-top:5px; margin-bottom:3px;">{{$t('mod_comp')}}</PrettyCheck>
+				<h3>{{$t('bots')}}</h3>
+				<input type="button" class="btn" :value="$t('add_bot')" :disabled="!isRoomOwner || players.length > 7" @click="(e)=>{this.$socket.emit('chat_message', '/addbot'); e.preventDefault()}"/>
+				<input type="button" class="btn" style="margin-left: 10pt;" :value="$t('remove_bot')" :disabled="!isRoomOwner || !isThereAnyBot" @click="(e)=>{this.$socket.emit('chat_message', '/removebot'); e.preventDefault()}"/>
+				<!-- <br> -->
+				<!-- <PrettyCheck @click.native="toggleReplaceWithBot" :disabled="!isRoomOwner" v-model="disconnect_bot" class="p-switch p-fill" style="margin-top:5px; margin-bottom:3px;">{{$t('disconnect_bot')}}</PrettyCheck> -->
+				<p v-if="players.length < 3" class="center-stuff" style="min-height: 19px;">{{$t('minimum_players')}}</p>
+				<p v-else style="min-height: 19px;"> </p>
+			</div>
 			<transition-group name="list" tag="div" class="players-table">
 				<Card v-if="startGameCard" key="_start_game_" :donotlocalize="true" :card="startGameCard" @click.native="startGame"/>
 				<div v-for="p in playersTable" v-bind:key="p.card.name" style="position:relative;">
@@ -42,25 +61,6 @@
 					</div>
 				</div>
 			</transition-group>
-					<!-- :style="p.style"/> -->
-				<!-- </div> -->
-			<!-- </div> -->
-			<div v-if="!started">
-				<p v-if="players.length < 3" class="center-stuff" style="min-height: 19px;">{{$t('minimum_players')}}</p>
-				<p v-else style="min-height: 19px;"> </p>
-				<h3>{{$t("expansions")}}</h3>
-				<div v-for="ex in expansionsStatus" v-bind:key="ex.id">
-					<PrettyCheck @click.native="toggleExpansions(ex.id)" :disabled="!isRoomOwner" :checked="ex.enabled" class="p-switch p-fill" style="margin-top:5px; margin-bottom:3px;">{{ex.name}}</PrettyCheck>
-					<br>
-				</div>
-				<h3>{{$t('mods')}}</h3>
-				<PrettyCheck @click.native="toggleCompetitive" :disabled="!isRoomOwner" v-model="is_competitive" class="p-switch p-fill" style="margin-top:5px; margin-bottom:3px;">{{$t('mod_comp')}}</PrettyCheck>
-				<h3>{{$t('bots')}}</h3>
-				<input type="button" class="btn" :value="$t('add_bot')" :disabled="!isRoomOwner || players.length > 7" @click="(e)=>{this.$socket.emit('chat_message', '/addbot'); e.preventDefault()}"/>
-				<input type="button" class="btn" style="margin-left: 10pt;" :value="$t('remove_bot')" :disabled="!isRoomOwner || !isThereAnyBot" @click="(e)=>{this.$socket.emit('chat_message', '/removebot'); e.preventDefault()}"/>
-				<!-- <br> -->
-				<!-- <PrettyCheck @click.native="toggleReplaceWithBot" :disabled="!isRoomOwner" v-model="disconnect_bot" class="p-switch p-fill" style="margin-top:5px; margin-bottom:3px;">{{$t('disconnect_bot')}}</PrettyCheck> -->
-			</div>
 			<div v-if="started">
 				<deck :endTurnAction="()=>{wantsToEndTurn = true}"/>
 				<player :isEndingTurn="wantsToEndTurn" :cancelEndingTurn="()=>{wantsToEndTurn = false}" :chooseCardFromPlayer="choose" :cancelChooseCardFromPlayer="()=>{hasToChoose=false}"/>

From dab5718eab591175af4f958c48b20faded695826 Mon Sep 17 00:00:00 2001
From: Alberto Xamin <alberto@xamin.it>
Date: Wed, 16 Jun 2021 20:27:39 +0200
Subject: [PATCH 2/2] command suggestion

---
 backend/bang/game.py             |  5 +++++
 frontend/src/components/Chat.vue | 14 ++++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/backend/bang/game.py b/backend/bang/game.py
index a964b2a..4e14807 100644
--- a/backend/bang/game.py
+++ b/backend/bang/game.py
@@ -77,6 +77,11 @@ class Game:
                 'available_expansions': self.available_expansions,
             })
         self.sio.emit('debug', room=self.name, data=self.debug)
+        if self.debug:
+            commands = ['/debug', '/set_chars', '/suicide', '/nextevent', '/notify', '/show_cards', '/ddc', '/dsh', '/togglebot', '/cancelgame', '/startgame', '/setbotspeed', '/addex', '/setcharacter', '/setevent', '/removecard', '/getcard', '/meinfo', '/gameinfo', '/mebot']
+            self.sio.emit('commands', room=self.name, data=commands)
+        else:
+            self.sio.emit('commands', room=self.name, data=['/debug'])
         self.sio.emit('spectators', room=self.name, data=len(self.spectators))
 
     def toggle_expansion(self, expansion_name):
diff --git a/frontend/src/components/Chat.vue b/frontend/src/components/Chat.vue
index ac30d90..5f9372e 100644
--- a/frontend/src/components/Chat.vue
+++ b/frontend/src/components/Chat.vue
@@ -5,6 +5,7 @@
 		<transition-group name="message" tag="div" id="chatbox">
 		<!-- <div id="chatbox"> -->
 			<p style="margin:1pt;" class="chat-message" v-for="(msg, i) in messages" v-bind:key="`${i}-c`" :style="`color:${msg.color}`">{{msg.text}}</p>
+			<p style="margin:1pt 15pt;" class="chat-message" v-for="(msg, i) in commandSuggestion" v-bind:key="`${i}-c`" :style="`color:orange`">{{msg}}</p>
 			<p class="end" key="end" style="color:#0000">.</p>
 		<!-- </div> -->
 		</transition-group>
@@ -31,7 +32,17 @@ export default {
 		messages: [],
 		text: '',
 		spectators: 0,
+		commands: ['/debug'],
 	}),
+	computed: {
+		commandSuggestion() {
+			this.text;
+			if (this.text.length < 1) {
+				return [];
+			}
+			return this.commands.filter(x => x.slice(0, this.text.length) == this.text);
+		},
+	},
 	sockets: {
 		chat_message(msg) {
 			// console.log(msg)
@@ -84,6 +95,9 @@ export default {
 		},
 		spectators(val) {
 			this.spectators = val
+		},
+		commands(list) {
+			this.commands = list;
 		}
 	},
 	methods: {