-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
@richardofsussex @DeniseColbert do you agree? I think option with slash will be consistent with the 'Age at Death/Date of Birth'. |
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. |
would '&' work? |
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' ? |
@AlOneill would the option 'Registration District & Reference' be accessible for all users? |
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. |
@AlOneill Please try open the link: https://test3.freebmd.org.uk/search_queries/6758687cbd30f272acbb4400?locale=en |
@KristinaGadzhieva I will test later with a screen reader. |
@AlOneill Thank you. So maybe one of the headings would be more clear: ''Registration District & Reference' or ''Registration District/Reference' or ''Registration District Information' ? |
@KristinaGadzhieva @richardofsussex 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. |
Thank you, Alison. As for sorting, please have a look at another story #609. |
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. |
Thank you. Can be deployed @Vino-S . |
Implemented and passed to @Vino-S for deployment. |
Deployed |
suggest to update Registration District Reference column heading to read 'Registration District/Reference' or 'Registration District & Reference' . Not very obvious and user friendly.
The text was updated successfully, but these errors were encountered: