Refactor terms declined flash message to use a partial

This allows us to include a link in the flash message, without having
to use html_safe. Also refactor to avoid having html angle brackets in
the translation strings, while still allowing locale-specific urls.

Refactor flash messages to use a flex row for positioning

This works better for narrow widths and needs less custom CSS.

Ensure that all floated elements in the header stay within it

In certain circumstance, due to either extreme narrow screen widths,
or due to padding on floating elements exceeding the header height,
some things could appear outwith the header. This clearfix avoids
this from happening.

Remove unnecessary padding-bottom override on the header h1

Use paragraphs instead of line breaks, and avoid using html in translation strings

Use hash-based flash objects to render complex flash messages

Since flash objects can only be String, Hash or Array (notably excluding SafeBuffers), then this approach is necessary to render complex html in a safe manner.

Each local can be treated as an (unsafe) string, and therefore escaped normally when rendered into the template. The template (and translation strings) can
contain html since they are no longer stored in the flash as a plain string.

Fixes #3215

Don't mark banner cookies as HttpOnly

Fixes #3231

Avoid accidental locale changes in tests

Fix new ruboop warnings

Update bundle

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

add banner for SotM 2021

Localisation updates from https://translatewiki.net.

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

Bump eslint from 7.28.0 to 7.29.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.28.0 to 7.29.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.28.0...v7.29.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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

Remove the Yahoo OpenID link as they no longer support OpenID

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

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

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

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

Wrap the rss and upload buttons in a div, with suitable padding for both vertical and horizontal tab layouts

This ensures it looks right when the tabs are collapsed vertically.

Rework tabs to use flexbox

This improves the behaviour on small widths, where the tabs stack neatly. It also avoids the use of ml-auto which isn't RTL compatible.

The change in indentation is because it was previously coded with multiple li nested inside another li

Make tab labels into nouns that describe what's on the tab

I think that works better with tabs, whereas actions (verbs) work better with buttons.

Rearrange tabs for inter-page consistency

The all traces tab comes first, since it's shown on every page as you click around. Otherwise the tabs appear to be constantly rearranging themselves.

Fixed remaining erblint complaint

Fixed missing translation test failures

Fix erblint complaints

Traces#index: Introduce tab navigation, fix tag-filter

- introduce bootstrap tabs to switch the views
- introduce `content_for :heading_class` to remove the padding below the bootstrap tabs
- update the rss-image to use a svg, adopted from https://icons.getbootstrap.com/icons/rss/ (without the outer border)
- move rss- and new-button away from the view-switching actions
- the `@tag` logic was broken. introduce new link to remove the tag-filter; the tabs keep the filter once given; use params[:tag] directly in the view
- use `&.` syntax nil-safety so we can remove `@display_name`

Remove unused padding classes

These were last used for sidebar padding, which is now handled more easily.

Rework sidebar to use padding on the sidebar_content div

This saves every header, list, paragraph etc having to sort out their own padding. The couple of instances where we want edge-to-edge can be acheived using the negative margin spacing utilities.

The padding is based on $spacer so that it can be adjusted automatically via bootstrap configuration.

This also means that we can remove many (mis-)uses of the browse-section class, which is only supposed to be for cases where there are multiple browse-sections in sequence (e.g. multiple nodes in the node history browse pages).

Fix the email address that's suggested for whitelisting

Refs #3215

Rework the routing credit to use a bootstrap class

The padding is unnecessary given it's in a paragraph anyway.

Match the width of the overlay-sidebar on narrow screens to the width on full-sized screens

It's not clear why they are different widths, since the non-overlay sidebar doesn't change.

Skip intermediate lengths when testing tag validity

It's really the upper and lower boundaries that we want to check,
and it's reasonable to assume all the intermediate values will work
fine if the boundary values are tested.

This also saves about 75% of the time taken and almost 85% of the
assertions when running all the model tests.

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

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

Set updated_by when resolving issues

Refs #3211

Merge pull request #3208 from osm-hr/update-rcn-color

fix map key color for regional OpenCycleMap

Fix case when user block durations are slightly negative

The output from ActiveSupport::Duration is wildly unhelpful in those cases

Fixes #3210

Ensure that short duration blockss are shown as '0 hours'

Passing `:count => nil` means that i18n.t will return the hash of translations, so use a default value of 0 when checking for hours to prevent this.

Refs #3210

Drop last vestiges of ruby 2.5 support

Update bundle

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

Localisation updates from https://translatewiki.net.

Bump eslint from 7.27.0 to 7.28.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.27.0 to 7.28.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.27.0...v7.28.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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

Fix erblint warnings

Set a referrer policy

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

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

Added missing dependency for fedora install

fixed nodejs package name for fedora

Localisation updates from https://translatewiki.net.

Fix new rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

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

Bump actions/cache from 2.1.5 to 2.1.6

Bumps [actions/cache](https://github.com/actions/cache) from 2.1.5 to 2.1.6.
- [Release notes](https://github.com/actions/cache/releases)
- [Commits](https://github.com/actions/cache/compare/v2.1.5...v2.1.6)

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

Localisation updates from https://translatewiki.net.

fix map key color for regional OpenCycleMap, as it changed from light cyan to purple on the map.
Closes: https://github.com/openstreetmap/openstreetmap-website/issues/2785

Update bundle

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

Bump ohauth from 1.0.0 to 1.0.1

Bumps [ohauth](https://github.com/osmlab/ohauth) from 1.0.0 to 1.0.1.
- [Release notes](https://github.com/osmlab/ohauth/releases)
- [Changelog](https://github.com/osmlab/ohauth/blob/main/CHANGELOG.md)
- [Commits](https://github.com/osmlab/ohauth/compare/v1.0.0...v1.0.1)

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

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

Localisation updates from https://translatewiki.net.

Bump eslint from 7.26.0 to 7.27.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.26.0 to 7.27.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.26.0...v7.27.0)

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

Fix translation text

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

Fix downloading of image variants

Use built-in translation lookups for form helps

Use Update and Register as actions, to be consistent with client_applications

Rework some of the OAuth2 views, adding links and messages when there are no applications.

Switch to using 'rails test:all' in documentation and CI

Fixes #3116.

This allows us to drop our custom patch for running system tests,
as well as opening up all the usual rails test goodies like running
individual files or directories.

Allow OAuth 2 authorizations to redirect to anywhere

Show scope names in application views

Allow OAuth 2 to redirect to plain HTTP for localhost

Add tests for OAuth2

Add support for OAuth2 using doorkeeper

Update bundle

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

Bump puma from 5.3.0 to 5.3.1

Bumps [puma](https://github.com/puma/puma) from 5.3.0 to 5.3.1.
- [Release notes](https://github.com/puma/puma/releases)
- [Changelog](https://github.com/puma/puma/blob/master/History.md)
- [Commits](https://github.com/puma/puma/compare/v5.3.0...v5.3.1)

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

Enable variant tracking for Active Storage

Fix eslint warnings

Enable some more rails 6.1 defaults

Mark javascript issued cookies with SameSite=Lax

Switch to using js-cookie to manage cookies from javascript

Default rails generated cookies to SameSite=Lax

Localisation updates from https://translatewiki.net.

Switch to new defaults for queue names

Enable new Active Job defaults

Enable link header for asset preloading

Enable new connection handling API

Enable use of URL safe CSRF tokens

Fix rubocop Rake/MethodDefinitionInTask warnings

Add ruby 3.0 to test matrix

Localisation updates from https://translatewiki.net.

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

Use image_processing macros to resize images