Skip to content

Fixed #36511 -- Ensured filters came before table in keyboard navigation in admin changelist. #19659

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 30, 2025

Conversation

Antoliny0919
Copy link
Contributor

@Antoliny0919 Antoliny0919 commented Jul 20, 2025

Trac ticket number

ticket-36511

Branch description

With the original HTML structure, accessing the sidebar filter on the changelist page required passing through the entire table element.

While keeping the existing layout, the filter was repositioned to the top within the changelist form container so that accessing the filter element no longer requires passing through the table.

Now, the changelist form container element is navigated in the following order....

sidebar filter -> search -> date hierarchy -> table

Checklist

  • This PR targets the main branch.
  • The commit message is written in past tense, mentions the ticket number, and ends with a period.
  • I have checked the "Has patch" ticket flag in the Trac system.
  • I have added or updated relevant tests.
  • I have added or updated relevant docs, including release notes if applicable.
  • I have attached screenshots in both light and dark modes for any UI changes.

@sarahboyce sarahboyce added selenium Apply to have Selenium tests run on a PR screenshots 🖼️ labels Jul 21, 2025
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works well for me

I think this highlights that our mobile design is not great for filters (should perhaps be a section that you can open from the side or a collapsed section at the top) but I think this can be agreed and worked on in a separate ticket 👍

Perhaps taking inspiration from tests.admin_views.test_skip_link_to_content.SeleniumTests.test_use_skip_link_to_content, can you write a test which shows that once we are at the main content of the changelist page, tabbing and confirming what gets focused

@Antoliny0919
Copy link
Contributor Author

Tested and works well for me

I think this highlights that our mobile design is not great for filters (should perhaps be a section that you can open from the side or a collapsed section at the top) but I think this can be agreed and worked on in a separate ticket 👍

Perhaps taking inspiration from tests.admin_views.test_skip_link_to_content.SeleniumTests.test_use_skip_link_to_content, can you write a test which shows that once we are at the main content of the changelist page, tabbing and confirming what gets focused

I'm also unsure about where filters should be placed in the mobile design, and I think this is something that will need to be discussed with the accessibility team.

Currently, there are two sidebar like features (model selection and filters), but in the mobile design, the model selection sidebar isn’t supported, and as you mentioned the filter usability isn’t great either.

I’ll create an ticket for this later :)

@Antoliny0919
Copy link
Contributor Author

New Accessibility Improvement Project 🎉🎉🎉

@sarahboyce sarahboyce changed the title Fixed #36511 -- Fixed keyboard focus skipping table when accessing filter on admin changelist page. Fixed #36511 -- Ensured filters came before table in keyboard navigation in admin changelist. Jul 30, 2025
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you ⭐

@sarahboyce sarahboyce merged commit 6ea3319 into django:main Jul 30, 2025
33 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
screenshots 🖼️ selenium Apply to have Selenium tests run on a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy