diff options
| author | Christophe Besson <cbesson@gmail.com> | 2026-09-08 03:06:02 +0200 |
|---|---|---|
| committer | Christophe Besson <cbesson@gmail.com> | 2026-09-08 03:06:02 +0200 |
| commit | d99861b7789915c7585b43f7488d4267dce1211d (patch) | |
| tree | ac1f115cedcb0b836c53a36c83fd94b9fbdcaa81 /packages/meshbay-hub | |
| parent | 11c039ed3f0dc2ecc5eb30512b3dafa647fc4520 (diff) | |
| download | meshbay-d99861b7789915c7585b43f7488d4267dce1211d.tar.gz | |
feat(hub): collapse "Your devices on this node" by default
It is the one section in group settings that is open on arrival while having
nothing to say in the ordinary case: the device you are reading it on is
already linked, and the panel exists for two rare errands — approving another
device, or removing one.
Uses the CollapsibleSection the sections around it already use, with
defaultOpen={false}; the component renders the same .settings-section
wrapper, so nothing about the layout moves.
The prompt that *is* actionable — "this browser is not linked to this node
yet" (device.add_title) — lives in group-page.js and is untouched, so a
browser that genuinely needs linking still says so on arrival.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V8EDjk6pkYZrCbo63m2x87
Diffstat (limited to 'packages/meshbay-hub')
| -rw-r--r-- | packages/meshbay-hub/src/meshbay_hub/static/group-settings.js | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/packages/meshbay-hub/src/meshbay_hub/static/group-settings.js b/packages/meshbay-hub/src/meshbay_hub/static/group-settings.js index c80cff5..437af8b 100644 --- a/packages/meshbay-hub/src/meshbay_hub/static/group-settings.js +++ b/packages/meshbay-hub/src/meshbay_hub/static/group-settings.js @@ -1022,9 +1022,14 @@ function GroupSettingsPanel({ groupId, group, token, transportRef, gekRef, </${CollapsibleSection}> `} + ${/* Collapsed by default, unlike the sections around it. Nothing here + needs attention in the ordinary case: the device you are reading + this on is already linked, and the panel exists for the two rare + errands — approving another device, or removing one. The prompt + that *is* actionable, "this browser is not linked to this node + yet" (device.add_title), lives in group-page.js and is unaffected. */''} ${connected && html` - <div class="settings-section"> - <h3 class="settings-heading">${t('device.mine_title')}</h3> + <${CollapsibleSection} title=${t('device.mine_title')} defaultOpen=${false}> <p class="settings-hint">${t('device.mine_hint')}</p> ${deviceMsg && html`<p class="settings-hint">${deviceMsg}</p>`} ${devices.length === 0 @@ -1057,7 +1062,7 @@ function GroupSettingsPanel({ groupId, group, token, transportRef, gekRef, <button class="admin-btn" type="submit">${t('device.approve_btn')}</button> </div> </form> - </div> + </${CollapsibleSection}> `} <${CollapsibleSection} title=${`${t('group.tab_members')} (${members.length})`}> |