CAMEL-23648: camel-jbang - TUI redesign Send Message, Classpath tab, diagram improvements, and F1 help#23654
Merged
Merged
Conversation
…n view Redesign SendMessagePopup from a small centered popup to a full-screen 3-panel layout (request/response/history) matching the HTTP Probe pattern. InOut responses now show exchange headers and scrollable body with pretty print toggle. Adds send history with replay support. Extract shared form utilities (HeaderEntry, handleTextInput, renderLabel, handlePaste) into FormHelper to eliminate duplication between SendMessagePopup and HttpTab. Add Clear + padding to both views for visual separation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…nspect badge count Remove redundant 1-9/1-0/1-2 tab switching hints from footer across all tabs since the tab bar at the top already shows available tabs. Fix the Inspect tab badge to show count of distinct exchanges instead of individual processing steps. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add a Reset Screen action to the F2 actions menu (Diagnostics group) that clears the terminal and resets TamboUI buffers, forcing a full redraw. Useful when the screen gets garbled from external interference. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…mples repo Extend the sync-example-catalog Maven profile to also download source files for all bundled examples from apache/camel-jbang-examples. Add examples/**/* excludes to both RAT and mycila license plugins so synced files don't need license headers. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fix missing Reset Screen entry in visual menu list, use dynamic visualActionCount instead of hardcoded sums, and use 🧹 emoji instead of 🖥️ which has a variation selector causing width mismatch. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…uge bars Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
Each tab now provides rich markdown help documentation accessible via F1, explaining all columns, metrics, and features in detail for users who may not be familiar with Camel or JVM internals. Includes ASCII art diagrams for circuit breaker states, memory gauges, and endpoint flow charts. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…examples Each tab's help text now includes real example screens, detailed metric explanations traced from DevConsole sources, diagnostic guidance, and educational content for users new to Camel and JVM internals. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
🧪 CI tested the following changed modules:
💡 Manual integration tests recommended:
All tested modules (9 modules)
|
Shrink the route diagram example to fit more easily in the help overlay viewport, fix when[cond] box alignment, and add scroll hint until TamboUI supports partial code block rendering. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Move Classpath from F2 action popup to its own tab in the More menu - Delete ClasspathPopup, create ClasspathTab implementing MonitorTab - Replace fragile int action constants with Action enum in ActionsPopup - Fix GROUP_SIZES after classpath removal (4,5,5 -> 4,4,5) - Rename "Show Documentation" to "Show Integration Doc" - Clear full area in doc viewer to prevent background bleed-through - Remove unnecessary F5 reload from ClasspathTab (classpath is static) - Add PgUp/PgDn navigation to ClasspathTab - Fix diagram flicker: skip loading placeholder on metric refreshes - Preserve diagram scroll position across F5 and metric refreshes - Bump screenshot notification from 3s to 5s Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Enable auto-refresh for both text and image diagrams when metrics are on, removing the need for manual F5. Use quiet reload for the m toggle to avoid flicker when switching metrics on/off. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…abled The drawArrowFromMerge method read stats unconditionally, causing the last node counter to appear even with metrics off. Gate it behind the metrics flag like drawArrow already does. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…Infra Services Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
drawArrowFromMergenot respecting metrics flagActionenum, fix GROUP_SIZES, rename "Show Documentation" to "Show Integration Doc", clear full area in doc viewergkey to trigger GC, dual gauge bars for heap usageClaude Code on behalf of Claus Ibsen