feedback5 WAC: players rebind the in-match shortcuts in 设置 → 快捷键 (community request 「快捷键可不可以自己设置」, the owner's decision of 2026-10-07)

- The settings (sp.pref.settings) hold a key map, action → KeyboardEvent.code, for refresh / freeze / level-up /
  retreat / sell / ready (the ready key also pauses a solo battle); the defaults are the 0.1.4 keys R F D Q X Space,
  so nothing changes for a player who never opens the section. sanitizeSettings keeps each saved key that may be a
  shortcut and gives any other its default; not a map, or two actions on one key, gives the defaults.
- One lookup, actionForKey (the character typed first, then the physical key: the 0.1.4 code-or-key match, identical
  for the defaults), serves shortcutFor(e, keys) in game.js and the facing wheel; the modifier, text-field, repeat and
  overlay guards are unchanged. The shop bar, ready / pause and underframe key hints read the map.
- 设置 → 快捷键 (the one SettingsModal, in a match and on the title screen): each action with its key; click or Enter
  waits, the next key binds; Esc cancels (the dialog stays open), Tab leaves; Ctrl / Alt / ⌘ combinations and the
  interface keys (Esc, Tab, Enter, arrows, F1–F12) are refused; a key in use is swapped and the line under the list
  names both; 恢复默认. Esc stays fixed. Touch-only devices: a keyboard note and a folded list; one column under 768 px.
- Tests: test/ui/feedback5-hotkeys.test.js (map, sanitising, conflicts, lookup, capture, handlers and hints with a
  rebound key), test/ui/gameLogic.test.js (sanitizeSettings now returns keys), test/ui/hotkeys.e2e.test.js (SP_E2E=1:
  撤退 rebound to W with the keyboard only in the mock match; phone width).

Golden unchanged (client UI only).
This commit is contained in:
sganggs committed 2026-10-07 01:56:19 +08:00
1 parent 30d9a41c51
commit f2bec76176
14 files changed
+772 -61

No files matched your search

+141
View File
@@ -0,0 +1,141 @@
// 0.2.0 rebindable shortcuts (设置 → 快捷键) in the mock match, headless Chrome, the DOM field (no art / WebGL needed):
// 撤退 rebound from Q to W with the keyboard only — Enter on the key waits, Esc ends the wait (the dialog stays), a refused
// key keeps it — a swap and 恢复默认 with their messages, Q then does nothing and W retreats, the underframe shows W, the
// choice survives a reload; at phone width (640×360, touch) the section fits and says the keys need a keyboard.
// Unit side: test/ui/feedback5-hotkeys.test.js.
// SP_E2E=1 CHROME_PATH=/path/to/chrome node --test test/ui/hotkeys.e2e.test.js
import { test, describe, before, after } from 'node:test';
import assert from 'node:assert/strict';
import { existsSync, mkdirSync } from 'node:fs';
import { fileURLToPath } from 'node:url';
const OUT = fileURLToPath(new URL('../e2e/out/', import.meta.url));
const CHROME = process.env.CHROME_PATH || '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome';
const ENABLED = process.env.SP_E2E === '1' && existsSync(CHROME);
describe('rebindable shortcuts (设置 → 快捷键)', { skip: !ENABLED && 'set SP_E2E=1 and CHROME_PATH to run' }, () => {
let srv, browser;
before(async () => {
const { startServer } = await import('../../server/index.js');
const puppeteer = (await import('puppeteer-core')).default;
srv = await startServer({ port: 0, host: '127.0.0.1', quiet: true });
browser = await puppeteer.launch({ executablePath: CHROME, headless: true, args: ['--no-sandbox', '--no-proxy-server'] });
mkdirSync(OUT, { recursive: true });
});
after(async () => { await browser?.close(); await srv?.close(); });
const sleep = (ms) => new Promise((r) => setTimeout(r, ms));
const mockState = (page) => page.evaluate(() => JSON.parse(JSON.stringify(globalThis.__MOCK__.S().priv)));
const keyOf = (page, action) => page.$eval(`.set-key[data-action="${action}"]`, (el) => el.textContent.trim());
const keyIs = (page, action, label) => page.waitForFunction((a, l) => document.querySelector(`.set-key[data-action="${a}"]`)?.textContent.trim() === l, {}, action, label);
const note = (page) => page.$eval('.set-keys__note', (el) => el.textContent.trim());
async function open(page, { width = 1280, height = 720, touch = false } = {}) {
const problems = [];
page.on('pageerror', (e) => problems.push(`pageerror: ${e.message}`));
page.on('console', (m) => { if (m.type() === 'error') problems.push(`console: ${m.text()}`); });
await page.setViewport({ width, height, isMobile: touch, hasTouch: touch });
await page.goto(`${srv.url}/dev/game-mock.html?shot=1&render=fallback&phase=PREP`, { waitUntil: 'domcontentloaded' });
await page.waitForSelector('.ff-piece');
return problems;
}
test('撤退 rebound to W in a mock match: keyboard only, swap, Esc, refusal, the HUD hint, a reload', async () => {
const page = await browser.newPage();
try {
const problems = await open(page);
await page.click('.gm__gear');
await page.waitForSelector('.modal .set-keys');
assert.deepEqual(await page.$$eval('.set-key[data-action]', (els) => els.map((el) => el.textContent.trim())), ['R', 'F', 'D', 'Q', 'X', 'Space'],
'the defaults are the keys of 0.1.4');
// a swap: 撤退 onto X (出售's key) — 出售 takes the old Q, and the line under the list says so
await page.click('.set-key[data-action="retreat"]');
await page.waitForSelector('.set-key.is-waiting[data-action="retreat"]');
assert.match(await note(page), /请按下「撤退选中干员」的新按键(Esc 取消)/);
await page.keyboard.press('KeyX');
await keyIs(page, 'retreat', 'X');
assert.equal(await keyOf(page, 'sell'), 'Q');
assert.match(await note(page), /「撤退选中干员」已改为 X;「出售选中干员」原来用 X,已换成 Q/);
await page.click('.set-keys__reset');
await keyIs(page, 'retreat', 'Q');
assert.equal(await keyOf(page, 'sell'), 'X');
assert.match(await note(page), /已恢复默认快捷键/);
assert.equal(await page.$eval('.set-keys__reset', (el) => el.disabled), true, '恢复默认 is off at the defaults');
// keyboard only: Enter on the focused key waits; Esc ends the wait and the dialog stays open
await page.focus('.set-key[data-action="retreat"]');
await page.keyboard.press('Enter');
await page.waitForSelector('.set-key.is-waiting[data-action="retreat"]');
await page.keyboard.press('Escape');
await page.waitForSelector('.set-key.is-waiting', { hidden: true });
assert.ok(await page.$('.modal .set-keys'), 'Esc ends the wait, not the dialog');
assert.match(await note(page), /已取消,「撤退选中干员」仍是 Q/);
// the focus stayed on the key: Enter waits again; Enter itself is refused and the wait goes on; W is taken
await page.keyboard.press('Enter');
await page.waitForSelector('.set-key.is-waiting[data-action="retreat"]');
await page.keyboard.press('Enter');
await sleep(100);
assert.match(await note(page), /Enter 不能设为快捷键/);
assert.ok(await page.$('.set-key.is-waiting[data-action="retreat"]'), 'a refused key keeps the wait');
await page.keyboard.press('KeyW');
await keyIs(page, 'retreat', 'W');
assert.match(await note(page), /「撤退选中干员」已改为 W/);
assert.equal(await page.evaluate(() => JSON.parse(localStorage.getItem('sp.pref.settings')).keys.retreat), 'KeyW', 'saved with the settings');
await page.keyboard.press('Escape'); // no wait: Esc closes the dialog as before
await page.waitForSelector('.modal', { hidden: true });
// the match: the underframe shows W, Q no longer retreats, W does
const initial = await mockState(page);
const unit = initial.board.find((p) => p.kind === 'chess' && !p.golden);
await page.click(`.ff-piece[data-uid="${unit.uid}"]`);
await page.waitForSelector('.uframe__btn--retreat');
assert.match(await page.$eval('.uframe__btn--retreat', (el) => el.textContent), /\[W\]/);
assert.equal(await page.$eval('.uframe__btn--retreat', (el) => el.getAttribute('aria-keyshortcuts')), 'W');
await page.keyboard.press('KeyQ');
await sleep(300);
assert.deepEqual(await mockState(page), initial, 'Q no longer retreats');
await page.keyboard.press('KeyW');
await page.waitForFunction((uid) => globalThis.__MOCK__.S().priv.hand.some((p) => p?.uid === uid), {}, unit.uid);
assert.equal((await mockState(page)).funds, initial.funds, 'W retreats (it does not sell)');
// the choice survives a reload; 恢复默认 brings Q back
await page.reload({ waitUntil: 'domcontentloaded' });
await page.waitForSelector('.ff-piece');
await page.click('.gm__gear');
await page.waitForSelector('.modal .set-keys');
assert.equal(await keyOf(page, 'retreat'), 'W');
await page.click('.set-keys__reset');
await keyIs(page, 'retreat', 'Q');
assert.deepEqual(problems, []);
} finally { await page.close(); }
});
test('phone width (640×360, touch): the keys need a keyboard, the folded list opens and fits, no horizontal scroll', async () => {
const page = await browser.newPage();
try {
const problems = await open(page, { width: 640, height: 360, touch: true });
await page.tap('.gm__gear');
await page.waitForSelector('.modal .set-keys');
assert.match(await page.$eval('.set-keys .set-hint', (el) => el.textContent), /快捷键需要实体键盘/);
assert.equal(await page.$eval('.set-keys__more', (el) => el.open), false, 'the list is folded on a touch-only device');
// (tap once the dialog's entrance animation is over: a tap during it lands where the summary is about to be)
await page.waitForFunction(() => !document.querySelector('.modal__box')?.getAnimations().some((a) => a.playState === 'running'));
await page.tap('.set-keys__more > summary');
await page.waitForFunction(() => document.querySelector('.set-keys__more')?.open === true);
const box = await page.$eval('.modal__body', (body) => {
const b = body.getBoundingClientRect();
const rows = [...body.querySelectorAll('.set-keys__row')].map((r) => r.getBoundingClientRect());
return { scroll: body.scrollWidth, client: body.clientWidth, left: b.left, right: b.right, rows: rows.map((r) => [r.left, r.right, r.width]) };
});
assert.ok(box.scroll <= box.client, `no horizontal scroll in the dialog (${box.scroll} > ${box.client})`);
assert.equal(box.rows.length, 7, 'six shortcuts and the fixed Esc');
for (const [l, r, w] of box.rows) assert.ok(l >= box.left - 0.5 && r <= box.right + 0.5 && w > 0, `a row inside the dialog: ${l}–${r}`);
assert.equal(new Set(box.rows.map(([l]) => Math.round(l))).size, 1, 'one column at phone width');
await page.$eval('.set-keys__more', (el) => el.scrollIntoView({ block: 'start' }));
assert.equal(await page.$eval('.set-keys__more', (el) => el.open), true);
await page.screenshot({ path: `${OUT}/hotkeys-phone.png` });
assert.deepEqual(problems, []);
} finally { await page.close(); }
});
});