From 9b55ed47e79fa3d01b58d33eff212102b764688b Mon Sep 17 00:00:00 2001
From: JannisX11 <jannis4236@gmail.com>
Date: Fri, 22 Jan 2021 21:31:57 +0100
Subject: [PATCH] Checkbox icon for wireframe toggle

---
 js/edit_sessions.js   | 7 +------
 js/interface/menu.js  | 1 +
 js/preview/preview.js | 3 ++-
 3 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/js/edit_sessions.js b/js/edit_sessions.js
index 49bbf6c6..b0106c54 100644
--- a/js/edit_sessions.js
+++ b/js/edit_sessions.js
@@ -324,12 +324,7 @@ EditSession.Client = class {
 
 const Chat = {
 	history: [],
-	expanded: true,
 	maxlength: 512,
-	toggle() {
-		this.expanded = !this.expanded;
-		BarItems.toggle_chat.setIcon( Chat.expanded ? 'keyboard_arrow_down' : 'keyboard_arrow_up' )
-	},
 	send(text) {
 		if (typeof text !== 'string') {
 			text = $('input#chat_input').val()
@@ -444,7 +439,7 @@ Interface.definePanels(function() {
 			template: `
 				<div>
 					<div class="bar next_to_title" id="chat_title_bar"></div>
-					<ul id="chat_history" v-if="expanded">
+					<ul id="chat_history">
 						<li v-for="msg in history">
 							<b v-if="msg.showAuthor()" v-bind:class="{self: msg.self}">{{ msg.author }}:</b>
 							<span class="text" v-bind:style="{color: msg.hex || 'inherit'}" v-html="msg.html"></span>
diff --git a/js/interface/menu.js b/js/interface/menu.js
index 3b24f74b..fd2b1cc9 100644
--- a/js/interface/menu.js
+++ b/js/interface/menu.js
@@ -646,6 +646,7 @@ const MenuBar = {
 			'toggle_wireframe',
 			'preview_checkerboard',
 			'painting_grid',
+			'_',
 			'toggle_quad_view',
 			'focus_on_selection',
 			{name: 'menu.view.screenshot', id: 'screenshot', icon: 'camera_alt', children: [
diff --git a/js/preview/preview.js b/js/preview/preview.js
index 746883a5..3473775b 100644
--- a/js/preview/preview.js
+++ b/js/preview/preview.js
@@ -1863,7 +1863,7 @@ function buildGrid() {
 
 BARS.defineActions(function() {
 	new Action('toggle_wireframe', {
-		icon: 'border_clear',
+		icon: 'check_box_outline_blank',
 		category: 'view',
 		keybind: new Keybind({key: 90}),
 		condition: () => Toolbox && Toolbox.selected && Toolbox.selected.allowWireframe,
@@ -1874,6 +1874,7 @@ BARS.defineActions(function() {
 				Animator.preview()
 			}
 			Blockbench.showQuickMessage('message.wireframe.' + (Prop.wireframe ? 'enabled' : 'disabled'))
+			this.setIcon(Prop.wireframe ? 'check_box' : 'check_box_outline_blank')
 		}
 	})
 	new Action('preview_checkerboard', {