Skip to content

Commit baa11b5

Browse files
JohnMcLearclaude
andcommitted
fix(gdpr): always render delete-with-token in settings popup
The rebase onto develop placed the delete-pad-with-token details inside the pad-settings-section conditional, which is only rendered when enablePadWideSettings is true AND the section is toggled visible. Second-device recovery (typing the captured token on a fresh browser) must work without pad-wide settings enabled, so move the details out to sit alongside the existing pad_deletion_token.spec.ts expectations. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f64ba91 commit baa11b5

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/templates/pad.html

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -240,15 +240,15 @@ <h2 data-l10n-id="pad.settings.padSettings"></h2>
240240
<% e.end_block(); %>
241241
</div>
242242
<button class="btn btn-danger" data-l10n-id="pad.settings.deletePad" id="delete-pad">Delete pad</button>
243-
<details id="delete-pad-with-token">
244-
<summary data-l10n-id="pad.deletionToken.deleteWithToken">Delete with token</summary>
245-
<label for="delete-pad-token-input" data-l10n-id="pad.deletionToken.tokenFieldLabel">Pad deletion token</label>
246-
<input type="password" id="delete-pad-token-input" autocomplete="off" spellcheck="false">
247-
<button id="delete-pad-token-submit" type="button" class="btn btn-danger"
248-
data-l10n-id="pad.settings.deletePad">Delete pad</button>
249-
</details>
250243
</div><% } %>
251244
</div>
245+
<details id="delete-pad-with-token">
246+
<summary data-l10n-id="pad.deletionToken.deleteWithToken">Delete with token</summary>
247+
<label for="delete-pad-token-input" data-l10n-id="pad.deletionToken.tokenFieldLabel">Pad deletion token</label>
248+
<input type="password" id="delete-pad-token-input" autocomplete="off" spellcheck="false">
249+
<button id="delete-pad-token-submit" type="button" class="btn btn-danger"
250+
data-l10n-id="pad.settings.deletePad">Delete pad</button>
251+
</details>
252252
<h2 data-l10n-id="pad.settings.about">About</h2>
253253
<span data-l10n-id="pad.settings.poweredBy">Powered by</span>
254254
<a href="https://etherpad.org" target="_blank" referrerpolicy="no-referrer" rel="noopener">Etherpad</a>

0 commit comments

Comments
 (0)