Skip to content

camel-jbang - TUI top mode, diagram toggle, and Shift+Tab navigation#23358

Merged
davsclaus merged 5 commits into
mainfrom
worktree-more-tui-4
May 20, 2026
Merged

camel-jbang - TUI top mode, diagram toggle, and Shift+Tab navigation#23358
davsclaus merged 5 commits into
mainfrom
worktree-more-tui-4

Conversation

@davsclaus
Copy link
Copy Markdown
Contributor

Summary

  • Add t key toggle on routes tab for top mode — shows performance metrics (mean/max/min/last/delta) with color-coded timing values (red >= 1000ms, yellow >= 100ms) and delta indicators (red slower, green faster)
  • Top mode processor table with flat sort by slowest, inline bar chart column scaled to max value
  • Sort columns in top mode limited to timing metrics only (mean/max/min/last/delta) via s/S keys
  • Tab label switches to 3 Top when top mode is active; hides irrelevant hints (d/D/a/c/p/P)
  • Add a key toggle for all/single route diagram mode on the route table screen
  • Shift+Tab cycles tabs backwards (fixed by checking isFocusPrevious() before isFocusNext())
  • Bold route titles with double empty line separators in text ASCII diagrams

Test plan

  • Toggle top mode with t on routes tab — verify timing columns, color coding, sort indicators
  • Press s/S to cycle sort in top mode — verify only timing columns cycle
  • Check inline bar chart in processor table scales correctly
  • Toggle a for all/single route diagram, then press d/D to view diagram
  • Verify Shift+Tab navigates tabs backwards
  • Verify tab label shows Top vs Route when toggling

Claude Code on behalf of Claus Ibsen

🤖 Generated with Claude Code

davsclaus and others added 5 commits May 20, 2026 09:42
Add 'a' key on route tab to toggle between showing all routes or just the
selected route in the diagram view. Route titles are rendered in bold in
text diagram mode, and routes are separated by double empty lines.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…lumns

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…nements

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions Bot added the dsl label May 20, 2026
@davsclaus davsclaus merged commit d9d387b into main May 20, 2026
6 checks passed
@davsclaus davsclaus deleted the worktree-more-tui-4 branch May 20, 2026 09:17
@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • dsl/camel-jbang/camel-jbang-plugin-tui
All tested modules (2 modules)
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: Launcher :: Container

⚙️ View full build and test results

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant