Merge remote-tracking branch 'upstream/pull/2496'

Fix typos

Added unit tests

Avoid double-escaping display names when sent through the translation system

They are escaped properly without the h() call, and doing that just double-escapes them

Remove no longer needed rubocop exemptions

Update tests to ensure referer is working

This reinstates the form tests in the controller test, but uses
the system tests to actually click the button and make sure that
it works

Remove the custom basic grid css rules

This is now unused, and the bootstrap grid is much more powerful.

Use bootstrap grid for signup page

This allows us to have the two sections one above the other on small screens,
with the call to action text first, and then side-by-side above the sm
breakpoint with the cta on the right.

Remove the custom layout css from the terms page

This uses bootstrap utilities where necessary. It also removes the
terms page from a padding override, which was then in turn overridden
by a padding class applied to the form!

Refactor the welcome page to use the bootstrap grid and alert box

The two columns work well side-by-side even at small screen widths.

Refactor the fixthemap page to use the bootstrap responsive grid

This ensures that on small screens, the 'how to help' sections
are one above the other, and on larger screens they are side-by-side

Remove extra styling for rss icon

This brings it into line with e.g. the new diary entry icon

Rework changeset descriptions to use h6 tags, and refactor changeset lists to use list-group-flush

The descriptions are probably best not as a header, but still needs to be something more prominent than surrounding text.

Remove font size override on sidebar tags tables

Refactor the query results to use bootstrap list-group-flush

Use a post link to logout

This avoids needing to access the session id, which is currently
only working with the memcache store.

The fallback page is preserved for anyone who wants to logout without
using javascript.

Refs #2488

Remove fallback to hostip.info

We haven't used hostip.info in production for years, except
for when we did so by accident, leading to a two hour partial
outage when it went down.

Localisation updates from https://translatewiki.net.

Make sure custom settings are picked up by tests

Version 2.1.0 of the config removes settings.local.yml from
the files read in the test environment.

Update bundle

Add some missing _html suffixes

Merge remote-tracking branch 'upstream/pull/2494'

Set X-Entity-Ref-ID header on notification emails

This disables threading in gmail completely - without it all
notifications of a given type are grouped together.

Use ActionView's safe_join to avoid using raw when joining arrays of text

Use _html suffix to avoid using raw when displaying translated strings

This is safer than raw, since any user input is still escaped.

Use _html suffix to avoid using raw when displaying translated strings

This is safer than raw, since any user input is still escaped.

JSON output, handle Accept header

Localisation updates from https://translatewiki.net.

Unify lat/lon formatting for json output

Merge remote-tracking branch 'upstream/pull/2491'

Merge remote-tracking branch 'upstream/pull/2490'

Remove the dashed underline from prose headings

No other headings are underlined, and the underlines look more like
a section separator (e.g. dashed hr) and so visually separated the
heading from its content.

Remove usused css for about page

Also removes some whitespace on sections and headings that can be
better left as default.

Split the legal_html into two, so that we can use proper paragraphs instead of a linebreak

Also add a missing full stop.

Reinstate background colour for sidebars

This is required when the sidebar overlaps the map, for example when
it is not full height on the welcome message.

Fix hostip.info fallback

Switch GeoIP code to use GeoIPv2 databases

Update bundle

Localisation updates from https://translatewiki.net.

Test language specific versions of about and copyright pages

Mark language and direction for about page

Add some missing route tests for the site controller

Merge remote-tracking branch 'upstream/pull/2444'

Merge remote-tracking branch 'upstream/pull/2462'

Added helper function set_references for mail header

Merge remote-tracking branch 'upstream/pull/2471'

Merge remote-tracking branch 'upstream/pull/2470'

Merge remote-tracking branch 'upstream/pull/2469'

Merge remote-tracking branch 'upstream/pull/2473'

Move default_format_xml to api_controller

API JSON output

travis: update structure.sql

travis: add missing packages

Use settings.local.yml to enable memcache during travis runs

Update bundle

Merge remote-tracking branch 'upstream/pull/2475'

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'upstream/pull/2480'

fixed "end tag" -> "source"

Fixed inattentiveness from my last commit.

Merge remote-tracking branch 'upstream/pull/2477'

Merge remote-tracking branch 'upstream/pull/2476'

removed end tag ("source" is a self-closing tag)

removed end tag ("source" is a self-closing tag)

removed end tag ("source" is a self-closing tag)

Fix errant CSS color properties

Translate units used by leaflet-locatecontrol

Merge remote-tracking branch 'upstream/pull/2474'

Fix CSS errors

Update to iD v2.17.0

Localisation updates from https://translatewiki.net.

travis: change dist to bionic

Require plugins in .rubocop_todo.yml

Fixes #2472

Use _html suffix to avoid using raw when displaying translated strings

This is safer than raw, since any user input is still escaped.

Merge pull request #2464 from hikemaniac/fix-comment-textarea

Improve textarea behaviour

Use _html suffix to avoid using raw when displaying translated strings

This is safer than raw, since any user input is still escaped.

Use _html suffix to avoid using raw when displaying translated strings

This is safer than raw, since any user input is still escaped.

Replace custom css class with bootstrap inline utility

This has actually been broken since the controller was renamed,
but nobody noticed.

Use badges for the trace visibility labels

Our green is currently just on the side of triggering black text,
hence the override to make it look consistent with the other badges.

Use text-danger style for pending trace warning

Remove font-size overrides for traces and user lists

This makes them much easier to read

Use a warning alert for reporting new issues

This removes custom colouring for the alert. The list overrides
need to stay until we unpick more of our custom styling.

Use offwhite instead of grey for browse pages tag tables

Remove more header font size overrides

This smooths out inconsistencies in the hierarchies, for example when
using header sizes beyond h3 in profile descriptions.

Minimially invasive fix for sidebar heading sizes

I'd prefer to avoid overriding the heading sizes, but this will involve
more extensive changes to the layout. For example, the h2 heading has
specific padding and margins that break when moving to h3.

Fixes #2467

Localisation updates from https://translatewiki.net.

Update COP names

Update to rails 6.0.2.1

Let all textareas only be vertically resized

Merge remote-tracking branch 'upstream/pull/2466'

Merge remote-tracking branch 'upstream/pull/2452'

Reformat params hashes to avoid super-long lines

Move Style/Documentation into the main config file

I don't think anyone intends to write documentation for the app

Note events use "opened" not "open" as the status

Rename translations for trace#delete to trace#destroy

Fixes #2465

Make all translation lookup errors throw exceptions in test mode

Avoid errors when /edit is called on an invalid object

Localisation updates from https://translatewiki.net.

Fixed comment textarea going off the sidebar, added border-radius to form fields

Update to rails 6.0.2

Update bundle