vrr/sillytavern-character-memory
2
0
Fork 0
mirror of https://github.com/bal-spec/sillytavern-character-memory.git synced 2026-04-28 03:39:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: use Save/Cancel buttons on consolidation and conversion dialogs

Browse source 
Replaces ambiguous Yes/No default buttons with explicit Save/Cancel
to prevent accidentally closing without saving.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
bal-spec 2026-03-06 19:51:03 -08:00
parent 4168f61075
commit 3e846a1bef
3 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
CHANGELOG.md
View file

@ -1,5 +1,11 @@
# Changelog
## 1.6.2
### Improvements
- **Clearer dialog buttons**: Consolidation and conversion dialogs now show "Save" and "Cancel" buttons instead of the ambiguous "Yes"/"No" defaults, preventing accidental loss of edits.
## 1.6.1
### New Features

4
index.js
View file

@ -982,7 +982,7 @@ async function previewConversion() {
const formatLabel = formatLabels[format] || format;
const method = useLLM && format !== 'memory_tags' ? 'LLM' : 'Heuristic';
const dialogHtml = buildConversionDialog(sourceContent, formatLabel, method, editorBlocks, editingSet, useLLM && format !== 'memory_tags');
const popup = callGenericPopup(dialogHtml, POPUP_TYPE.CONFIRM, '', { wide: true, allowVerticalScrolling: true });
const popup = callGenericPopup(dialogHtml, POPUP_TYPE.CONFIRM, '', { wide: true, allowVerticalScrolling: true, okButton: 'Save', cancelButton: 'Cancel' });
// === Editor event delegation (same card classes as consolidation, different namespaces) ===
@ -4194,7 +4194,7 @@ async function consolidateMemories() {
// Build and show the interactive dialog
const dialogHtml = buildConsolidationDialog(memories, beforeCount, editorBlocks, editingSet);
const popup = callGenericPopup(dialogHtml, POPUP_TYPE.CONFIRM, '', { wide: true, allowVerticalScrolling: true });
const popup = callGenericPopup(dialogHtml, POPUP_TYPE.CONFIRM, '', { wide: true, allowVerticalScrolling: true, okButton: 'Save', cancelButton: 'Cancel' });
// Set up the strategy dropdown and prompt viewer to match current setting
const currentStrategy = extension_settings[MODULE_NAME].consolidationStrategy || 'balanced';

2
manifest.json
View file

@ -6,7 +6,7 @@
"js": "index.js",
"css": "style.css",
"author": "bal-spec",
"version": "1.6.1",
"version": "1.6.2",
"homePage": "",
"auto_update": false
}