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

FS-5006: improvements for the all questions pages #272

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nuwan-samarasinghe
Copy link
Contributor

@nuwan-samarasinghe nuwan-samarasinghe commented Feb 4, 2025

Ticket: https://mhclgdigital.atlassian.net/browse/FS-5005

Change description

In this PR we have mostly focused on following changes

  1. remove numbered references and give textual representation
  2. remove hard number system
  3. simplify html generation for more readability
  • Unit tests and other appropriate tests added or updated
  • README and other documentation has been updated / added (if needed)
  • Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

Templates's Questions View
image

Application build question view
image

All questions page view
image

Copy link
Contributor

@MarcUsher MarcUsher left a comment

Choose a reason for hiding this comment

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

Nice work Nuwan! Based on our discussion yesterday I know we said we're only doing minimal tweaks to these views for MVP as it's not crucial but just a few points below as design-wise we've still got a few discrepancies between each of the places that renders viewing questions (the first two I think are using the same method to render so changes to one should go to both). It might be nice to at least make these all consistent:

Build application > View all application questions

image

  • missing the navigation (see designs)
  • the xl header for an h1 seems a bit overkill (but I guess we need it to differentiate with the L, M and S further down the page)?

Build application > View questions (for single template)

image

  • missing the navigation (see designs)
  • the xl header for an h1 seems a bit overkill (but I guess we need it to differentiate with the L, M and S further down the page)?
  • the field types seem to be showing on this view but no others
  • this view is essentially the same as the one below (letting the user view questions of a single template) but the design of both pages is quite different eg. here we have a table of contents when it's just one template, which doesn't seem necessary? Also header styles, the [Preview], potentially missing the 'This template contains the following questions."

Template details > view quesitons

image

  • this one most closely matches Mika's intended design so is maybe the pattern to follow for the others (except for the 'view all application questions' still including the table of contents which everyone agreed was a helpful thing?)
  • however the section breaks are full-width where the others are not (and also not on the designs)

I think the first govuk-section-break above each section could also maybe be dropped, as I think that's causing the double section break we see on the 'View all application questions' page?

@nuwan-samarasinghe
Copy link
Contributor Author

As per the discussion had with @MarcUsher we are going to do the changes as much as we can

Copy link

sonarqubecloud bot commented Feb 7, 2025

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

Successfully merging this pull request may close these issues.

2 participants