mirror of
https://github.com/got-feedBack/feedBack.git
synced 2026-08-19 06:52:38 +00:00
topkoa
cb425ed48d
Six more findings from CodeRabbit on #928. Three are real bugs. 1. THE CHIP HID DOCKED PANES. `_onOpened` decided "did the pane take my element?" from `ownerDocument !== document`. That is true for a pane in a pop-out window — and false for a pane moved into the DOCK, which lives in this very document. So docking a pane stamped `.fb-pane-detached` (display:none !important) onto the panel the user was looking at, and put the stub next to it instead of at its home. The element cannot answer this question — `isConnected` is true in a pane window, `ownerDocument` is this one in the dock. Both were live bugs. Ask the manager, which knows exactly what it handed to the host: `panes.elementOf(id)`. That holds for every host, and for reconciling after the fact (detail == null), which is what a plugin rebuilding its panel mid-pop-out triggers. 2. `.fb-paned` FORCED `display: block !important`. A panel that is `display:flex` or `grid` would be silently re-laid-out while detached — the exact opposite of "placement only", and precisely the kind of surprise this feature exists to avoid. Removed. Making a hidden panel visible is a separate job, and it now belongs to the manager, which does it without touching the panel's display MODE: clear `hidden`, and clear an inline `display:none` if that is how the panel hides. 3. VISIBILITY IS NOW RESTORED. The hosts used to set `el.hidden = false` and never put it back, so the docs' "core only changes placement" was a lie and a panel's hidden state was quietly lost. The manager stashes both `hidden` and the inline `display` on open and restores them on dock: a panel that was closed when you opened its pane from the tray goes back to being closed; one that was open stays open. Plus: - The launcher rebuilt its whole list on every panes:opened/closed — including the one fired by clicking a button in that list — destroying the button under the user's finger and dropping focus to <body>. It now restores focus to the toggled pane's button. - `_copyStyles` cloned every stylesheet link, including the panes.css that pane.html already loads. Skip sheets the pane document already has. - Docs: the chip may route to the DOCK, not always a window (it goes through detach() → the host router). `header` precedence was documented backwards — an explicit `header` wins. And the visibility contract above is now written down rather than being a surprise. Signed-off-by: topkoa <topkoa@gmail.com>
114 lines
4.3 KiB
JavaScript
114 lines
4.3 KiB
JavaScript
/*
|
|
* fee[dB]ack — pane dock (the in-window pane host).
|
|
*
|
|
* A right-edge stack of cards, one per open pane. Deliberately NOT a rail popover:
|
|
* the rail is exclusive (player-chrome.js's openPopFor closes the last one before
|
|
* opening the next), which is exactly why you cannot watch the mixer while riding
|
|
* the camera. Cards here coexist.
|
|
*
|
|
* As everywhere in this system, the card holds the plugin's REAL element — moved,
|
|
* not copied. The dock is a frame; the panel inside it is the panel.
|
|
*
|
|
* Song-switch survival is structural, not defended: #fb-pane-dock is a <body>
|
|
* child outside every .screen, so the per-song teardown never sees it.
|
|
*
|
|
* Registers as the `dock` host at priority 0 — the floor. Whatever else exists
|
|
* (an OS window), a pane can always land here, so opening one can never fail.
|
|
*/
|
|
(function () {
|
|
'use strict';
|
|
|
|
const panes = window.feedBack && window.feedBack.panes;
|
|
if (!panes || typeof panes.registerHost !== 'function') {
|
|
console.error('[panes] pane-manager.js must load before pane-dock.js');
|
|
return;
|
|
}
|
|
|
|
let dockEl = null;
|
|
const cards = new Map(); // paneId -> card element
|
|
|
|
function dock() {
|
|
if (dockEl && dockEl.isConnected) return dockEl;
|
|
dockEl = document.getElementById('fb-pane-dock');
|
|
if (!dockEl) {
|
|
dockEl = document.createElement('div');
|
|
dockEl.id = 'fb-pane-dock';
|
|
dockEl.className = 'fb-pane-dock';
|
|
dockEl.setAttribute('role', 'region');
|
|
dockEl.setAttribute('aria-label', 'Panes');
|
|
document.body.appendChild(dockEl);
|
|
}
|
|
return dockEl;
|
|
}
|
|
|
|
function _syncEmpty() {
|
|
dock().classList.toggle('is-empty', cards.size === 0);
|
|
}
|
|
|
|
function place(spec, el) {
|
|
const card = document.createElement('section');
|
|
card.className = 'fb-pane-card';
|
|
card.dataset.paneId = spec.id;
|
|
card.setAttribute('aria-label', spec.title);
|
|
|
|
const head = document.createElement('header');
|
|
head.className = 'fb-pane-card-head';
|
|
|
|
const title = document.createElement('span');
|
|
title.className = 'fb-pane-card-title';
|
|
// textContent, not innerHTML — a pane title comes from a plugin.
|
|
title.textContent = spec.icon + ' ' + spec.title;
|
|
|
|
const close = document.createElement('button');
|
|
close.type = 'button';
|
|
close.className = 'fb-pane-card-btn';
|
|
close.setAttribute('aria-label', 'Close ' + spec.title);
|
|
close.title = 'Close';
|
|
close.textContent = '✕';
|
|
close.addEventListener('click', () => panes.close(spec.id));
|
|
|
|
head.appendChild(title);
|
|
head.appendChild(close);
|
|
|
|
const body = document.createElement('div');
|
|
body.className = 'fb-pane-card-body';
|
|
|
|
// Same neutralisation as the window host: the panel was a fixed overlay
|
|
// pinned to a corner of the app, and inside a card that positioning is
|
|
// nonsense. .fb-paned unpins it and nothing else.
|
|
el.classList.add('fb-paned');
|
|
body.appendChild(el);
|
|
|
|
card.appendChild(head);
|
|
card.appendChild(body);
|
|
dock().appendChild(card);
|
|
cards.set(spec.id, card);
|
|
_syncEmpty();
|
|
}
|
|
|
|
function unplace(id, el) {
|
|
// Hand the element back unmarked. The manager returns it to its home right
|
|
// after this, and it must arrive as the plugin left it — a panel that
|
|
// stayed .fb-paned would come back with its own positioning stripped.
|
|
if (el) el.classList.remove('fb-paned');
|
|
const card = cards.get(id);
|
|
if (card) card.remove();
|
|
cards.delete(id);
|
|
_syncEmpty();
|
|
}
|
|
|
|
function focus(id) {
|
|
const card = cards.get(id);
|
|
if (!card) return;
|
|
card.scrollIntoView({ block: 'nearest', behavior: 'smooth' });
|
|
// Re-trigger the flash even if the class is still there — repeat focus of
|
|
// the same card would otherwise be a no-op animation.
|
|
card.classList.remove('is-flash');
|
|
void card.offsetWidth;
|
|
card.classList.add('is-flash');
|
|
setTimeout(() => card.classList.remove('is-flash'), 700);
|
|
}
|
|
|
|
panes.registerHost({ id: 'dock', priority: 0, available: () => !!document.body, place, unplace, focus });
|
|
})();
|