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

Remove extra container div from fixthemap page

This created extra margins and padding either side of the blocks
of text, which was unnecessary

Rework welcome and fixthemap pages

* Use lead class for lede paragraphs
* Change headings to start from h2
* Remove custom css and use bootstrap utilities where required

Remove font size overrides for code and pre blocks

These are taken care of, where necessary, by bootstrap

Remove font size overrides from sidebar headings

These were interfering with the response font sizing via RFS

Use th tags on trace show table

This brings consistency with other similar tables

Fix the font weight on the about page title, and use lead styling for the lede

Remove unnecessary br tags

The margin on the table takes care of the spacing.

Test fetching the iD frame when not logged in

Test that verified emails from OAuth signups are accepted automatically

Update bundle

Don't try and write directly to the user status

This causes an exception when processing the authentication callback for
providers like google where we treat the email address as verified.

Require a logged in user for the iD frame

The frame will error if the user is not logged in as it wants
to be able to access their OAuth token.

Localisation updates from https://translatewiki.net.

Monkey patch MailDeliveryJob to work with delayed_job under ruby 3

The delayed_job gem does not currently perserve keyword arguments
properly leading to errors when running under ruby 3 and try to run
a job that requires them.

This affects our deferred mail deliveries so we monkey patch the
mail delivery job to unpack the hash from the normal arguments and
use it to provide the expected keyword arguments.

https://github.com/collectiveidea/delayed_job/issues/1134

Fix test for change in class

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

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

Bootstrap 5: Change rounded-lg to rounded-3

Uncomment asserts for namespaced elements in note rss test

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

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

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

Add xmlns:dc namespace to note show rss actions

Localisation updates from https://translatewiki.net.

Bootstrap 5: Use dropdown-menu-end for dropdown alignment

Bootstrap 5: Fix table striping background colour

Bootstrap-5: use logical alternative for text alignment

Using start/end rather than left/right, for better RTL support.

Bootstrap 5: Use symmetrical padding in toggled dropdown

Required since pl-* classes have been removed, but also works fine
without bringing start/end considerations into play.

Rework trace index to use secondary-actions

The border between items is familiar UI, so we can keep it, but lets
use the secondary-actions approach instead.

This avoids using left-right specific border, margin and padding
classes

Bootstrap 5: Remove the padding from trace nav buttons on larger sizes

If additional py is added when the tabs are laid out in full, then the
other tabs start to become detatched from the rest of the page and a
line appears under the active tab.

We don't need the padding in the horizontal layout, so remove it, and
tweak the size of the padding when collapsed.

Bootstrap 5: Override button text colours, force to white

Use bootstrap variables to style links

Bootstrap 5 now has a hover color, and instead of trying to override
with CSS, it's easier to use the variables instead.

Bootstrap 5: data-toggle renamed to data-bs-toggle for dropdown menus

Bootstrap 5: no-gutters renamed to g-0

Bootstrap 5: Replace card-deck with a row-cols-* based grid

Bootstrap 5: Replace all form-group classes with mb-3

Bootstrap 5: Add form-label class to hand-crafted form labels

This is done automatically for forms created by bootstrap_form

Bootstrap 5: Override float attribute on legends

I don't fully understand why bootstrap added this in v5, as the
discussion on the issue tracker isn't clear. But we neither want
nor need this.

Bootstrap 5: Update manual select boxes to use form-select class

Bootstrap 5: Remove use of form-row class

This has been replaced by simple `row` class in v5. We can also use the
gx-N classes here, which tighten up the horizontal gutter sizes for
space-constrained forms.

Bootstrap 5: Enable negative margins

Bootstrap 5: change rounded-sm to rounded-1

Upgrade to bootstrap 5

Switch actions workflows to use ruby/setup-ruby

Update bundle

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

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

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

Strip leading/trailing whitespace from comma-separated trace tags

Bump eslint from 8.19.0 to 8.20.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.19.0 to 8.20.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.19.0...v8.20.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 branch 'master' into trace-description-maxlength

Localisation updates from https://translatewiki.net.

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

optimize image file size

remove blue circle in top right corner, improving contrast with "x" button

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

Update bundle

add banner for sotm 2022

Set trace description input max length

Localisation updates from https://translatewiki.net.

Update to rails 7.0.3.1

Update bundle

Localisation updates from https://translatewiki.net.

Add a configuration option to disable HTTP basic authentication

Log any use of basic authentication

There is no need to do setup_user_auth and authorize for the same action

Localisation updates from https://translatewiki.net.

Update bundle

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

Bump eslint from 8.18.0 to 8.19.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.18.0 to 8.19.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.18.0...v8.19.0)

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

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

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Fix new rubocop warning

Update bundle

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

Bump qs from 6.10.5 to 6.11.0

Bumps [qs](https://github.com/ljharb/qs) from 6.10.5 to 6.11.0.
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](https://github.com/ljharb/qs/compare/v6.10.5...v6.11.0)

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

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

Localisation updates from https://translatewiki.net.

Fix eslint warnings

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update bundle

Restore intended title for GPS trace view

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

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

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

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

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

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

Fix map edit controls min-zoom behaviour

The map dropdown and tooltip of the map edit controls should
behave similar when zooming in and out.
This is achieved by disabling the button-group and the buttons.

Bump eslint from 8.17.0 to 8.18.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.17.0 to 8.18.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.17.0...v8.18.0)

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

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

Localisation updates from https://translatewiki.net.

Update to iD v2.21.1

Fix small error in _header.html.erb

only parse hashParams if there is a hash

Presumably this was an oversight in the original implementation, as `OSM.params` falls back to the page's `query` string if the function's parameter (here: the page's `hash`) is an empty string.

This fixes some inconsistent behaviour of the website. In the example below, the first URL sets the `background` to `none` while in the second example it doesn't.

* https://www.openstreetmap.org/edit.html?background=none
* https://www.openstreetmap.org/edit.html?background=none#map=…

Fix new rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

Remove duplicate keys from en locale data

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