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

Use new grid layout placeholders for the cookie and header bar #129

Merged
merged 1 commit into from
Jan 22, 2015
Merged

Use new grid layout placeholders for the cookie and header bar #129

merged 1 commit into from
Jan 22, 2015

Conversation

gemmaleigh
Copy link
Contributor

inner-block and outer-block have been deprecated, remove these
and replace with the new grid layout placeholders.

This affects the global header bar (the blue line underneath the
header) and the global cookie message (shown at first visit, or when js
is disabled).

Smaller screens:
govuk-template-mobile

Tablet:
govuk-template-tablet

Desktop:
govuk-template-desktop

`.inner-block` and `.outer-block` have been deprecated, remove these
and replace with the new grid layout placeholders.

This affects the global header bar (the blue line underneath the
header) and the global cookie message (shown at first visit, or when js
is disabled).
@gemmaleigh gemmaleigh changed the title Use new grid layout mixins for the cookie and header bar Use new grid layout placeholders for the cookie and header bar Jan 20, 2015
@edds
Copy link
Contributor

edds commented Jan 22, 2015

This makes the code much nicer. Thanks for this @gemmaleigh!

edds added a commit that referenced this pull request Jan 22, 2015
…from-header

Use new grid layout placeholders for the cookie and header bar
@edds edds merged commit ac74595 into alphagov:master Jan 22, 2015
@dsingleton
Copy link
Contributor

This is 👍, long live the new grid.

Is there a risk of anyone hanging styles/behaviour off the removed selectors? Might be worth mentioning in release notes when we do.

@gemmaleigh
Copy link
Contributor Author

The only thing I can think of - %site-width-container styles are applied using the paragraph selector #global-cookie-message p so if the cookie message was to have any other content, e.g. if <%= yield :cookie_message %> was used and the content inside it wasn't a paragraph, that content wouldn't have the correct alignment.

I didn't want to start adding extra inner divs, but it might be to sensible to also wrap <%= yield :cookie_message %> in a <p> tag, but only if we expect this content to be a single paragraph.

gemmaleigh pushed a commit to alphagov/govuk_elements that referenced this pull request Jun 4, 2015
# 0.13.0

- Add `homepage_url` for overriding the href for the header logo link
("https://www.gov.uk" by default)

- Add `global_header_text` for overriding the header logo text
("GOV.UK" by default)

- Removed deprecated grid layout mixins:
alphagov/govuk_template#129
gemmaleigh pushed a commit to alphagov/govuk-prototype-kit that referenced this pull request Jun 4, 2015
# 0.13.0

- Add `homepage_url` for overriding the href for the header logo link
("https://www.gov.uk" by default)

- Add `global_header_text` for overriding the header logo text
("GOV.UK" by default)

- Removed deprecated grid layout mixins:
alphagov/govuk_template#129
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.

None yet

3 participants