Skip to content
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

Results page: update Registration District Reference column heading #606

Closed
nurifreembd opened this issue Nov 26, 2024 · 16 comments
Closed
Assignees
Labels
MVP Minimum Viable Product (Requirement for launch) ready for deployment ready to be rolled out to live. Results page

Comments

@nurifreembd
Copy link
Collaborator

nurifreembd commented Nov 26, 2024

suggest to update Registration District Reference column heading to read 'Registration District/Reference' or 'Registration District & Reference' . Not very obvious and user friendly.

@nurifreembd nurifreembd changed the title Results page: show search from and to quarters in 'view search criteria' details Results page: update Registration District Reference column heading Nov 26, 2024
@KristinaGadzhieva
Copy link
Collaborator

@richardofsussex @DeniseColbert do you agree? I think option with slash will be consistent with the 'Age at Death/Date of Birth'.

@KristinaGadzhieva KristinaGadzhieva added the MVP Minimum Viable Product (Requirement for launch) label Dec 3, 2024
@richardofsussex
Copy link

Again, if you look closely you will see that the two things are different. Registration District is one piece of information, and the results are sortable by District, indicated by the underline. Reference is just that - the reference - and it isn't sortable. Putting a slash between them implies that they are alternative names/concepts, which is not the case.

@nurifreembd
Copy link
Collaborator Author

would '&' work?

@KristinaGadzhieva
Copy link
Collaborator

I understand, but I agree it could be confusing for users. Can the word 'Reference' be omitted? (Just 'Registration District') Or maybe 'Registration District Information' ?

@KristinaGadzhieva
Copy link
Collaborator

@AlOneill would the option 'Registration District & Reference' be accessible for all users?

@AlOneill
Copy link
Collaborator

I would like to see a Results page so that I can understand what exactly is being discussed but neither beta nor test3 would allow me to search. I could not get past the initial login on beta; a search on t3 results in "something went wrong".

I'll try again tomorrow.

@KristinaGadzhieva
Copy link
Collaborator

@AlOneill
Copy link
Collaborator

@KristinaGadzhieva
Thank you. I think that most sighted users would understand that the RD was the bit in red and that the reference was the bit in black, even if it took a second or two. However, it is not clear that the column headers are sortable — a pair of triangles is the norm. Although the header maybe looks like a link, the situation could be clearer.

I will test later with a screen reader.

@KristinaGadzhieva
Copy link
Collaborator

@AlOneill Thank you. So maybe one of the headings would be more clear: ''Registration District & Reference' or ''Registration District/Reference' or ''Registration District Information' ?
As for sorting options, we are considering adding a note for users clarifying that the column headers are sortable.

@AlOneill
Copy link
Collaborator

@KristinaGadzhieva @richardofsussex
Although the screen reader announces the RD and then 'Reference', it would be better to have an '& Reference' so that it is clearer that it is still the same column rathe than two columns, 'RD' and 'reference'.

The links in the headers are announced as links — there is no mention of sorting. I think sorting requires treatment as a button because it does something rather than going somewhere. See https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-sort for some help with this.

You should be aware that the abbreviation 'Apr' is announced as 'A P R' whereas 'Jun' is read as 'June'.

The hyphens in the otherwise empty cells are announced as 'dash' (which is somewhat depressing, for me at least!).

If you are going to add any kind of note about the table contents, please make sure it has a descriptive heading or a screen reader user might never find it.

@KristinaGadzhieva
Copy link
Collaborator

Thank you, Alison.
@AlOneill @richardofsussex So, regarding the 'Registration District' heading, if we go with clickable headings for sorting, would it work to make the entire 'Registration District & Reference' clickable, rather than just 'Registration District'? This would make it easier for users to comprehend.

As for sorting, please have a look at another story #609.

@richardofsussex
Copy link

I would be happy if we were to make it into a single, clickable, heading.

BTW, having dashes in otherwise empty cells does mean that those columns can be sorted. If they are completely empty, trying to sort that column causes a meltdown.

@KristinaGadzhieva KristinaGadzhieva added the ready for deployment ready to be rolled out to live. label Dec 19, 2024
@KristinaGadzhieva
Copy link
Collaborator

KristinaGadzhieva commented Dec 19, 2024

Thank you. Can be deployed @Vino-S .

@KristinaGadzhieva
Copy link
Collaborator

KristinaGadzhieva commented Jan 14, 2025

No changes on Beta image.png

@richardofsussex
Copy link

Implemented and passed to @Vino-S for deployment.

@Vino-S
Copy link
Collaborator

Vino-S commented Jan 21, 2025

Deployed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MVP Minimum Viable Product (Requirement for launch) ready for deployment ready to be rolled out to live. Results page
Projects
None yet
Development

No branches or pull requests

5 participants