fix(KB-083): fix SpecEditor layout with proper CSS and remove hardcoded rows
- Add comprehensive CSS styles for SpecEditor component layout - Remove hardcoded rows prop from SpecEditor for flexible sizing - Add layout tests for SpecEditor to verify responsive behavior - Include changeset for patch release of @dustinbyrne/kb package
This commit is contained in:
11
.changeset/fix-spec-editor-layout.md
Normal file
11
.changeset/fix-spec-editor-layout.md
Normal file
@@ -0,0 +1,11 @@
|
|||||||
|
---
|
||||||
|
"@dustinbyrne/kb": patch
|
||||||
|
---
|
||||||
|
|
||||||
|
Fix SpecEditor layout to fill available modal space
|
||||||
|
|
||||||
|
- Added CSS styles for `.spec-editor` component with flex layout
|
||||||
|
- Removed hardcoded `rows={20}` from textarea in favor of CSS-based sizing
|
||||||
|
- Toolbar stays fixed at top while content scrolls
|
||||||
|
- AI revision section stays at bottom
|
||||||
|
- Added responsive styles for mobile viewports
|
||||||
@@ -128,7 +128,6 @@ export function SpecEditor({
|
|||||||
onChange={(e) => setEditContent(e.target.value)}
|
onChange={(e) => setEditContent(e.target.value)}
|
||||||
disabled={isSaving}
|
disabled={isSaving}
|
||||||
placeholder="Enter task specification in Markdown..."
|
placeholder="Enter task specification in Markdown..."
|
||||||
rows={20}
|
|
||||||
/>
|
/>
|
||||||
) : content ? (
|
) : content ? (
|
||||||
<div className="markdown-body">
|
<div className="markdown-body">
|
||||||
|
|||||||
@@ -452,4 +452,37 @@ Test mission description.
|
|||||||
|
|
||||||
expect(screen.queryByText(/Ctrl/)).toBeNull();
|
expect(screen.queryByText(/Ctrl/)).toBeNull();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("textarea fills available width in edit mode", () => {
|
||||||
|
render(<SpecEditor content={mockContent} onSave={mockOnSave} />);
|
||||||
|
|
||||||
|
fireEvent.click(screen.getByText("Edit"));
|
||||||
|
|
||||||
|
const textarea = screen.getByRole("textbox") as HTMLTextAreaElement;
|
||||||
|
expect(textarea.classList.contains("spec-editor-textarea")).toBe(true);
|
||||||
|
// Verify textarea has the CSS class for proper sizing
|
||||||
|
expect(getComputedStyle(textarea).width).toBeDefined();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("spec-editor-content has correct flex direction", () => {
|
||||||
|
const { container } = render(<SpecEditor content={mockContent} onSave={mockOnSave} />);
|
||||||
|
|
||||||
|
const contentDiv = container.querySelector(".spec-editor-content");
|
||||||
|
expect(contentDiv).toBeTruthy();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("toolbar remains visible when content is long", () => {
|
||||||
|
const longContent = "# Title\n\n" + "Content line\n".repeat(100);
|
||||||
|
render(<SpecEditor content={longContent} onSave={mockOnSave} />);
|
||||||
|
|
||||||
|
fireEvent.click(screen.getByText("Edit"));
|
||||||
|
|
||||||
|
// Toolbar should still be visible
|
||||||
|
const toolbar = document.querySelector(".spec-editor-toolbar");
|
||||||
|
expect(toolbar).toBeTruthy();
|
||||||
|
|
||||||
|
// View and Edit buttons should still be present
|
||||||
|
expect(screen.getByText("View")).toBeTruthy();
|
||||||
|
expect(screen.getByText("Edit")).toBeTruthy();
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -5802,6 +5802,200 @@ html .column.drag-over * {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* === Spec Editor === */
|
||||||
|
.spec-editor {
|
||||||
|
display: flex;
|
||||||
|
flex-direction: column;
|
||||||
|
height: 100%;
|
||||||
|
min-height: 0;
|
||||||
|
overflow: hidden;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Toolbar - fixed at top */
|
||||||
|
.spec-editor-toolbar {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: space-between;
|
||||||
|
padding: 8px 0;
|
||||||
|
border-bottom: 1px solid var(--border);
|
||||||
|
flex-shrink: 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-mode-toggle {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
gap: 4px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-actions {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
gap: 8px;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Content area - flexible and scrollable */
|
||||||
|
.spec-editor-content {
|
||||||
|
flex: 1;
|
||||||
|
min-height: 0;
|
||||||
|
overflow-y: auto;
|
||||||
|
overflow-x: hidden;
|
||||||
|
padding: 12px 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Textarea - fills available space in edit mode */
|
||||||
|
.spec-editor-textarea {
|
||||||
|
width: 100%;
|
||||||
|
min-height: 200px;
|
||||||
|
height: 100%;
|
||||||
|
flex: 1;
|
||||||
|
padding: 12px;
|
||||||
|
background: var(--bg);
|
||||||
|
border: 1px solid var(--border);
|
||||||
|
border-radius: var(--radius);
|
||||||
|
color: var(--text);
|
||||||
|
font-family: "SF Mono", Monaco, Consolas, monospace;
|
||||||
|
font-size: 13px;
|
||||||
|
line-height: 1.6;
|
||||||
|
resize: none;
|
||||||
|
outline: none;
|
||||||
|
transition: border-color 0.15s, box-shadow 0.15s;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-textarea:focus {
|
||||||
|
border-color: var(--todo);
|
||||||
|
box-shadow: 0 0 0 2px rgba(88, 166, 255, 0.15);
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-textarea:disabled {
|
||||||
|
opacity: 0.6;
|
||||||
|
cursor: not-allowed;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Empty state */
|
||||||
|
.spec-editor-empty {
|
||||||
|
padding: 24px;
|
||||||
|
text-align: center;
|
||||||
|
color: var(--text-muted);
|
||||||
|
font-size: 13px;
|
||||||
|
font-style: italic;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Keyboard hint - stays above revision section */
|
||||||
|
.spec-editor-hint {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
gap: 4px;
|
||||||
|
padding: 8px 0;
|
||||||
|
font-size: 12px;
|
||||||
|
color: var(--text-muted);
|
||||||
|
border-top: 1px solid var(--border);
|
||||||
|
flex-shrink: 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-hint kbd {
|
||||||
|
display: inline-block;
|
||||||
|
padding: 2px 6px;
|
||||||
|
background: var(--card);
|
||||||
|
border: 1px solid var(--border);
|
||||||
|
border-radius: 4px;
|
||||||
|
font-size: 11px;
|
||||||
|
font-family: "SF Mono", Monaco, Consolas, monospace;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* AI Revision section - fixed at bottom */
|
||||||
|
.spec-editor-revision {
|
||||||
|
padding: 16px 0;
|
||||||
|
border-top: 1px solid var(--border);
|
||||||
|
flex-shrink: 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-revision h4 {
|
||||||
|
font-size: 13px;
|
||||||
|
font-weight: 600;
|
||||||
|
margin: 0 0 4px 0;
|
||||||
|
color: var(--text);
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-revision-help {
|
||||||
|
font-size: 12px;
|
||||||
|
color: var(--text-muted);
|
||||||
|
margin: 0 0 8px 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-feedback {
|
||||||
|
width: 100%;
|
||||||
|
padding: 10px 12px;
|
||||||
|
background: var(--bg);
|
||||||
|
border: 1px solid var(--border);
|
||||||
|
border-radius: var(--radius);
|
||||||
|
color: var(--text);
|
||||||
|
font-family: inherit;
|
||||||
|
font-size: 13px;
|
||||||
|
line-height: 1.5;
|
||||||
|
resize: vertical;
|
||||||
|
outline: none;
|
||||||
|
transition: border-color 0.15s, box-shadow 0.15s;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-feedback:focus {
|
||||||
|
border-color: var(--todo);
|
||||||
|
box-shadow: 0 0 0 2px rgba(88, 166, 255, 0.15);
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-feedback:disabled {
|
||||||
|
opacity: 0.6;
|
||||||
|
cursor: not-allowed;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-revision-actions {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: space-between;
|
||||||
|
margin-top: 8px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-char-count {
|
||||||
|
font-size: 11px;
|
||||||
|
color: var(--text-muted);
|
||||||
|
font-family: "SF Mono", Monaco, Consolas, monospace;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Mobile responsive */
|
||||||
|
@media (max-width: 768px) {
|
||||||
|
.spec-editor-content {
|
||||||
|
padding: 8px 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-textarea {
|
||||||
|
min-height: 150px;
|
||||||
|
padding: 10px;
|
||||||
|
font-size: 14px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-toolbar {
|
||||||
|
padding: 6px 0;
|
||||||
|
flex-wrap: wrap;
|
||||||
|
gap: 8px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-hint {
|
||||||
|
padding: 6px 0;
|
||||||
|
font-size: 11px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-revision {
|
||||||
|
padding: 12px 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-revision h4 {
|
||||||
|
font-size: 12px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.spec-editor-revision-help {
|
||||||
|
font-size: 11px;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/* === Quick Entry Box === */
|
/* === Quick Entry Box === */
|
||||||
.quick-entry-box {
|
.quick-entry-box {
|
||||||
padding: 8px 10px;
|
padding: 8px 10px;
|
||||||
|
|||||||
Reference in New Issue
Block a user