[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>

[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 Shrikant Sudam Giri):

Hi everyone 👋
I’m interested in contributing to this issue as part of learning Django’s
admin internals.
Before I start working on a patch, could you please confirm whether the
accessibility team has reached a final decision on this?
If it’s still open for discussion, I’d be happy to explore options that
balance clarity for screen readers with minimal verbosity.

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

[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 VIZZARD-X):

Hello,

I'm interested in working on this issue and would like to move it forward.
Before preparing a patch, could we confirm the intended direction from the
accessibility team?

Specifically:
Should the pagination landmark always include the model name (e.g.,
"Pagination Release")?
Or should the model name be omitted when there's only one pagination
component on the page?
Once there is agreement on the desired behaviour, I’ll be happy to
implement the change and prepare a PR aligned with the decision.
--
Ticket URL: <https://code.djangoproject.com/ticket/36510#comment:5>