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

Fix default text for global cookie message #261

Merged
merged 3 commits into from
Jun 20, 2016

Conversation

gemmaleigh
Copy link
Contributor

@gemmaleigh gemmaleigh commented Jun 18, 2016

This pull request is another step towards getting the GOV.UK elements app to work in the same way as the prototype kit. 🎆

Add a config file, ensure that server.js uses the config file, then define a cookieText variable and pass the cookieText variable to all views.

Also add the missing blocks for the cookie message in both layout files.

Screenshot before (empty blue cookie banner at the top of the screen):

gov uk elements - before

Screenshot after (cookie banner with text at the top of the screen):

gov uk elements - after

- Include this in /layout.html and /layout-example.html as all
templates use these files as a base template
Use this to define a variable for the cookieText
@gemmaleigh
Copy link
Contributor Author

cc. @joelanman

@joelanman
Copy link
Contributor

looks good!

@joelanman joelanman merged commit 611eed1 into master Jun 20, 2016
@gemmaleigh gemmaleigh deleted the fix-default-text-for-global-cookie-message branch September 12, 2016 10:06
gemmaleigh pushed a commit that referenced this pull request Jun 8, 2017
- Increase skiplink colour contrast PR #263 - alphagov/govuk_template#263
- Fix Scala compile issues for Play template PR #261 - alphagov/govuk_template#261
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