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

Localisation updates from https://translatewiki.net.

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

Ensure message destroy button has danger colour when showing the message, and add button to show page when viewing sent messages

Ensure the message title shows in the heading for both sent and received messages

I'm not sure why they were ever different.

Add a title in the heading to match the main menu text

Convert mailbox navigation to use tabs

Move profile-related settings to their own form

Refs #3167

Remove code complexity around resetting language preferences

This was originally introduced since we saved the user and showed
the result on the same action. Now that the preferences controller
saves and redirects, the user model and associated language preferences
are reloaded between requests, and this code is no longer required.

Split user preferences into a separate page

Refs #3167

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

Bump coverallsapp/github-action from 1.1.2 to 1.1.3

Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 1.1.2 to 1.1.3.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](https://github.com/coverallsapp/github-action/compare/v1.1.2...1.1.3)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

Allow cross origin access to OAuth 2 token endpoints

Allow cross origin access to trace data

Fixes #3252

Update bundle

Allow acls to match on parent domains

Added historic=rune_stone to en.yml

Localisation updates from https://translatewiki.net.

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

Bump eslint from 7.29.0 to 7.30.0

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

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

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

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

Update banner FAQ.

This adds a contrast requirement for the upper-right corner so the
'X' can be seen. It also adds a requirement that PRs be created
before the banner needs to be shown, in order to allow for proper
review.

Check that the permission is non-zero

Fixes #3241

Localisation updates from https://translatewiki.net.

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

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

Refactor About header to use bootstrap and flex

This reduces the amount of custom CSS required for the layout, and
makes it responsive on small screen sizes.

Alight the max-width of the content to one of the bootstrap breakpoints

This makes various things easier to reason about, as well as making
some grid things easier too.

Use bootstrap font family

This contains and expanded list of fonts, covering more platforms
and edge cases than ours, so there's no need to override.

Remove some default styling from body

These mirror, either exactly or pretty close to exactly, what is set
by the bootstrap reboot style, so there's no need for us to duplicate.

Remove some references to webmaster

Fix some broken translation lookups

Fix new erblint warnings

Fix new rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

Switch web site to use OAuth 2

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

Make /api/0.6/permissions work for OAuth 2 clients

Bump leaflet.locatecontrol from 0.73.0 to 0.74.0

Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.73.0 to 0.74.0.
- [Release notes](https://github.com/domoritz/leaflet-locatecontrol/releases)
- [Changelog](https://github.com/domoritz/leaflet-locatecontrol/blob/gh-pages/CHANGELOG.md)
- [Commits](https://github.com/domoritz/leaflet-locatecontrol/compare/v0.73.0...v0.74.0)

---
updated-dependencies:
- dependency-name: leaflet.locatecontrol
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

Restore browse-section class to changeset view

Fixes #3234

Update to rails 6.1.4

Revert "Localisation updates from https://translatewiki.net."

This reverts commit 93660ec9e86b683bb7bce6e0f51c8affcf3066f7.

Enable access token reuse for OAuth 2

Update doorkeeper configuration file from master template

Localisation updates from https://translatewiki.net.

Update bundle

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

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

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>