From bd215cbd2e6f46870cb69729673c3e5218f4ee86 Mon Sep 17 00:00:00 2001 From: Adam Soltys Date: Mon, 3 Aug 2026 23:17:33 -0700 Subject: The chat window fits the screen, and the header stops asking questions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The message log had a fixed 62vh height, so the card was head + panels + log + composer tall and the input fell off the bottom of the screen. The card is now sized to the viewport and the log is the only part that flexes: opening the invite or member panel shrinks the scroll area instead of pushing the composer out of reach. Three header changes, all the same idea — don't show a control for a choice nobody is making: '+' becomes 'Add person', because a plus sign doesn't say what it adds. A lone channel isn't named at all. '#general' next to a composer that says 'Message #general' points at a choice of one; several channels become a select, which is what a choice looks like. The new-channel field hides behind a 'New channel' button. Sitting open right after you create a community, it reads as something you're supposed to fill in. tools/chat-layout-test.js holds the line at three viewport sizes with both panels open, and covers the header behaviour through actually adding a second channel. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01Wvj4aoeMZwA7VUGdJP9Swi --- src/features/messages.js | 44 ++++++++++++++++++++++++++++---------------- 1 file changed, 28 insertions(+), 16 deletions(-) (limited to 'src/features/messages.js') diff --git a/src/features/messages.js b/src/features/messages.js index ed16f76..8fbc27a 100644 --- a/src/features/messages.js +++ b/src/features/messages.js @@ -462,6 +462,7 @@ export function messagesFeature(ctx) { const ok = await publishOn(room.relays, wrap); if (!ok) { toast(t('msgSendFailed')); return; } ui.msgNewChannel = ''; + ui.msgChannelPanel = false; // the field has done its job — put it away toast('#' + name.replace(/^#/, '')); } @@ -1494,13 +1495,14 @@ export function messagesFeature(ctx) { memberCount ? t('msgMembers', { n: memberCount }) : t('msgEncrypted'), onlineCount ? h('span', { class: 'online-count' }, ' · ', t('msgNOnline', { n: onlineCount })) : null))), h('div', { class: 'row gap6', style: 'align-items:center' }, + // One channel needs no label — naming it only asks people to notice a + // choice they don't have. Several become a picker. chans.length > 1 - ? h('div', { class: 'seg' }, chans.map((c) => - h('button', { - class: c.id === ch?.id ? 'active' : '', - onClick: () => { ui.msgChannel = c.id; ui.msgStick = true; subChannel(room, c.id); render(); }, - }, '#' + c.name))) - : h('div', { class: 'tag' }, '#' + (ch ? ch.name : '')), + ? h('select', { + class: 'chan-pick', + onChange: (e) => { ui.msgChannel = e.target.value; ui.msgStick = true; subChannel(room, e.target.value); render(); }, + }, chans.map((c) => h('option', { value: c.id, selected: c.id === ch?.id }, '#' + c.name))) + : null, h('button', { class: 'iconbtn bell' + (roomNotify(jm.community_id) ? ' on' : ''), title: roomNotify(jm.community_id) ? t('msgNotifyOn') : t('msgNotifyOff'), @@ -1510,9 +1512,9 @@ export function messagesFeature(ctx) { : '', }), h('button', { - class: 'iconbtn', title: t('msgInviteTitle'), + class: 'btn-sm', title: t('msgInviteTitle'), onClick: () => { ui.msgInvitePanel = !ui.msgInvitePanel; render(); }, - }, '+'))), + }, t('msgAddPerson')))), ui.msgInvitePanel ? invitePanel(room) : null, ui.msgMembers ? memberPanel(room, members) : null, h('div', { @@ -1524,7 +1526,7 @@ export function messagesFeature(ctx) { }, ...(ch ? messageRows(room, ch.id) : [])), ch ? typingLine(room, ch.id) : null, composer( - t('msgPlaceholder', { channel: ch ? ch.name : '' }), + chans.length > 1 ? t('msgPlaceholder', { channel: ch ? ch.name : '' }) : t('msgPlaceholderPlain'), () => ch && sendMessage(room, ch.id), () => ch && ping(room, ch.id, TYPING))); } @@ -1580,14 +1582,24 @@ export function messagesFeature(ctx) { class: 'btn-sm', onClick: () => { sendDirectInvite(room, ui.msgInviteTo); ui.msgInviteTo = ''; render(); }, }, t('msgSend'))), - // owners can add channels (an owner-signed control edition) + // Owners can add channels (an owner-signed control edition), but a naming + // field sitting open is a question nobody asked — it reads as something + // you're meant to fill in right after making a community. myPubkeys().includes(room.jm.owner) - ? h('div', { class: 'row gap6' }, - h('input', { - class: 'grow', type: 'text', placeholder: t('msgChannelPlaceholder'), maxlength: '64', - value: ui.msgNewChannel || '', onInput: (e) => { ui.msgNewChannel = e.target.value; }, - }), - h('button', { class: 'btn-sm', onClick: () => createChannel(room, ui.msgNewChannel) }, t('msgCreate'))) + ? (ui.msgChannelPanel + ? h('div', { class: 'row gap6' }, + h('input', { + class: 'grow', type: 'text', placeholder: t('msgChannelPlaceholder'), maxlength: '64', + value: ui.msgNewChannel || '', onInput: (e) => { ui.msgNewChannel = e.target.value; }, + onKeydown: (e) => { if (e.key === 'Enter') createChannel(room, ui.msgNewChannel); }, + }), + h('button', { class: 'btn-sm', onClick: () => createChannel(room, ui.msgNewChannel) }, t('msgCreate')), + h('button', { class: 'btn-ghost btn-sm', onClick: () => { ui.msgChannelPanel = false; render(); } }, t('cancel'))) + : h('div', { class: 'row' }, + h('button', { + class: 'btn-sm', + onClick: () => { ui.msgChannelPanel = true; render(); }, + }, t('msgNewChannel')))) : null, // leaving discards the keys on this identity — two taps, default community exempt builtin ? null : h('div', { class: 'row' }, -- cgit v1.2.3