From ede4516e2255bb65d22dd1a74d0e8332b0417194 Mon Sep 17 00:00:00 2001 From: Adam Soltys Date: Mon, 17 Aug 2026 06:57:10 -0700 Subject: The wallet row's delete earns a trash can instead of an ambiguous ✕, and Clear all says Delete all — the word for what it does MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01JtaatFfAbx91Nh3xqKVqKz --- src/app.js | 2 +- src/i18n.js | 2 +- tools/lock-wallet-test.js | 12 ++++++------ tools/padlock-flow-test.js | 2 +- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/app.js b/src/app.js index de3e890..3a9a8d2 100644 --- a/src/app.js +++ b/src/app.js @@ -2340,7 +2340,7 @@ function accountsScreen() { }, (isActive ? '● ' : '○ ') + viewLabel(a, view) + tag), h('button', { class: 'btn-sm wallet-row-btn', title: t('renameWallet'), onClick: () => { ui.editId = a.id; ui.editView = view; ui.editLabel = viewLabel(a, view); render(); } }, '✎'), h('button', { class: 'btn-sm wallet-row-btn', title: t('walletSettings'), onClick: () => openAccountSettings(a.id), html: '' }), - h('button', { class: 'btn-sm wallet-row-btn', title: t('remove'), onClick: () => { ui.confirmRemove = a.id; render(); } }, '✕') + h('button', { class: 'btn-sm wallet-row-btn', title: t('remove'), onClick: () => { ui.confirmRemove = a.id; render(); }, html: '' }) ), a.type === 'full' && view === viewsOf(a)[viewsOf(a).length - 1] ? h('button', { class: 'linklike small', style: 'align-self:flex-start', onClick: () => (a.persisted ? startForget(a.id) : startSave(a.id)) }, diff --git a/src/i18n.js b/src/i18n.js index 8e8dcd2..e2416af 100644 --- a/src/i18n.js +++ b/src/i18n.js @@ -38,7 +38,7 @@ const STR = { lockWallet: 'Lock wallet', accounts: 'Wallets', addWallet: 'Add wallet', - clearAll: 'Clear all', + clearAll: 'Delete all', clearAllWarn: '⚠ This removes every wallet from this device. Any wallet whose recovery phrase or key you have not backed up will be lost for good.', removeWalletTitle: 'Remove wallet', removeWalletWarn: '⚠ Remove “{name}” from this device? It will be gone from this browser — make sure you have its recovery phrase. Other wallets aren’t affected.', diff --git a/tools/lock-wallet-test.js b/tools/lock-wallet-test.js index 5a25615..30f5982 100644 --- a/tools/lock-wallet-test.js +++ b/tools/lock-wallet-test.js @@ -1,4 +1,4 @@ -// "Log out" locks; only "Clear all" deletes. +// "Log out" locks; only "Delete all" deletes. // // Wallets save themselves to this device under an empty password unless the // user sets one. Logging out of a passwordless vault offers a password ONCE @@ -119,14 +119,14 @@ try { check('the password opens it', await waitText('receive')); await sleep(1000); - console.log('\n[clear all is the destructive one]'); + console.log('\n[delete all is the destructive one]'); await click('button', 'Wallets'); await sleep(800); - await click('button', 'Clear all'); + await click('button', 'Delete all'); await sleep(600); txt = await body(); - check('it warns first', /clear all/i.test(txt) && /(delete|remove|erase|wipe|lose)/i.test(txt), txt.slice(0, 160).replace(/\n+/g, ' | ')); - await page.evaluate(() => { const b = [...document.querySelectorAll('.btn-primary')].find((e) => /clear all/i.test(e.textContent)); if (b) b.click(); }); + check('it warns first', /delete all/i.test(txt) && /(delete|remove|erase|wipe|lose)/i.test(txt), txt.slice(0, 160).replace(/\n+/g, ' | ')); + await page.evaluate(() => { const b = [...document.querySelectorAll('.btn-primary')].find((e) => /delete all/i.test(e.textContent)); if (b) b.click(); }); await sleep(1200); txt = await body(); check('everything is gone', (await vaultSize()) === 0); @@ -137,5 +137,5 @@ try { await browser.close(); server.stop(true); } -console.log(fails ? '\n❌ failures above' : '\n✅ log out locks, clear all deletes'); +console.log(fails ? '\n❌ failures above' : '\n✅ log out locks, delete all deletes'); process.exit(fails ? 1 : 0); diff --git a/tools/padlock-flow-test.js b/tools/padlock-flow-test.js index ce37f1a..4b659aa 100644 --- a/tools/padlock-flow-test.js +++ b/tools/padlock-flow-test.js @@ -59,7 +59,7 @@ try { await sleep(600); txt = await body(); check('declining lands back home', /receive/i.test(txt) && /balance/i.test(txt), txt.slice(0, 120).replace(/\n+/g, ' | ')); - check('not on the wallets list', !/add wallet|clear all/i.test(txt)); + check('not on the wallets list', !/add wallet|delete all/i.test(txt)); console.log('\n[the logo still means home]'); await page.evaluate(() => document.querySelector('.brand')?.click()); -- cgit v1.2.3