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

Add Ascend/Descend when routing with GraphHopper

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

Update rubocop

Update to iD v2.1.0

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

Convert the test_users_nearby to user factories.

Use factory for user language tests

Refactor the friend_users test, and remove the tests which duplicate the (renamed) test_friends_with above.

Convert test_friend_with to use factories, and use alice/bob/charlie for easier understanding.

User factory for uniqueness tests

Convert class method tests to use User factory.

Also improve the tests by being explicit about which results should
be returned, rather than just hoping that the fixtures cover all
possibilities.

Add status traits to user factory.

Add moderator_user and administrator_user factories.

Add a user_role factory

Naming the association in the model makes the relationship easier
to express in the factory.

Add with_home_location trait for user factories.

Create a simple user factory, and convert some basic tests.

Reduce maximum changeset size to 10k changes

Ten thousand changes ought to be enough for any normal mapping activity.

Automatic edits and imports are controlled by scripts anyway so it doesn't
make a difference to them as they just have to adapt the limit.

The reason for my suggestion is that large changesets are becoming
increasingly difficult to handle due to frequent timeouts when trying
to load and process them.

The changeset size limit is returned by the API in the "capabilities" request
so client software that honours that information will automatically pick up
the new limit.

Closes #1259

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

That new message in the HTML version of the mail should also be added to the plain text version of the mail

Duh.

Use each_with_object and make the code pithier

Renamed `apply_inline_css` to `style_message` as suggested

Set border=0 on that img

IE9 shows a big nasty blue border on linked images if you don't specify border=0

Convert client applications fixtures into a factory

Closes #1415

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

Load user fixtures for ChangesetComment model tests

Localisation updates from https://translatewiki.net.

Left-align logo

Apply inline styling to all <p> tags

Solved the problem of <p> tags being given different padding or color by different clients, without having to copy-paste a string of CSS into every <p> tag in the templates.

Remove unused user_tokens fixture and pointless test.

Put the user avatar in a <p> block

This improves rendering on those clients that set the margin to 0 on <p> blocks.