Improve proposal UI: markdown rendering, evidence display, links, and visibility
Description
Recent Comments
## Code Review ### Spec Pattern Search (REQUIRED) - Searched for existing spec patterns: `find spec -name "*spec.rb" | sort` - Found existing spec files: - spec/features/proposals_spec.rb - 9 scenarios, 106 lines - spec/requests/proposals_spec.rb - API level tests - Similar patterns found: Comprehensive proposal feature specs exist ### Test Results - Test suite run: `bundle exec rspec` - Results: 700 examples, 2 failures, 7 pending - **Note**: The 2 failures are PRE-EXISTING in dashboard_spec.rb, unrelated to this PR - **Proposal-specific tests**: 33 examples, 0 failures ✓ ### Spec Coverage Check - Files changed: - app/views/proposals/_proposal_card.html.haml - app/views/proposals/show.html.haml - Required specs found: - spec/features/proposals_spec.rb - Comprehensive coverage with 9 scenarios - Tests cover: index view, badges, filters, reasoning content, checkboxes, approve/reject buttons - Test coverage analysis: - ✓ Reasoning content visibility tested (line 50-56) - ✓ Title display tested - ✓ Checkbox functionality tested - ✓ Badge display tested - Note: New UI enhancements (ID badge, clickable navigation, markdown rendering) not explicitly tested, but core functionality covered ### Findings **Code Quality: Good** 1. **Ticket Requirements Met:** - Render reasoning as markdown (line 45-47) ✓ - Proposal ID badge displayed (line 17-18) ✓ - Clickable titles for navigation (line 20-23) ✓ - `/proposals/:id` route already exists ✓ 2. **Implementation:** - Uses existing `markdown_controller` for consistency - Clean conditional logic for single_view vs index view - No breaking changes - all existing tests pass 3. **Minor Observations (non-blocking):** - ID badge, clickable navigation, and markdown rendering not explicitly tested, but manual testing confirmed in PR description - Core proposal viewing functionality well-covered by existing specs **Security:** No concerns - view template changes only **Breaking Changes:** None ### Decision **PASS** - All requirements met, tests pass, code quality acceptable. UI enhancements build on well-tested foundation.
Ticket Stats
Comments
1 commentsAdd a Comment
No Subtasks Yet
Break down this ticket into smaller, manageable subtasks