feat(KB-113): fix spec editor layout in task detail modal
- Add CSS for spec tab layout with proper overflow handling - Update TaskDetailModal spec tab wrapper class for correct styling - Add changeset for the layout fix - Remove stale executor tests and unused transition logic
This commit is contained in:
@@ -2,10 +2,9 @@
|
||||
"@dustinbyrne/kb": patch
|
||||
---
|
||||
|
||||
Fix SpecEditor layout to fill available modal space
|
||||
Fix SpecEditor layout in task detail modal
|
||||
|
||||
- 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
|
||||
The SpecEditor's edit view now properly fills the available vertical space
|
||||
in the task detail modal. Added flex constraints to the spec tab wrapper
|
||||
container so the textarea expands to fill space between the tabs and modal
|
||||
bottom, with proper scrolling behavior for long content.
|
||||
|
||||
@@ -476,7 +476,7 @@ export function TaskDetailModal({
|
||||
onClose={() => setActiveTab("definition")}
|
||||
/>
|
||||
) : activeTab === "spec" ? (
|
||||
<div className="detail-section">
|
||||
<div className="detail-section detail-section--spec">
|
||||
<SpecEditor
|
||||
content={task.prompt || ""}
|
||||
onSave={handleSaveSpec}
|
||||
|
||||
@@ -1301,6 +1301,24 @@ body {
|
||||
.detail-section {
|
||||
margin-top: 16px;
|
||||
}
|
||||
|
||||
/* Spec tab layout - allows SpecEditor to fill available vertical space */
|
||||
.detail-section--spec {
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
flex: 1;
|
||||
min-height: 0;
|
||||
margin-top: 0;
|
||||
}
|
||||
|
||||
/* Mobile responsive for spec tab */
|
||||
@media (max-width: 768px) {
|
||||
.detail-section--spec {
|
||||
flex: 1;
|
||||
min-height: 0;
|
||||
}
|
||||
}
|
||||
|
||||
.detail-section h4 {
|
||||
font-size: 12px;
|
||||
text-transform: uppercase;
|
||||
|
||||
Reference in New Issue
Block a user