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

Add an ‘inside header’ inserter #167

Merged
merged 1 commit into from
Oct 6, 2016
Merged

Conversation

36degrees
Copy link
Contributor

This processor allows an application to inject a block of HTML after the logo by including a .inside-header element in their application’s output.

This is effectively like adding content to the inside_header yield block in govuk_template.

The service manual is using this to insert a title into the header.

@36degrees 36degrees force-pushed the insert-header-inserter branch from 9547137 to 87a86d8 Compare October 5, 2016 14:58
@36degrees 36degrees changed the title Add an ‘inside header’ inserter [DO NOT MERGE] Add an ‘inside header’ inserter Oct 5, 2016
This processor allows an application to inject a block of HTML after the logo by including a `.inside-header` element in their application’s output.

This is effectively like adding content to the inside_header yield block in govuk_template.

The service manual is using this to insert a title into the header.
@36degrees 36degrees force-pushed the insert-header-inserter branch from 87a86d8 to f571595 Compare October 6, 2016 14:13
@NickColley
Copy link
Contributor

I've talked to @36degrees offline, we think that long term being able to consider the header and footer as components that can be overridden per rendering application will give us the flexibility to make this kind of change but for now this makes sense and can easily be removed in the future.. (famous last words)

@NickColley NickColley merged commit fa0d8d5 into master Oct 6, 2016
@NickColley NickColley deleted the insert-header-inserter branch October 6, 2016 14:32
tijmenb added a commit to alphagov/finder-frontend that referenced this pull request Feb 27, 2017
Full changelog:

# 10.1.3

* Fix memory leak in components

# 10.1.2

* Bugfix for request URI's encoded as ASCII

# 10.1.1

* Bugfix for caching behaviour

# 10.1.0

* Use `Rails.cache` as the cache for templates, locales and components.
You can
remove `config.slimmer.use_cache` for your application, as you can no
longer
opt-out of caching.
* Add a `User-Agent` header to all outgoing API requests

# 10.0.0

* Removes the need_id meta tag, which is no longer used.
* Removes the functionality for breadcrumbs, related links and
artefact-powered
metatags.
* Drop support for old Rails & Ruby versions. This gem now supports
Rails 4.2 and 5.X
on Ruby 2.1 and 2.2.
* Renames `Slimmer::SharedTemplates` to `Slimmer::GovukComponents`

# 9.6.0

* Adds an 'inside header inserter' processor which allows an
application to
inject a block of HTML after the logo by including a .inside-header
element
in their application’s output.
(PR #167 alphagov/slimmer#167)

* Remove `MetaViewportRemover` processor as it is no longer used.
(PR #166 alphagov/slimmer#166)

# 9.5.0

* Adds a Cucumber helper that makes it easy for host applications to
configure Slimmer correctly under test.

(PR #162 alphagov/slimmer#162)

# 9.4.0

* Adds an RSpec helper that makes it easy for host applications to
configure Slimmer correctly under test.

Fixes `stub_shared_component_locales` helper to correctly stub HTTP
requests to fetch locale information when rendering its templates.

(PR #159 alphagov/slimmer#159)

# 9.3.2

* Bugfix: Over time, the I18n backend would be chained in each request,
causing the stack to grow too large and use too much memory

(PR #157 alphagov/slimmer#157)

# 9.3.1

* Allows frontend apps to stub component locales for example

```ruby
class ActiveSupport::TestCase
include Slimmer::TestHelpers::SharedTemplates

def setup
stub_shared_component_locales
end
end
```

(PR #155 alphagov/slimmer#155)

# 9.3.0

* Integrates translations from GOVUK Components to be used in
applications

When including `Slimmer::SharedComponents`, the I18nBackend will be
chained to `Slimmer::I18nBackend` allowing translations in `static` to
work in the frontend applications

(PR #152 alphagov/slimmer#152)

# 9.2.1

* Replaces deprecated `before_filter` calls in shared templates.

# 9.2.0

* Raise a custom `CouldNotRetrieveTemplate` exception when a connection
to the assets server can't be made because of an SSL problem (PR #143).

# 9.1.0

* Allow applications to request components using full or partial
component
paths, eg "name", "name.raw" and "name.raw.html.erb". This allows
components to be nested within other components.
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