Skip to content

Add Support for BackTab#57

Open
strategicpause wants to merge 1 commit intoavirtuos:mainfrom
strategicpause:shift-tab
Open

Add Support for BackTab#57
strategicpause wants to merge 1 commit intoavirtuos:mainfrom
strategicpause:shift-tab

Conversation

@strategicpause
Copy link

@strategicpause strategicpause commented Mar 9, 2026

This helps with navigating both forward and backwards in the clawmux interface. Also using this as an opportunity to play around with clawmux, hence including 11.2 as a completed task.

Add a KeyCode::BackTab arm in handle_task_list_input that decrements
app.active_tab with wrapping using (active_tab + 8) % 9.

Includes tests for backward cycling and wrap-around from first to last
tab.
@strategicpause strategicpause marked this pull request as ready for review March 9, 2026 05:16
@avirtuos avirtuos self-assigned this Mar 9, 2026
Copy link
Owner

@avirtuos avirtuos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good. One change to consider before I think this can be merged.

KeyCode::Tab => {
app.active_tab = (app.active_tab + 1) % 9;
}
KeyCode::BackTab => {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BackTab is already used by the Research Tab to switch between Plan and Act modes. I like your proposal to use BackTab for tab navigation. Do you mind changing the short-cut used for Plan/Act in the research tab as part of this PR? Maybe just a simple Shift+p to switch modes?

Copy link
Author

@strategicpause strategicpause Mar 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a commit missing? I only see KeyCode::Tab referenced here and KeyCode::BackTab is only referenced in this PR. There also doesn't seem to be a research Plan & Act mode in the research tab which suggests something might be missing from the main branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants