Remove work overlay from TUI and consolidate functionality into work tabs and detail panel #161
Conversation
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
- Delete tui_panel_work_overlay.go file - Remove PanelWorkOverlay and ViewWorkOverlay enums - Remove StatusBarContextWorkOverlay enum - Remove workOverlay field from planModel - Remove '0' hotkey that opened the overlay - Update work selection to use 1-9 keys with work tabs - Add findWorkByID helper to replace overlay methods - Track work tiles in planModel for tabs bar - Rename calculateWorkOverlayHeight to calculateWorkPanelHeight - Update all comments and messages to remove overlay references - Add 1-9 keys documentation to help text Work selection now handled through work tabs (1-9 keys) and work details panel.
- Added creation time display to work header section showing age (Xm/Xh/Xd ago) - Added prominent progress percentage display with color-coded styling - Added warning badges for unassigned beads and pending PR feedback - Created comprehensive Work Overview section when root issue is selected - Enhanced root issue details with work statistics and task breakdown - Preserved all metadata from removed overlay in accessible format
…anel - Added WorkDetailActionDestroy to the action enum - Added 'd' hotkey handler to WorkDetailsPanel.Update() - Implemented confirmation dialog for work destruction using ViewDestroyConfirm - Added validation to prevent destroying work that's currently processing - Added '[d]estroy' button to status bar in work detail context - Updated button detection for click support
- Added new PanelWorkTabs panel constant for work tabs bar focus state - Updated Tab/Shift+Tab to cycle through three panels: work tabs, work details, and issues - Added keyboard navigation when work tabs panel is focused (arrow keys to select different works, enter to jump to details) - Added visual focus indicator on work tabs bar when it's the active panel - Updated mouse click handling to set focus to work tabs when clicking on the bar - Work selection via number keys (1-9) continues to work from all panels - Mouse support for tab selection continues to work correctly The work tabs bar now handles all work selection logic exclusively, with proper focus management and three-panel navigation support.
… removal - Verified work selection via number keys (1-9) - Verified mouse click selection on tabs - Verified work destruction from detail panel - Verified Tab/Shift+Tab navigation between panels - Verified all work metadata displays correctly - Verified edge cases (no works, single work, many works) - Confirmed no orphaned code or dead references - Created comprehensive test plan documentation All tests pass - TUI fully functional without work overlay
- Fix work details panel height mismatch between left and right panels - Left panel uses Height(contentHeight - 1) - Right panel uses Height(contentHeight - 3) - Compensates for lipgloss rendering differences with different widths - Add scrolling capability to work details right panel - Track scroll offset and total lines - Add keyboard controls (j/k) for scrolling when focused - Add scroll indicators showing position - Add focus management for work details panels - Tab/Shift+Tab cycles between left and right panels - Visual focus indicator with orange border The height adjustments ensure both panels render at the same height despite lipgloss adding different amounts of padding based on panel width. Co-Authored-By: Claude Opus 4.1 <noreply@anthropic.com>
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
This PR completes the removal of the work overlay from the TUI interface, consolidating its functionality into the existing work tabs bar and work detail panel. This simplification improves the user experience by reducing the number of view modes and making work management more intuitive.
Changes
🗑️ Removed Components
tui_panel_work_overlay.go(617 lines removed)ViewWorkOverlayview mode from the TUI✨ Enhanced Work Detail Panel
🔄 Refactored Work Selection
📚 Documentation and Testing
tui_test_plan.md)🧹 Code Cleanup
tui_plan.go(simplified state management, ~125 lines removed)tui_panel_status.go(removed overlay references)tui_plan_render.go(simplified rendering logic)Resolved Issues
Testing Performed
Comprehensive testing was completed (documented in
tui_test_plan.md), including:Breaking Changes
Review Notes
The code has undergone a review cycle with no issues identified. The implementation successfully consolidates all overlay functionality while simplifying the codebase by removing ~387 net lines of code.
Screenshots/Demo
The TUI now provides a cleaner, more intuitive interface:
This PR represents a significant simplification of the TUI architecture while preserving all essential functionality.