obsidian-plugin/murashit_codex-panel
1
0
Fork 0
mirror of https://github.com/murashit/codex-panel.git synced 2026-07-22 17:30:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Simplify rewrite selection popover output

Browse source
This commit is contained in:
murashit 2026-05-18 12:56:42 +09:00
parent 0fd33fc1cc
commit d0f8b1fbea
2 changed files with 20 additions and 41 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
src/editor-rewrite/popover.ts
View file

@ -28,7 +28,6 @@ export class RewriteSelectionPopover {
private generateButton: HTMLButtonElement | null = null;
private applyButton: HTMLButtonElement | null = null;
private statusEl: HTMLElement | null = null;
private previewEl: HTMLElement | null = null;
private diffEl: HTMLElement | null = null;
private debugEl: HTMLDetailsElement | null = null;
private readonly cleanups: Cleanup[] = [];
@ -67,11 +66,7 @@ export class RewriteSelectionPopover {
};
this.statusEl = root.createDiv({ cls: "codex-panel-rewrite-popover__status" });
this.previewEl = root.createEl("pre", { cls: "codex-panel-rewrite-popover__preview" });
this.diffEl = root.createDiv({ cls: "codex-panel-rewrite-popover__diff" });
const debugEl = root.createEl("details", { cls: "codex-panel-rewrite-popover__debug" });
debugEl.createEl("summary", { text: "Debug output" });
this.debugEl = debugEl;
const footer = root.createDiv({ cls: "codex-panel-rewrite-popover__footer" });
const applyButton = footer.createEl("button", {
@ -92,7 +87,7 @@ export class RewriteSelectionPopover {
}
});
this.setStatus("Enter an instruction, then generate a patch.");
this.setStatus("");
this.syncControls();
this.position();
instructionEl.focus();
@ -106,7 +101,6 @@ export class RewriteSelectionPopover {
this.generateButton = null;
this.applyButton = null;
this.statusEl = null;
this.previewEl = null;
this.diffEl = null;
this.debugEl = null;
}
@ -124,10 +118,11 @@ export class RewriteSelectionPopover {
this.options.session.streamText = "";
this.options.session.replacementText = null;
this.options.session.debugText = null;
this.previewEl?.setText("");
this.diffEl?.empty();
this.debugEl?.remove();
this.debugEl = null;
this.renderDebug();
this.setStatus("Generating patch", { active: true });
this.setStatus("Generating", { active: true });
this.syncControls();
try {
@ -142,7 +137,7 @@ export class RewriteSelectionPopover {
this.options.session.replacementText = output.replacementText;
this.options.session.status = "preview";
this.renderDiff();
this.setStatus("Review the patch before applying it.");
this.setStatus("");
} catch (error) {
this.options.session.status = "failed";
this.options.session.debugText = error instanceof RewriteOutputError ? error.rawText : null;
@ -156,7 +151,6 @@ export class RewriteSelectionPopover {
private updatePreview(text: string): void {
this.options.session.streamText = text;
this.previewEl?.setText(text);
this.setStatus("Writing replacement", { active: true });
this.position();
}
@ -176,16 +170,15 @@ export class RewriteSelectionPopover {
}
private renderDebug(): void {
if (!this.debugEl) return;
const debugText = this.options.session.debugText;
this.debugEl.empty();
this.debugEl.createEl("summary", { text: "Debug output" });
if (debugText) {
this.debugEl.createEl("pre", { text: debugText });
this.debugEl.classList.remove("is-hidden");
} else {
this.debugEl.classList.add("is-hidden");
}
this.debugEl?.remove();
this.debugEl = null;
if (!debugText || !this.rootEl) return;
const debugEl = this.rootEl.createEl("details", { cls: "codex-panel-rewrite-popover__debug" });
debugEl.createEl("summary", { text: "Debug output" });
debugEl.createEl("pre", { text: debugText });
this.debugEl = debugEl;
}
private apply(): void {

28
styles.css
View file

@ -1400,10 +1400,10 @@
box-sizing: border-box;
display: flex;
flex-direction: column;
gap: var(--size-4-3, 12px);
gap: var(--size-4-2, 8px);
width: min(520px, calc(100vw - 24px));
max-height: min(560px, calc(100vh - 24px));
padding: var(--size-4-3, 12px);
padding: var(--size-4-2, 8px);
overflow: auto;
border: 1px solid var(--background-modifier-border);
border-radius: var(--radius-m);
@ -1413,7 +1413,7 @@
}
.codex-panel-rewrite-popover__instruction {
min-height: 92px;
min-height: 72px;
resize: vertical;
}
@ -1430,6 +1430,10 @@
font-size: var(--font-ui-small);
}
.codex-panel-rewrite-popover__status:empty {
display: none;
}
.codex-panel-rewrite-popover__status.is-active {
color: var(--text-faint);
}
@ -1450,24 +1454,6 @@
animation-delay: 0.36s;
}
.codex-panel-rewrite-popover__preview {
display: none;
max-height: 120px;
overflow: auto;
margin: 0;
padding: 8px;
border-radius: var(--radius-s);
background: var(--code-background, var(--background-secondary));
color: var(--text-muted);
font-family: var(--font-monospace);
font-size: var(--font-smaller);
white-space: pre-wrap;
}
.codex-panel-rewrite-popover__preview:not(:empty) {
display: block;
}
.codex-panel-rewrite-popover__diff .codex-panel__diff {
max-height: min(300px, 38vh);
margin: 0;