Edit attachment names inline in speed tagging.
Make attachment renaming JS only.
Functional tests for inline attachment renaming.
Got feedback from the user that & are out of style
Add truncate option to link_to_attachment helper.
Style improvements for inline attachment editing.
Use new button name in test.
Try and make detailed guides look more like others

So that they have the same metadata stuff so we can add more metadata.
Add parent category document metadata

To replace functionality previously provided by the breadcrumb
Change "Detailed guidance" to "Guidance"

User and Editor feedback was that the term detailed guidance was confusing.
Fix to not barf when there is no primary category

Only detailed guides have primary categories. Pass these in explicitly and display if present.
Remove business link test
Remove test for breadcrumb inclusion
New class for cleaning document filter parameters

We have been getting random requests where filter parameters are being
specified as incorrect types. For example:

  {
    "action" => "index",
    "controller" => "publications",
    "departments" => "[\"department-of-energy-climate-change\"]",
    "locale" => "en",
    "page" => {
      "&#36;acunetix" => "1"
    },
    "publication_type" => "transparency-data"
  }

This new class returns a clean set of parameters that can be used by
the various search backends to filter documents.

It incorporates the correcting of array parameters that become
malformed when passed through the Facebook referrer from here:
https://github.com/alphagov/whitehall/blob/858d4d789cbdd0e9bd45e0cc47f70c0e54f7d331/app/controllers/application_controller.rb#L75
Remove unused hide extra logos
Remove unused title partials

Removed partials and styles which were only there for the removed
markup
Update placement of BL and EU logos

For the new heading structure the headings need to be in new places to
not look broken.
Wire in Whitehall::DocumentFilter::CleanedParams

Note that we no longer need to set a default page number as this is
now done in Whitehall::DocumentFilter::Filter.

This also removes the controller tests that were testing whether
unpermitted parameter exceptions are being raised as we no longer
raise them in production anyway.
Cleanup unused code.

All the behaviour of these methods is now encapsulated in
Whitehall::DocumentFilter::CleanedParams and in
`ConsultationController`s case, calling `clean_search_filter_params`
is redundant anyway. Note that there will be a slight change in
behaviour: previously, any scaler params that were incorrectly passed
in as an array (e.g. page: []) are now dropped instead of taking the
first value of the array as the actual value. We don't generate urls
like so it doesn't seem reasonable that we should guess as to what
they should be. As long as we don't blow up when they are passed in,
all should be good.
DetailedGudes are no longer unique