Don't minimize chats by clicking the header

This prevents the burger menu from opening
This commit is contained in:
JC Brand 2020-08-24 11:24:36 +02:00
parent 9f0dfcdbdb
commit 8c799efc58
4 changed files with 3 additions and 6 deletions

View File

@ -447,7 +447,6 @@
height: calc(100% - var(--overlayed-chat-head-height)); height: calc(100% - var(--overlayed-chat-head-height));
} }
.chatbox-title { .chatbox-title {
cursor: pointer;
padding: 0.5rem 0.75rem 0 0.75rem; padding: 0.5rem 0.75rem 0 0.75rem;
} }
.chatbox-title--no-desc { .chatbox-title--no-desc {
@ -455,7 +454,8 @@
} }
converse-dropdown { converse-dropdown {
.btn--standalone { .btn--standalone {
padding: 0 0 0 0.5em; padding: 0 0.2em;
margin: 0 0 0 0.5em;
} }
} }
} }

View File

@ -681,8 +681,7 @@ describe("Groupchats", function () {
const toggle = view.el.querySelector('.hide-topic'); const toggle = view.el.querySelector('.hide-topic');
expect(toggle.textContent).toBe('Hide topic'); expect(toggle.textContent).toBe('Hide topic');
toggle.click(); toggle.click();
await u.waitUntil(() => !u.isVisible(topic_el)); await u.waitUntil(() => view.el.querySelector('.hide-topic').textContent === 'Show topic');
expect(view.el.querySelector('.hide-topic').textContent).toBe('Show topic');
done(); done();
})); }));

View File

@ -172,7 +172,6 @@ converse.plugins.add('converse-chatview', {
events: { events: {
'click .chatbox-navback': 'showControlBox', 'click .chatbox-navback': 'showControlBox',
'click .chatbox-title': 'minimize',
'click .new-msgs-indicator': 'viewUnreadMessages', 'click .new-msgs-indicator': 'viewUnreadMessages',
'click .send-button': 'onFormSubmitted', 'click .send-button': 'onFormSubmitted',
'click .toggle-clear': 'clearMessages', 'click .toggle-clear': 'clearMessages',

View File

@ -163,7 +163,6 @@ converse.plugins.add('converse-muc-views', {
is_chatroom: true, is_chatroom: true,
events: { events: {
'click .chatbox-navback': 'showControlBox', 'click .chatbox-navback': 'showControlBox',
'click .chatbox-title': 'minimize',
'click .hide-occupants': 'hideOccupants', 'click .hide-occupants': 'hideOccupants',
'click .new-msgs-indicator': 'viewUnreadMessages', 'click .new-msgs-indicator': 'viewUnreadMessages',
// Arrow functions don't work here because you can't bind a different `this` param to them. // Arrow functions don't work here because you can't bind a different `this` param to them.