#33047 closed defect (bug) (fixed)
Table headers shouldn't be empty or shouldn't be table headers
Reported by: | afercia | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | normal | Version: | 4.2 |
Component: | Administration | Keywords: | |
Focuses: | accessibility | Cc: |
Description
Spotted a couple of places where some th are empty. The th element conveys a semantic meaning and it's used by assistive technologies to establish a relationship with the related table cells. This implies a table header shouldn't be empty. Most of all, when used just for layout purposes, shouldn't be a th in the first place and a td should be used instead.
Attachments (2)
Change History (7)
comment:3 @afercia — 2 months ago
Refreshed patch now that the user session table header has some proper text, see see 33368. See screenshot:
comment:4 @SergeyBiryukov — 2 months ago
- Resolution set to fixed
- Status changed from assigned to closed
In 33456:
comment:5 @SergeyBiryukov — 2 months ago
- Milestone changed from Awaiting Review to 4.3
Note: See
TracTickets for help on using
tickets.
As I see it, there are two possible ways to solve this:
The proposed patch takes the second approach. Open to hear any thoughts and feedback.