Don't try to introduce top-level Makefile at this time

Fix mismatched quote and pluralization

Update Docker instructions to avoid copying entire settings file

Add Docker README and Makefile for helper commands

Introduce docker-compose config for development

Avoid password manager autocompletion on user settings page

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

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

Remove conditional for polyline layer

When a user searches for a route and one is not found the polyline
layer is removed - if they then change the mode of transportation the
event listener will not reroute if there is no polyline layer.

This commit removes the conditional that prevents the routing code from
re-running after the polyline has been removed.

Fixes #3050
Closes #3055

Provide better descriptions for sr-only labels

Use boolean instead of string for hiding labels

Refactor account form to use bootstrap

There's further refactoring that could be done, but this is sufficient for now.

Convert issue comments to use bootstrap

The reassign checkbox could be simplified in future by changing the expected parameter in the controller

Converge redaction forms to use bootstrap

Convert note form to use bootstrap

Fix locate control popup on the main map to use new translations

Fixes #3072

Use model attributes and attribute help for the registration fields

The disply name keeps its specific help, since it is different on the account settings page

Update bundle

Localisation updates from https://translatewiki.net.

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

reserve 60x60 in top right of survey banner

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

community survey banner

Avoid concatenating text to a translation

Fixes #3064

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

Localisation updates from https://translatewiki.net.

Avoid recommending installation of the Ubuntu bundler package

Due to a problem with namespacing and vendoring, the Ubuntu package
leads to namespace conflicts with Thor 1+

Fixes #3013

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

Refactor diary comment form to use bootstrap

Refactor user registration form to use bootstrap

Convert to use bootstrap form

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

Fix a small typo and tests that depend on it.

Redirect after saving settings in case the username has changed

Fixes #3059

Update bundle

Localisation updates from https://translatewiki.net.

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

Bump eslint from 7.17.0 to 7.18.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.17.0 to 7.18.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v7.17.0...v7.18.0)

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

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

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

Refactor user_blocks form to use bootstrap

Further refactoring could involve a virtual attribute for duration.

Bump qs from 6.9.4 to 6.9.6

Bumps [qs](https://github.com/ljharb/qs) from 6.9.4 to 6.9.6.
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ljharb/qs/compare/v6.9.4...v6.9.6)

Signed-off-by: dependabot[bot] <support@github.com>

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

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

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

Refactor login form to use bootstrap

Care has been taken to preserve exact tabindexes, field values etc.

Refactor richtext fields to use a custom bootstrap_form input.

This allows us to use form_group_builder and get all the label and
help text handling in line with other bootstrap_form inputs.

Align the richtext help text height with the height of the textarea.

This works by setting the card to the same height as its containing flexbox. For perfect alignment, we need to make sure the extra margin at the bottom of the text-area stops applying when they are shown side by side.

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

rerun script/misc/update-wiki-pages

Use latest major version for all github actions

Restore ruby 2.5 compatibility

Fix new rubocop warnings

Remove redundant gem requirements for unsupported ruby versions

Unpin rubocop now erb_lint is fixed

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Merge branch 'pull/3035'

Remove spacing tweaks and legacy pagination spacing CSS

No need to fight against something that we no longer use.

Merge branch 'pull/3024'

Remove link_to_if since it's in a clause with the same condition

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

Remove both Potlatch versions

Fixes #2622

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

Localisation updates from https://translatewiki.net.

Bump eslint from 7.16.0 to 7.17.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.16.0 to 7.17.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v7.16.0...v7.17.0)

Signed-off-by: dependabot[bot] <support@github.com>

Traces: Migrate pagination to bootstrap

Same feature, different style.
- Change pagination to use bootstrap default styles
- … but with empty span-tags instead of non-active a-tags
- refactor partial to not use @-variables but get the variables via
the render-call, simplify render-call-syntax
- remove the additional `if @traces.size > 1` inside the pagination
partial since that caused the pagination to disappear on the last page.
The partial will be visible with inactive links.
- Add bootstrap spacer-classes to overwrite commons-css so removing. Eg.
commons also has a pagination class same as bootstrap.

Diary entries #index: Simplify bootstrap pagination

- We can use a span instead of a disabled link as well
- Update to use double quotes everywhere

Add noindex meta tag to diary index pages

Google (and other search engines) should follow all links on this page, but only index the actual diary_entry#show and not the #index. This meta tag tells google to do just that (https://developers.google.com/search/docs/advanced/crawling/block-indexing?hl=en).

Part of https://github.com/openstreetmap/openstreetmap-website/issues/2851

Update bundle

Remove unnecessary inflection

Switch to using the zeitwork autoloader

Verify that public and trackable traces don't return identity details

Diary entries #index: Migrate to bootstrap pagination

Use https://getbootstrap.com/docs/4.5/components/pagination/ for the pagination.

I tried using link_to_if but this does not work well in this case since multiple changes need to happen for the disabled state.

Revert "Switch to using the zeitwork autoloader"

This reverts commit 127880a73fa461400c53b29c98721e34660bca5e.

Fix case for zeitwork

Switch to using the zeitwork autoloader

Update bundle

Localisation updates from https://translatewiki.net.

Merge pull request #3018 from tordans/line-break-for-long-urls

Break long URLs in blog post show

Break long URLs where richtext is used.

For example on the blog post show page.

This uses https://getbootstrap.com/docs/4.4/utilities/text/#word-break to break long URLs on the blog post show view.

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

Refactor diary entry forms to use a custom form builder

Ensure richtext editing style rules are only applied on standard forms

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

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

Return the centre of the image if the bbox has no extent

Fixes #3007. If all the provided points in the gpx file have the same
lat/lon, then an image where they are all at the centre seems plausible.

Remove SystemTimer and use stdlib Timeout directly

SystemTimer was only needed on ruby 1.8, and we dropped support for
that a long time ago.

Localisation updates from https://translatewiki.net.

Update bundle

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

Bump eslint from 7.15.0 to 7.16.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.15.0 to 7.16.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v7.15.0...v7.16.0)

Signed-off-by: dependabot[bot] <support@github.com>

Make coverage testing work with parallel tests

https://github.com/simplecov-ruby/simplecov/issues/718#issuecomment-538201587

Localisation updates from https://translatewiki.net.

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

Merge pull request #3009 from gravitystorm/svg_icons

Add updated authentication icons

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

Increase memory given to the virtualbox vm

Fixes #2972

Like others, I found that the previous setting caused OOM failures, and
4096 was sufficient. I suspect lower values will also work, but I don't
have time or inclination to bisect a smaller value.