Inlining review and fact check activity fields

www.agileplannerapp.com/boards/173808/cards/5629

... with edition form. these were left out earlier,
causing review activities to not save edition changes.

part of a larger change:

https://github.com/alphagov/publisher/pull/251
Removing unnecessary template

this template is not being used anywhere. instead
shared/_review gets rendered. looks like it was
left behind when common templates were moved to
the shared folder.
Fix tests failing due to overlapping elements

Poltergeist is seeing '.help-block' overlapping
the 'Save' button, hence can't click it without
a `.trigger('click')` on that element.

This needs to be followed by an assertion that
waits for the edition to get saved, or else
capybara jumps to the next statement.
Renaming constants to convey intent clearly

.. and moving locals logic into a helper,
so that we don't have to pass locals around
to detect the diff view.
Merge pull request #253 from alphagov/save-and-transition-in-on-go-feedback

Inlining review and fact check activity fields