refactor(devplanner): optimize UI layout and code structure #352

- Adjust UI components' borders and spacing for better visual consistency
- Refactor some code for improved readability and maintainability
- Update Chinese translation for a more natural reading experience

Author: phodal

core/src/main/kotlin/cc/unitmesh/devti/gui/planner/AutoDevPlannerToolWindow.kt

@@ -31,7 +31,7 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
 
     private var markdownEditor: MarkdownLanguageField? = null
     private val contentPanel = JBUI.Panels.simplePanel()
-    
+
     private var currentView: PlannerView? = null
     private var currentCallback: ((String) -> Unit)? = null
     private val plannerResultSummary = PlannerResultSummary(project, mutableListOf())
@@ -43,10 +43,10 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
                 ActionUtil.getAction("AutoDevPlanner.ToolWindow.TitleActions") as ActionGroup,
                 true
             )
-            
+
         val toolbarPanel = JPanel(FlowLayout(FlowLayout.RIGHT))
         toolbarPanel.add(toolbar.component)
-        
+
         add(toolbarPanel, BorderLayout.NORTH)
         add(contentPanel, BorderLayout.CENTER)
 
@@ -58,14 +58,14 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
 
         connection.subscribe(PlanUpdateListener.Companion.TOPIC, object : PlanUpdateListener {
             override fun onPlanUpdate(items: MutableList<AgentTaskEntry>) {
-                if (currentView is PlanSketchView) {
-                    runInEdt {
-                        planLangSketch.updatePlan(items)
-                        contentPanel.components.find { it is LoadingPanel }?.let {
-                            contentPanel.remove(it)
-                            contentPanel.revalidate()
-                            contentPanel.repaint()
-                        }
+                switchToPlanView()
+
+                runInEdt {
+                    planLangSketch.updatePlan(items)
+                    contentPanel.components.find { it is LoadingPanel }?.let {
+                        contentPanel.remove(it)
+                        contentPanel.revalidate()
+                        contentPanel.repaint()
                     }
                 }
             }
@@ -92,7 +92,7 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
 
         currentView = view
         view.initialize(this)
-        
+
         contentPanel.revalidate()
         contentPanel.repaint()
     }
@@ -118,16 +118,16 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
     override fun dispose() {
         markdownEditor = null
     }
-    
+
     interface PlannerView {
         val viewType: PlannerViewType
         fun initialize(window: AutoDevPlannerToolWindow)
     }
-    
+
     enum class PlannerViewType {
         PLAN, EDITOR, ISSUE_INPUT, LOADING
     }
-    
+
     inner class PlanSketchView : PlannerView {
         override val viewType = PlannerViewType.PLAN
         override fun initialize(window: AutoDevPlannerToolWindow) {
@@ -138,12 +138,12 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
                         .resizableColumn()
                 }
             }
-            
+
             contentPanel.add(planPanel, BorderLayout.CENTER)
             contentPanel.add(plannerResultSummary, BorderLayout.SOUTH)
         }
     }
-    
+
     inner class EditPlanView : PlannerView {
         override val viewType = PlannerViewType.EDITOR
         override fun initialize(window: AutoDevPlannerToolWindow) {
@@ -165,11 +165,11 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
             contentPanel.add(editPlanViewPanel, BorderLayout.CENTER)
         }
     }
-    
+
     inner class IssueInputView : PlannerView {
         override val viewType = PlannerViewType.ISSUE_INPUT
         private lateinit var viewPanel: IssueInputViewPanel
-        
+
         override fun initialize(window: AutoDevPlannerToolWindow) {
             viewPanel = IssueInputViewPanel(
                 project,
@@ -182,13 +182,13 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
                     switchToPlanView()
                 }
             )
-            
+
             contentPanel.add(viewPanel, BorderLayout.CENTER)
             viewPanel.setText("")
             viewPanel.requestTextAreaFocus()
         }
     }
-    
+
     inner class LoadingView : PlannerView {
         override val viewType = PlannerViewType.LOADING
         override fun initialize(window: AutoDevPlannerToolWindow) {
@@ -199,7 +199,7 @@ class AutoDevPlannerToolWindow(val project: Project) : SimpleToolWindowPanel(tru
                         .resizableColumn()
                 }
             }
-            
+
             contentPanel.add(planPanel, BorderLayout.CENTER)
             contentPanel.add(LoadingPanel(project), BorderLayout.NORTH)
         }

