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

Show the current release version on the index page #235

Merged
merged 2 commits into from
Aug 17, 2016

Conversation

gemmaleigh
Copy link
Contributor

We already log this version to the console when the app is run and it is included in a comment
at the bottom of each page.

Add the release version above the title on the index page.
This will also make it easy to see that the kit when deployed to Heroku is up to date.

This fixes #121.

We already log this version to the console and include it in a comment
at the bottom of each page. Add the release version above the title on
the index page.

This fixes #121.
@gemmaleigh
Copy link
Contributor Author

cc. @timpaul, @edwardhorsford, @joelanman.

@joelanman
Copy link
Contributor

thanks Gemma! I think it looks/reads a bit better underneath - what do you reckon?

image

@joelanman
Copy link
Contributor

also reminds me it'd be good to look at #163

@edwardhorsford
Copy link
Contributor

I think I prefer it above.

@joelanman
Copy link
Contributor

a screen reader would read out 'v3.0.0 The GOV.UK prototype kit', which is a bit odd

@edwardhorsford
Copy link
Contributor

I think that's a general issue we have with this design - the accessibility team is about to work on improving the markup for these 'subheads' or whatever we're calling them so they read better for screen readers.

@timpaul
Copy link
Contributor

timpaul commented Aug 17, 2016

This is a version number not a section or category subheading, so below makes more sense.

@timpaul timpaul merged commit bcc269f into master Aug 17, 2016
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.

Add version to index page
4 participants