Skip to content

Commit

Permalink
Add comment about key check
Browse files Browse the repository at this point in the history
  • Loading branch information
miniscruff committed Jun 8, 2024
1 parent b893f16 commit 112ee62
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions ui/ui.go
Original file line number Diff line number Diff line change
Expand Up @@ -291,6 +291,9 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
}
return m, cmd

// I do not think this state machine works quite well enough with custom keybinds
// as it kinda breaks down if you use two different keys across the views.
// Is it worth refactoring this to first check the "view" before the key?
case key.Matches(msg, keys.IssueKeys.Assign), key.Matches(msg, keys.PRKeys.Assign):
m.sidebar.IsOpen = true
if m.ctx.View == config.PRsView {
Expand Down

0 comments on commit 112ee62

Please sign in to comment.