refactor(ui): clean up plan content and improve task panel layout #331

phodal · phodal · commit ca041d5a9eaa · 2025-03-17T11:08:42.000+08:00
- Remove hardcoded plan content in `AutoDevPlanerToolWindowFactory`.
- Set minimum size for plan sketch panel to improve UI responsiveness.
- Refactor `TaskPanel` layout to use `BoxLayout` for better alignment.
- Simplify task label HTML rendering and adjust background colors.
diff --git a/core/src/main/kotlin/cc/unitmesh/devti/gui/AutoDevPlanerToolWindowFactory.kt b/core/src/main/kotlin/cc/unitmesh/devti/gui/AutoDevPlanerToolWindowFactory.kt
@@ -56,29 +56,7 @@ class AutoDevPlanerTooWindow(val project: Project) : SimpleToolWindowPanel(true,
     override fun getName(): @NlsActions.ActionText String? = "AutoDev Planer"
     var connection = ApplicationManager.getApplication().messageBus.connect(this)
 
-    val content = """1. 更新实体类 BlogPost
-    - [✓] 添加 category 字段
-    - [✓] 更新构造函数
-    - [✓] 添加 getter 和 setter
-
-2. 更新 DTO 类 CreateBlogRequest
-    - [✓] 添加 category 字段
-
-3. 更新 BlogService
-    - [✓] 确保 createBlog 方法正确处理 category 字段
-
-4. 更新 BlogRepository
-    - [✓] 添加 findByCategory 方法
-
-5. 更新 BlogController
-    - [✓] 添加按 category 查询的 API 端点
-    - [✓] 更新 Swagger 文档
-
-6. 测试验证
-    - [ ] 编写 Postman 测试用例
-    - [ ] 创建带 category 参数的集成测试
-    - [ ] 验证空分类场景处理
-"""
+    val content = ""
     var planSketch: PlanSketch = PlanSketch(project, content, MarkdownPlanParser.parse(content).toMutableList(), true)
 
     init {
diff --git a/core/src/main/kotlin/cc/unitmesh/devti/sketch/ui/plan/PlanSketch.kt b/core/src/main/kotlin/cc/unitmesh/devti/sketch/ui/plan/PlanSketch.kt
@@ -11,6 +11,7 @@ import com.intellij.ui.components.JBPanel
 import com.intellij.util.ui.JBEmptyBorder
 import com.intellij.util.ui.JBUI
 import java.awt.BorderLayout
+import java.awt.Dimension
 import javax.swing.Box
 import javax.swing.BoxLayout
 import javax.swing.JComponent
@@ -97,6 +98,7 @@ class PlanSketch(
         planController.renderPlan()
         add(contentPanel, BorderLayout.CENTER)
 
+        minimumSize = Dimension(200, 0)
         background = JBUI.CurrentTheme.ToolWindow.background()
     }
 
diff --git a/core/src/main/kotlin/cc/unitmesh/devti/sketch/ui/plan/TaskPanel.kt b/core/src/main/kotlin/cc/unitmesh/devti/sketch/ui/plan/TaskPanel.kt
@@ -13,6 +13,7 @@ import com.intellij.util.ui.JBUI
 import java.awt.Dimension
 import java.awt.FlowLayout
 import javax.swing.BorderFactory
+import javax.swing.BoxLayout
 import javax.swing.JButton
 import javax.swing.JComponent
 import javax.swing.JLabel
@@ -26,10 +27,11 @@ class TaskPanel(
     private val project: Project,
     private val task: AgentPlanStep,
     private val onStatusChange: () -> Unit
-) : JBPanel<JBPanel<*>>(FlowLayout(FlowLayout.LEFT, 2, 0)) {
+) : JBPanel<TaskPanel>() {
     private val taskLabel: JLabel
 
     init {
+        layout = BoxLayout(this, BoxLayout.X_AXIS)
         border = JBUI.Borders.empty(4, 16, 4, 0)
         background = JBUI.CurrentTheme.ToolWindow.background()
         taskLabel = createStyledTaskLabel()
@@ -67,9 +69,9 @@ class TaskPanel(
     private fun createExecuteButton(): JButton {
         return JButton(AllIcons.Actions.Execute).apply {
             border = BorderFactory.createEmptyBorder()
-            isOpaque = true
             preferredSize = Dimension(20, 20)
             toolTipText = "Execute"
+            background = JBUI.CurrentTheme.ToolWindow.background()
 
             addActionListener {
                 AutoDevToolWindowFactory.Companion.sendToSketchToolWindow(project, ChatActionType.SKETCH) { ui, _ ->
@@ -81,16 +83,16 @@ class TaskPanel(
 
     private fun createStyledTaskLabel(): JLabel {
         val labelText = getLabelTextByStatus()
-        return JLabel(labelText).apply {
+        return JLabel("<html>$labelText</html>").apply {
             border = JBUI.Borders.emptyLeft(5)
         }
     }
 
     private fun getLabelTextByStatus(): String {
         return when (task.status) {
-            TaskStatus.COMPLETED -> "<html><strike>${task.step}</strike></html>"
-            TaskStatus.FAILED -> "<html><span style='color:red'>${task.step}</span></html>"
-            TaskStatus.IN_PROGRESS -> "<html><span style='color:blue;font-style:italic'>${task.step}</span></html>"
+            TaskStatus.COMPLETED -> "<strike>${task.step}</strike>"
+            TaskStatus.FAILED -> "<span style='color:red'>${task.step}</span>"
+            TaskStatus.IN_PROGRESS -> "<span style='color:blue;font-style:italic'>${task.step}</span>"
             TaskStatus.TODO -> task.step
         }
     }
