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

Remove last custom reset styles

Remove more reset styles that are handled by bootstrap

Remove some reset styles and clearfix

These are included in bootstrap

Remove list style reset, and add overrides where necessary

This allows the bootstrap list typography to work without being overridden. It also means
that lists look like normal lists by default, instead of needing to be recreated from
scratch each time. Only the cases where padding and discs are unwanted need to have
custom CSS.

Rename form-row to standard-form-row to avoid clashing with a bootstrap class

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

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

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

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

Rework issues display page to be responsive

This change uses the bootstrap grid to make the issues page responsive,
and avoids having lots of custom widths of divs specified via CSS.

It also changes the display of the reports and comments to be more
like diary entry comments, by having all metadata on one line and shown
before the comment / report text.

Minor tweaks to copyright heading

This fixes a small bug where the hr was shown even when not required,
by ensuring the test is run before creating the content.

Also fixes where an h1 tag was used instead of a div.

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

Localisation updates from https://translatewiki.net.

Remove extra div wrappings from list items

These are unnecessary, and also prevent the secondary-actions vertical
border from appearing.

Localisation updates from https://translatewiki.net.

Avoid double escaping a name

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

Fix translation names

Don't validate the revoker for a block if it hasn't changed

The revoker only has to be a moderator at the time they make the
revocation - as things stood trying to view a block that had been
revoked by somebody that was no longer a moderator failed.

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

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

Fix indentation of query results

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

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

Fix typos

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

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.

Localisation updates from https://translatewiki.net.

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'

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