From b830ed9744a4bceb08ecc44ca50120f18af83121 Mon Sep 17 00:00:00 2001 From: Richard Hansen Date: Mon, 6 Dec 2021 17:11:28 -0500 Subject: [PATCH] chat: Rename `hide` method to `reduce` It doesn't hide: If sticky, it unsticks. Otherwise, it minimizes. --- src/static/js/chat.js | 4 ++-- src/static/js/pad.js | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/static/js/chat.js b/src/static/js/chat.js index ac6e96880..acb471b39 100755 --- a/src/static/js/chat.js +++ b/src/static/js/chat.js @@ -79,7 +79,7 @@ exports.chat = (() => { .toggleClass('chatAndUsers popup-show stickyUsers', userAndChat); $('#chatbox').toggleClass('chatAndUsersChat', userAndChat); }, - hide() { + reduce() { // decide on hide logic based on chat window being maximized or not if ($('#options-stickychat').prop('checked')) { this.stickToScreen(); @@ -221,7 +221,7 @@ exports.chat = (() => { $('#options-stickychat').on('click', () => this.stickToScreen()); $('#options-chatandusers').on('click', () => this.chatAndUsers()); $('#chaticon').on('click', () => { this.show(); return false; }); - $('#titlecross').on('click', () => { this.hide(); return false; }); + $('#titlecross').on('click', () => { this.reduce(); return false; }); $('#titlesticky').on('click', () => { this.stickToScreen(true); return false; }); $('#chatinput').on('keydown', (evt) => { // If the event is Alt C or Escape & we're already in the chat menu diff --git a/src/static/js/pad.js b/src/static/js/pad.js index c37920ead..d6cebdd90 100644 --- a/src/static/js/pad.js +++ b/src/static/js/pad.js @@ -76,7 +76,7 @@ const getParameters = [ callback: (val) => { if (val === 'false') { settings.hideChat = true; - chat.hide(); + chat.reduce(); $('#chaticon').hide(); } }, @@ -480,7 +480,7 @@ const pad = { } if (window.clientVars.readonly) { - chat.hide(); + chat.reduce(); $('#myusernameedit').attr('disabled', true); $('#chatinput').attr('disabled', true); $('#chaticon').hide();