Skip to content

Conversation

@matleh
Copy link
Contributor

@matleh matleh commented Jan 23, 2026

Refs: beans-k5mz

Summary

  • Adds 'H' (shift+h) key binding to toggle visibility of completed/scrapped beans in the TUI
  • Uses existing GraphQL ExcludeStatus filter
  • Works independently from tag filter - both can be active simultaneously
  • List title shows active filters: [hiding completed]
  • Changed from 'h' to 'H' to avoid conflict with vim navigation keys

Implementation Details

  • Added hideCompleted boolean field to listModel struct
  • Modified loadBeans() to use GraphQL ExcludeStatus filter when toggle is active
  • Added toggleHideCompleted() method to flip the state
  • Implemented 'H' key binding handler to toggle the filter
  • Updated list title to show both tag and hide-completed filters when active
  • Added 'H' shortcut to help overlay (? key)
  • Updated both View() and ViewConstrained() methods for consistency

Testing

  • Verified GraphQL ExcludeStatus filter correctly excludes completed/scrapped beans
  • Tested toggle functionality in TUI
  • Build successful with no compilation errors

@matleh
Copy link
Contributor Author

matleh commented Jan 23, 2026

Only after I opened the PR I saw that there is already a different concept at #41 .

- Add hideCompleted boolean field to listModel
- Use GraphQL ExcludeStatus filter when toggle is active
- Update list title to show active filters
- Add 'H' shortcut (shift+h) to help overlay to avoid conflict with vim 'h' motion
- Both tag and hide-completed filters can be active simultaneously
- Default state shows all beans (matching current behavior)
@matleh matleh changed the title feat: add toggle to hide completed/scrapped beans in TUI feat: add 'H' toggle to hide completed/scrapped beans in TUI Jan 23, 2026
@matleh matleh force-pushed the hide-completed-toggle branch from 7a0dcb5 to 3ca3155 Compare January 23, 2026 14:24
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.

1 participant