[Django] #36510: Header text for pagination contains title of table while it's the only pagination on the page.

[Django]

#36510: Header text for pagination contains title of table while it's the only
pagination on the page.
-------------------------------------+-------------------------------------
Reporter: laurenherregodts | Type:
| Cleanup/optimization
Status: new | Component:
| contrib.admin
Version: 5.2 | Severity: Normal
Keywords: accessibility, | Triage Stage:
pagination | Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------
On the page [http://localhost:8000/en/admin/demo/release/ Release] in the
django admin demo project, the pagination underneath the table has
`Pagination release` as `h2` header text.

This has been identified while using the Landmark check of AXE. No
accessibility issue was flagged, but just wondering if the `release` in
the header text is needed.
I believe `release` can be removed from the header text since there is
only one pagination present on the page.
--
Ticket URL: <https://code.djangoproject.com/ticket/36510>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

[Django]

#36510: Header text for pagination contains title of table while it's the only
pagination on the page.
-------------------------------------+-------------------------------------

Reporter: laurenherregodts | Owner: (none)
Type: | Status: new
Cleanup/optimization |
Component: contrib.admin | Version: 5.2
Severity: Normal | Resolution:
Keywords: accessibility, | Triage Stage: Accepted
pagination |

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------

Changes (by Thibaud Colas):

* cc: Thibaud Colas (added)
* stage: Unreviewed => Accepted

Comment:

Thank you! This looks like a recent change from #36366, in PR
[https://github.com/django/django/pull/19448 Fixed #36366 -- Improved
accessibility of pagination in the admin. #19448].

I think this label tweak would work much better for its intended audience,
but this will probably need accessibility team discussion on how best to
achieve the desired results. The issue I see is that for _landmark names_
verboseness isn’t desirable. If we wanted the added context we would be
better off with `aria-describedby`.
--
Ticket URL: <https://code.djangoproject.com/ticket/36510#comment:1>

[Django]

#36510: Header text for pagination contains title of table while it's the only
pagination on the page.
-------------------------------------+-------------------------------------
Reporter: laurenherregodts | Owner: (none)
Type: | Status: new
Cleanup/optimization |
Component: contrib.admin | Version: 5.2
Severity: Normal | Resolution:
Keywords: accessibility, | Triage Stage: Accepted
pagination |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------

Changes (by Antoliny):

* cc: Antoliny (added)

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

[Django]

#36510: Header text for pagination contains title of table while it's the only
pagination on the page.
-------------------------------------+-------------------------------------
Reporter: laurenherregodts | Owner: (none)
Type: | Status: new
Cleanup/optimization |
Component: contrib.admin | Version: 5.2
Severity: Normal | Resolution:
Keywords: accessibility, | Triage Stage: Accepted
pagination |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------

Comment (by Antoliny):

First of all, thank you for submitting the ticket :)
As you mentioned, if multiple paginations are applied, using "pagination +
model_name" is naturally necessary.
However, even if only one pagination is used on the admin `ChangeList`
page, I do not think that "pagination + model_name" becomes excessive
information that interferes with screen reader users.
On the contrary, I believe it is necessary information that provides
clarity.
Since a web page can usually contain many models and therefore many
different `ChangeList` pages for those models, "pagination + model_name"
can help screen reader users perceive at once which model's pagination
this is.
So, I feel there is no need to remove the `model_name` deliberately.
--
Ticket URL: <https://code.djangoproject.com/ticket/36510#comment:3>