[Django] #34919: Missing scope attribute in admin doc table headers

16 views
Skip to first unread message

Django

unread,
Oct 20, 2023, 12:18:28 PM10/20/23
to django-...@googlegroups.com
#34919: Missing scope attribute in admin doc table headers
-------------------------------------+-------------------------------------
Reporter: Louis- | Owner: Louis-Cloud
Cloud |
Type: Bug | Status: assigned
Component: | Version: 4.2
contrib.admindocs | Keywords: table, scope,
Severity: Normal | screenreader
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 1 |
-------------------------------------+-------------------------------------
There are a number of tables throughout [https://static-django-
demo.netlify.app/django_admin_tests/v5.0/english/admin/docs/models/contenttypes.contenttype/
Django's admin docs] that are lacking the `scope` attribute. Having this
attribute in the table headers makes table navigation easier for a screen
reader user.

=== Relevant Files:
* django/contrib/admindocs/templates/admin_doc/model_detail.html
* django/contrib/admindocs/templates/admin_doc/model_index.html

=== Acceptance Criteria
* All table column header (th) needs to have a `scope = "col"` attribute.
* All table row headers (first column th) should have a `scope = "row"`
attribute

=== For more information
* [https://www.w3.org/WAI/tutorials/tables/two-headers/#table-with-header-
cells-in-the-top-row-and-first-column W3 - Tables with Two Headers]:
* [https://developer.mozilla.org/en-
US/docs/Learn/HTML/Tables/Advanced#using_column_and_row_headers MDN Web
Docs - Using column and row headers]

--
Ticket URL: <https://code.djangoproject.com/ticket/34919>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Oct 20, 2023, 12:19:21 PM10/20/23
to django-...@googlegroups.com
#34919: Missing scope attribute in admin doc table headers
-------------------------------------+-------------------------------------
Reporter: Louis-Cloud | Owner: Louis-

| Cloud
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.2
Severity: Normal | Resolution:
Keywords: table, scope, | Triage Stage:
screenreader | Unreviewed
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------
Description changed by Louis-Cloud:

Old description:

> There are a number of tables throughout [https://static-django-
> demo.netlify.app/django_admin_tests/v5.0/english/admin/docs/models/contenttypes.contenttype/
> Django's admin docs] that are lacking the `scope` attribute. Having this
> attribute in the table headers makes table navigation easier for a screen
> reader user.
>
> === Relevant Files:
> * django/contrib/admindocs/templates/admin_doc/model_detail.html
> * django/contrib/admindocs/templates/admin_doc/model_index.html
>
> === Acceptance Criteria
> * All table column header (th) needs to have a `scope = "col"` attribute.
> * All table row headers (first column th) should have a `scope = "row"`
> attribute
>
> === For more information
> * [https://www.w3.org/WAI/tutorials/tables/two-headers/#table-with-

> header-cells-in-the-top-row-and-first-column W3 - Tables with Two


> Headers]:
> * [https://developer.mozilla.org/en-
> US/docs/Learn/HTML/Tables/Advanced#using_column_and_row_headers MDN Web
> Docs - Using column and row headers]

New description:

There are a number of tables throughout [https://static-django-
demo.netlify.app/django_admin_tests/v5.0/english/admin/docs/models/contenttypes.contenttype/
Django's admin docs] that are lacking the `scope` attribute. Having this
attribute in the table headers makes table navigation easier for a screen
reader user.

=== Relevant Files:
* django/contrib/admindocs/templates/admin_doc/model_detail.html
* django/contrib/admindocs/templates/admin_doc/model_index.html

=== Acceptance Criteria
* All table column header (th) needs to have a `scope = "col"` attribute.
* All table row headers (first column th) should have a `scope = "row"`
attribute

=== For more information
* [https://www.w3.org/WAI/tutorials/tables/two-headers/#table-with-header-
cells-in-the-top-row-and-first-column W3 - Tables with Two Headers]
* [https://developer.mozilla.org/en-
US/docs/Learn/HTML/Tables/Advanced#using_column_and_row_headers MDN Web
Docs - Using column and row headers]

--

--
Ticket URL: <https://code.djangoproject.com/ticket/34919#comment:1>

Django

unread,
Oct 20, 2023, 12:20:05 PM10/20/23
to django-...@googlegroups.com

Old description:

New description:

=== For more information


* [https://www.w3.org/WAI/tutorials/tables/two-headers/#table-with-header-
cells-in-the-top-row-and-first-column W3 - Tables with Two Headers]
* [https://developer.mozilla.org/en-
US/docs/Learn/HTML/Tables/Advanced#using_column_and_row_headers MDN Web
Docs - Using column and row headers]

--

--
Ticket URL: <https://code.djangoproject.com/ticket/34919#comment:2>

Django

unread,
Oct 20, 2023, 12:25:03 PM10/20/23
to django-...@googlegroups.com
#34919: Missing scope attribute in admin doc table headers
-------------------------------------+-------------------------------------
Reporter: Louis-Cloud | Owner: Louis-
| Cloud
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.2
Severity: Normal | Resolution:
Keywords: table, scope, | Triage Stage:
screenreader | Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------
Description changed by Louis-Cloud:

Old description:

> There are a number of tables throughout [https://static-django-
> demo.netlify.app/django_admin_tests/v5.0/english/admin/docs/models/contenttypes.contenttype/
> Django's admin docs] that are lacking the `scope` attribute. Having this
> attribute in the table headers makes table navigation easier for a screen
> reader user.
>

> === Acceptance Criteria
> * All table column header (th) needs to have a `scope = "col"` attribute.
> * All table row headers (first column th) should have a `scope = "row"`
> attribute
>
> === Relevant Files:
> * django/contrib/admindocs/templates/admin_doc/model_detail.html
> * django/contrib/admindocs/templates/admin_doc/model_index.html
>
> === For more information
> * [https://www.w3.org/WAI/tutorials/tables/two-headers/#table-with-
> header-cells-in-the-top-row-and-first-column W3 - Tables with Two
> Headers]
> * [https://developer.mozilla.org/en-
> US/docs/Learn/HTML/Tables/Advanced#using_column_and_row_headers MDN Web
> Docs - Using column and row headers]

New description:

There are a number of tables throughout [https://static-django-
demo.netlify.app/django_admin_tests/v5.0/english/admin/docs/models/contenttypes.contenttype/
Django's admin docs] that are lacking the `scope` attribute. Having this
attribute in the table headers makes table navigation easier for a screen
reader user.

=== Acceptance Criteria
* All table column header (th) needs to have a `scope = "col"` attribute.

* All table row headers (first column) should have a `scope = "row"`
attribute

=== Relevant Files:
* django/contrib/admindocs/templates/admin_doc/model_detail.html
* django/contrib/admindocs/templates/admin_doc/model_index.html

=== For more information
* [https://www.w3.org/WAI/tutorials/tables/two-headers/#table-with-header-
cells-in-the-top-row-and-first-column W3 - Tables with Two Headers]
* [https://developer.mozilla.org/en-
US/docs/Learn/HTML/Tables/Advanced#using_column_and_row_headers MDN Web
Docs - Using column and row headers]

--

--
Ticket URL: <https://code.djangoproject.com/ticket/34919#comment:3>

Django

unread,
Oct 22, 2023, 11:53:00 PM10/22/23
to django-...@googlegroups.com
#34919: Missing scope attribute in admin doc table headers
-------------------------------------+-------------------------------------
Reporter: Louis-Cloud | Owner: Louis-
| Cloud
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.2
Severity: Normal | Resolution:
Keywords: table, scope, | Triage Stage:
screenreader, accessibility | Unreviewed

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* cc: Thibaud Colas, Sarah Abderemane, Tom Carrick (added)
* keywords: table, scope, screenreader => table, scope, screenreader,
accessibility


--
Ticket URL: <https://code.djangoproject.com/ticket/34919#comment:4>

Reply all
Reply to author
Forward
0 new messages