detached e16502c

fix(daemon): reapply the session theme once the floating spare is published

commit
e16502ca1030f8c44c6ba6c4295ad2042102b0d9
author
brice · · 2026-10-08 08:56 UTC
committer
brice · · 2026-10-08 09:31 UTC
parent
1d86a91
signature
GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email
1 file · +7 -0
M internal/usecase/daemon/floating_spare.go +7 -0
@@ -85,9 +85,11 @@ func (d *Daemon) ensureFloatingSpare(sess *session, tb *tab) {
8585 // starts with the session theme, as a new tab does.
8686 d.applySessionThemeToPane(sess, p)
8787 }
88+ var published *pane
8889 sess.floatingLaunchMu.Lock()
8990 sess.floatingSpareStarting = false
9091 if p != nil && !sess.floatingLaunchStopping {
92+ published = p
9193 sess.floatingSpare = &floatingSpare{pane: p, key: spareKey(spec)}
9294 p.prestarted = true
9395 spare := p
@@ -108,6 +110,11 @@ func (d *Daemon) ensureFloatingSpare(sess *session, tb *tab) {
108110 if p != nil {
109111 closeFloatingPane(p)
110112 }
113+ if published != nil {
114+ // A theme change between the first apply and publication could
115+ // not see the spare; any later one does, so apply once more.
116+ d.applySessionThemeToPane(sess, published)
117+ }
111118 })
112119 }
113120
@@ -85,9 +85,11 @@ func (d *Daemon) ensureFloatingSpare(sess *session, tb *tab) {
85 // starts with the session theme, as a new tab does.85 // starts with the session theme, as a new tab does.
86 d.applySessionThemeToPane(sess, p)86 d.applySessionThemeToPane(sess, p)
87 }87 }
88+ var published *pane
88 sess.floatingLaunchMu.Lock()89 sess.floatingLaunchMu.Lock()
89 sess.floatingSpareStarting = false90 sess.floatingSpareStarting = false
90 if p != nil && !sess.floatingLaunchStopping {91 if p != nil && !sess.floatingLaunchStopping {
92+ published = p
91 sess.floatingSpare = &floatingSpare{pane: p, key: spareKey(spec)}93 sess.floatingSpare = &floatingSpare{pane: p, key: spareKey(spec)}
92 p.prestarted = true94 p.prestarted = true
93 spare := p95 spare := p
@@ -108,6 +110,11 @@ func (d *Daemon) ensureFloatingSpare(sess *session, tb *tab) {
108 if p != nil {110 if p != nil {
109 closeFloatingPane(p)111 closeFloatingPane(p)
110 }112 }
113+ if published != nil {
114+ // A theme change between the first apply and publication could
115+ // not see the spare; any later one does, so apply once more.
116+ d.applySessionThemeToPane(sess, published)
117+ }
111 })118 })
112 }119 }
113 120

Keyboard shortcuts

Single-key shortcuts are on. ctrl k and esc work whatever this says.

ctrl k :
go to a section, branch, tag, file or repository
?
this list
j k
move the row cursor down / up
enter
open the row under the cursor; on Files, a directory opens or closes in place
↑ ↓ ← → home end
in the file tree (focus on a row): previous / next row, close / open a directory or go to its parent / first entry, first / last row. They never open a file
l h
on Files: open a directory or a file / close a directory or go up to its parent, then to the parent page. Elsewhere: into the side panel (on a merge request, its files) / back to the content; there j k move between its controls
g g G
first / last row
/
filter the list; on Files, find a file anywhere in the repository (esc clears, enter opens the best match)
b
switch branch or tag
g s g f g c g b g m g i g r g a
tabs: summary, files, commits, branches & tags, merge requests, issues, releases, actions (when available)
g h
home
w
wide mode: use the whole width of the screen, or not (remembered)
y
copy a permalink pinned to the commit
d u x
diff: split / unified / collapse or expand all
e E
diff: the whole file in view / every file, with the lines the hunks hide
n
merge request: next note in the diff
esc
close a dialog, leave a field

Branches 12

Tags 11

↑↓ moveenter openesc close