fix(jetbrains): address session review feedback

This commit is contained in:
kirillk
2026-05-22 14:48:12 -04:00
parent c9126435ff
commit 2e753bb039
2 changed files with 4 additions and 4 deletions
+2 -2
View File
@@ -366,11 +366,11 @@ For common spacing lookups, prefer `JBUI.CurrentTheme` area-specific insets (e.g
| Simple `BorderLayout` panels | `JBUI.Panels.simplePanel(...)`, `BorderLayoutPanel` |
| Simple vertical custom Swing groups | `VerticalLayout` |
| Fluent platform panels | `JBPanel.withBorder(...)`, `.andTransparent()`, `.andOpaque()`, `.withBackground(...)` |
| Single-component alignment wrapper | `ai.kilocode.client.ui.Align` — see section below |
| Single-component alignment wrapper | `ai.kilocode.client.ui.layout.Align` — see section below |
### Align — Single-Component Alignment Wrapper
Use `Align` (`ai.kilocode.client.ui.Align`) when a single Swing component must be positioned inside available space without adding visual chrome. It is a transparent, no-border, no-color `JPanel(null)` that lays out its one child according to independent horizontal (`HAlign`) and vertical (`VAlign`) modes. `CenterShrinkPanel` has been removed; use `child.alignCenter()` as a direct replacement.
Use `Align` (`ai.kilocode.client.ui.layout.Align`) when a single Swing component must be positioned inside available space without adding visual chrome. It is a transparent, no-border, no-color `JPanel(null)` that lays out its one child according to independent horizontal (`HAlign`) and vertical (`VAlign`) modes. `CenterShrinkPanel` has been removed; use `child.align(HAlign.CENTER, VAlign.CENTER)` as a direct replacement.
**Alignment modes:**
@@ -493,10 +493,10 @@ class SessionController(
}
}
recoverPending(id)
for (child in discovered) trackChild(child)
runEdt {
if (disposed) return@runEdt
if (sid != id) return@runEdt
for (child in discovered) trackChild(child)
showSession()
loaded(!model.isEmpty())
}
@@ -540,9 +540,9 @@ class SessionController(
}
}
recoverPending(session.id)
for (child in discovered) trackChild(child)
runEdt {
if (disposed) return@runEdt
for (child in discovered) trackChild(child)
subscribeEvents()
showSession()
loaded(!model.isEmpty())