←
TRCKR-502
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