core/src/main/kotlin/cc/unitmesh/devti/gui/planner/PlannerResultSummary.kt

@@ -70,25 +70,23 @@ class PlannerResultSummary(
     }
 
     init {
-        background = JBUI.CurrentTheme.ToolWindow.background()
-        border = JBUI.Borders.empty(10)
+        border = JBUI.Borders.customLine(UIUtil.getBoundsColor(), 1, 0, 0, 0)
 
         val titlePanel = JPanel(BorderLayout()).apply {
-            isOpaque = false
-            border = JBUI.Borders.emptyBottom(10)
+            border = JBUI.Borders.compound(
+                JBUI.Borders.customLine(UIUtil.getBoundsColor(), 0, 0, 1, 0),
+                JBUI.Borders.empty(4)
+            )
 
             val titleLabelPanel = JPanel(BorderLayout()).apply {
-                isOpaque = false
                 add(JBLabel(AutoDevBundle.message("planner.change.list.title")).apply {
                     foreground = UIUtil.getLabelForeground()
                     font = JBUI.Fonts.label().asBold()
                 }, BorderLayout.WEST)
                 add(statsLabel, BorderLayout.EAST)
             }
 
-            val actionsPanel = JPanel(FlowLayout(FlowLayout.RIGHT, 5, 0)).apply {
-                isOpaque = false
-
+            val actionsPanel = JPanel(FlowLayout(FlowLayout.RIGHT, 4, 0)).apply {
                 val discardAllButton = HyperlinkLabel(AutoDevBundle.message("planner.action.discard.all")).apply {
                     icon = AllIcons.Actions.Cancel
                     addHyperlinkListener(object : HyperlinkListener {
@@ -139,13 +137,13 @@ class PlannerResultSummary(
         changesPanel.removeAll()
 
         if (changes.isEmpty()) {
-            statsLabel.text = AutoDevBundle.message("planner.stats.no.changes")
+            statsLabel.text = " - " + AutoDevBundle.message("planner.stats.no.changes")
             changesPanel.add(JBLabel(AutoDevBundle.message("planner.no.code.changes")).apply {
                 foreground = UIUtil.getLabelDisabledForeground()
-                border = JBUI.Borders.empty(10)
+                border = JBUI.Borders.empty(8)
             })
         } else {
-            statsLabel.text = AutoDevBundle.message("planner.stats.changes.count", changes.size)
+            statsLabel.text = " - " + AutoDevBundle.message("planner.stats.changes.count", changes.size)
             changes.forEach { change ->
                 val filePath = change.virtualFile?.relativePath(project) ?: "Unknown"
                 val fileName = filePath.substringAfterLast('/')
@@ -167,7 +165,7 @@ class PlannerResultSummary(
         return JPanel().apply {
             isOpaque = true
             background = UIUtil.getListBackground()
-            border = JBUI.Borders.empty(5, 8)
+            border = JBUI.Borders.empty(4, 8)
             layout = BoxLayout(this, BoxLayout.X_AXIS)
 
             val changeIcon = when (change.type) {
@@ -190,7 +188,7 @@ class PlannerResultSummary(
             }
 
             add(fileLabel)
-            add(Box.createHorizontalStrut(5))
+            add(Box.createHorizontalStrut(4))
 
             val pathLabel = JBLabel(filePath).apply {
                 foreground = UIUtil.getLabelDisabledForeground()

core/src/main/resources/messages/AutoDevBundle_zh.properties

@@ -187,7 +187,7 @@ planner.stats.changes.empty=没有变更
 planner.change.list.title=变更列表
 planner.action.discard.all=丢弃全部
 planner.action.accept.all=接受全部
-planner.stats.no.changes= 没有变更
+planner.stats.no.changes=没有变更
 planner.no.code.changes=没有代码变更
 planner.stats.changes.count= (共 {0} 个文件变更)
 planner.action.view.changes=查看变更