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

Change title of and introduction to Records 'page' to reflect its actual contents and purpose #611

Open
richardofsussex opened this issue Jan 15, 2025 · 13 comments

Comments

@richardofsussex
Copy link

As discussed, 'Records' is not a helpful title for this area of the site (for a start, it's more than a 'page'), so we need a better title. Similarly, the initial text and choices offered suggest that it is solely about Districts and counties. It isn't.

@richardofsussex
Copy link
Author

@nurifreembd suggests (for page title): 'BMD Records' or 'live records', 'explore records', 'review records', 'check records', 'records report'.

@richardofsussex
Copy link
Author

This is what I have come up with so far:

image

@nurifreembd
Copy link
Collaborator

nurifreembd commented Jan 15, 2025 via email

@richardofsussex richardofsussex self-assigned this Jan 16, 2025
@richardofsussex
Copy link
Author

This is now live on beta, and we have all the page maps working. (But don't do what I just did and use the default values for the date-based page map - there are no records for Q1 1837, as we all know!)

@AlOneill
Copy link
Collaborator

I am a little overwhelmed by the link. However, I did see a possible problem:

Xnip2025-01-16_12-20-54.jpeg

@richardofsussex
Copy link
Author

Sorry, I'm getting the non-image content-type error so can't see your screen shot. Can you paste it into the Freebmd2 Slack channel please?

@AlOneill
Copy link
Collaborator

Don't have access to Slack. This sentence seems to be missing a word (Districts that have been?):

"Select a county for a list of links to Districts have been located within that county."

@AlOneill
Copy link
Collaborator

Mind you, the sentence is passive voice which is tricky for some people.

@richardofsussex
Copy link
Author

"... Districts within that county" should be sufficient.

@AlOneill
Copy link
Collaborator

Also, the tabbing order is a little odd: one has to tab to the link in "Database coverage" before being taken to the "On this page" menu. I suggest that "On this page" should be before that first section.

The introductory paragraphs need to have their width limited — too many lines that are too wide to read comfortably (on a wide screen).

@richardofsussex
Copy link
Author

OK, I have taken these comments on board, and updated beta accordingly.

@AlOneill
Copy link
Collaborator

@richardofsussex
Thank you. That is grand.

For the various form controls, each one should have a programmatically linked label: you might choose to hide the label from sighted users, but it must be there for screen reader users.

There does not appear to be an enclosing div element, say, with margins to keep some of the form controls away from the very edge of the page—this is apparent when viewing in a less-than-maximum-width window, where some of the form controls are hard up against the very (left) edge of the page.

Now, this could be regarded as a matter of opinion: I find the lack of consistent horizontal alignment unsettling; also the (apparently unnecessary) width of some of the form controls.

@KristinaGadzhieva
Copy link
Collaborator

@nurifreembd To include in the testing scenarious

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants