-
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
FS-5006: improvements for the all questions pages #272
base: main
Are you sure you want to change the base?
Conversation
8e2cb66
to
33e35cf
Compare
33e35cf
to
3410533
Compare
There was a problem hiding this 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
- 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)
- 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
- 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?
5ce3a0d
to
84cfad9
Compare
84cfad9
to
9cea4e0
Compare
As per the discussion had with @MarcUsher we are going to do the changes as much as we can |
…plify the process fix all questions previews fix all questions previews
9cea4e0
to
0a7030a
Compare
Quality Gate passedIssues Measures |
Ticket: https://mhclgdigital.atlassian.net/browse/FS-5005
Change description
In this PR we have mostly focused on following changes
Templates's Questions View
Application build question view
All questions page view