mirror of
https://github.com/ancsemi/Haven.git
synced 2026-11-02 12:05:31 +00:00
Amnibro
e735ac80e5
Reported by birdycrazy: the layout "seems to completely remove the ability to join or create channels", and the Encryption menu entry does nothing. Both are real. Join and Create were not merely collapsed — they could not be reopened at all. _collapseJoinCreate() added its own `braid-collapsed` class to the section alongside the stock `.collapsed` class on the body, and the stylesheet hid `.braid-collapsed .collapsible-section-body` with !important. Nothing ever removed that class, so the stock toggle went on flipping `.collapsed` underneath while the body stayed hidden. Traced live: clicking the header moves the stock class exactly as it should, computed display stays `none`, and dropping `braid-collapsed` by hand returns it to `block`. That left an admin unable to create a channel and everyone unable to join one for as long as the layout was on. Visibility now comes only from the stock class, which the collapse helper still sets, so the sections keep opening collapsed and the header reopens them. The Encryption entry proxied a click to #e2e-menu-btn, whose handler only toggles #e2e-dropdown. That dropdown lives inside #e2e-menu-wrapper, which the app sets to display:none outside a DM — so in a normal channel the entry appeared and did nothing whatsoever. Inside a DM it fared no better: this layout hides the header chrome the dropdown anchors to, and the click that opened it kept bubbling to the document handler that closes it again. Proxy the three actions instead — Verify encryption, Recover keys, Reset keys — each of which opens its own modal and does not depend on the dropdown. The menu is built once, so they are marked and resolved when it opens: shown where encryption applies, hidden everywhere else.