Rename 'clickable' to 'interactive' per leaflet 1.x change

Update to iD v2.1.3

Revert to nokogumbo 1.4.9

https://github.com/rubys/nokogumbo/issues/50

Update to rails 4.2.8

Blacklist VWorld imagery

Fixes #1464

User factories for browse, oauth_clients and site controller tests.

Merge remote-tracking branch 'openstreetmap/pull/1436'

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

User factory associations for all factories.

Remove a few unnecessary attributes from moderator users

Make all factory-generated users 'normal'

The use of create(:normal_user) makes the tests needlessly harder to
understand and reason about, particularly when we start using
associations in the factories.

Fix rubocop warning

Merge remote-tracking branch 'openstreetmap/pull/1458'

Index oauth_tokens and client_applications by user_id

Merge remote-tracking branch 'openstreetmap/pull/1457'

Convert notes_controller tests to user factories.

Revert changes where the user is a query parameter

Refactor api_controller tests to avoid users fixtures.

Refactor amf_controller tests to avoid users fixtures

In all honesty I wouldn't be surprised if there are some semi-hidden
dependencies on the user fixtures here, given the number of magic numbers
all over the place. But this commit takes care of the explicit
dependencies for now.

Pass the user object, rather than the id, for sessions.

This simplifies the code slightly.

Merge remote-tracking branch 'openstreetmap/pull/1449'

Enable context menu on mouse down with no shift pressed

Merge remote-tracking branch 'openstreetmap/pull/1451'

Add a couple of notes to the vagrant instructions

Use created at for RSS format with note#show

This matches other formats and is also what the tests expect.

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Refactor diary entry tests to use User factory

Use simple string matches for user_blocks_helper tests

Testing against the translations is not something we do anywhere
else is and is dangerously close to testing the helper against
itself which is a bit pointless.

Use the factory to create user blocks

Merge remote-tracking branch 'openstreetmap/pull/1445'

Make rupocop happy and correct placement of include

Improve wording and add test

Fix shift key handling in contextmenu

Small improvement to block status display

Merge branch 'contextmenu'

Fix rubocop warnings

Merge remote-tracking branch 'openstreetmap/pull/1443'

Make test for a "zero hour" block more robust

Apparently some fool things the colon is part of the protocol

Actually use user_block

Add functionality to return a specific message for zero hour blocks

Preserve protocol in standard links

Preserve protocol in short links

Initialise locale before looking up user blocked error

Implement the OWG credit policy for hosting partners.

See https://operations.osmfoundation.org/policies/hosting/

Merge remote-tracking branch 'openstreetmap/pull/1439'

Add line segment to each steps of GraphHopper routing

Fix latLng from GraphHopper engine

latLng is an array and not a latLng object

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Fixed quote style to please rubocop

Allow a POST with no arguments to trace#edit to fetch the form

Remove U+200C unicode "zero width non-joiner" from html.dir

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

merging master into html-mail-fixes

Allow context menu to add notes without panning the map

Update Leaflet.contextmenu to the 1.2.1 release

Make context menu entries translateable

Improve context menu initialisation to avoid namespace pollution

Merge remote-tracking branch 'openstreetmap/pull/1030' into next

A better way to contrain the width of the message box whilst also allowing it to shrink on narrow displays

Setting a hard "width" it very compatible, but won't squeeze on narrow screens. Setting max-width works well, except in Outlook and Hotmail. We resort to old skool table tricks: a table with width=100%, with 3 columns, only oneof which, the middle one, had content, and a fixed with. This works.

Two more "nowrap" instructions removed to accommodate narrower displays

Remove nowrap on footer text, to accommodate narrow screens

Set font on title

Remove invalid YML

Closes #1434

Make the mails squeezable on narrow screens

Only add traces for POST requests

Only create diary entries for POST requests

Test that GET on message#new can't send a message

Fixed alignment & underlining bugs in Outlook / WinMail

Render message properly on error

Only send messages for POST requests

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Avoid squished-up grey borders around the message on narrow screens

Even out the widths of the white bars around the message box

Better alt text for the logo

This is what clients that block images will display, so shorter is better.

Use the same fonts for the footer text as for the rest of the mail

Fix title alignment in Outlook/WindowsMail

Don't display a box at all if the note event doesn't include a comment body

Before this it just displays en empty box, which looks wrong.

Removed italic styling on message body

Contributor BushmanK pointed out quite rightly that italic isn't appropriate for a full body of text.

Force the font on every <p> tag

Without this Outlook and Windows Mail don't honour the font set at the top level. Note that this change predates all the screenshots I uploaded to the wiki on 2017-02-02, I'd just forgotten to commit them.

Tweak cell width for proper centering on narrow screens

Only record a visible object if it exists

Fixes #1430

Update to iD v2.1.2

Update to iD v2.1.1

Merge remote-tracking branch 'openstreetmap/pull/1426'

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Link GPX fixture files instead of stubbing

Cleanup double comment markers

Make TraceController#list sort by id instead of timestamp

The effect is much the same but id is guaranteed to give a stable
sort if two traces have the same timestamp.

Merge remote-tracking branch 'openstreetmap/pull/1347' into master

Fix rubocop warnings

Use .first in place of [0]

Merge remote-tracking branch 'openstreetmap/pull/1401'

Update bundle