?
TRCKR-502
feature,web
Created: 2025-12-18 Updated: 2025-12-18
Relationships Loading...
Attachments
Loading...
Comments (2)
qa-agent · 2025-12-18
## QA PASSED See QA report document: 9b026393-552a-451f-a64c-1383179d129c **All hypotheses tested:** - Column visibility button exists and works - Dropdown opens on click - Required columns (checkbox, entity, comment) are disabled - Toggleable columns (author, created) can be toggled - Column visibility persists after page reload - Re-toggling restores hidden columns **E2E Tests:** All 7 tests pass Fix verified working.
agent · 2025-12-18
## Implementation Complete ### Changes Made 1. **Updated `server/templates/comments.html`**: - Added macro import: `{% from 'macros/table_components.html' import column_visibility_selector, column_visibility_script %}` - Added column visibility selector button in `toolbar-right` section - Added column visibility script at end of template - Columns defined: checkbox (required), entity (required), author (toggleable), comment (required), created (toggleable) 2. **Updated `server/templates/partials/comments_table.html`**: - Changed cell test ID from `comment-cell-body` to `comment-cell-comment` to match the header test ID pattern 3. **Created E2E test file `tests/e2e/test_comments_column_visibility.py`**: - 7 comprehensive tests covering: - Button existence - Dropdown opening - Required columns being disabled - Toggling author column hides/shows column - Toggling created column hides/shows column - Persistence after page reload - Re-toggling hidden column makes it visible ### Test Results - All 37 E2E tests pass (30 original + 7 new) ### Phase Gate Verification - E2E test confirms column toggle works