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

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

Enable more brakeman checks

These have been refactored already, so no longer need to be skipped.

Localisation updates from https://translatewiki.net.

Create a second title_user translation, to avoid using html_safe

Reconfigure the suspended flash message to avoid html_safe

This also avoids having raw html in the translation strings

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

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

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

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

Merge pull request #3779 from AntonKhorev/spinner

Replace searching.gif images with Bootstrap spinners

Use rails 7.0 configuration defaults

All the entries in the new framework defaults were enabled a while
ago, apart from the cookies serializer which is already defined in
config/initializers/cookies_serializer.rb

Relax strict version constraint on rails gem

This lets us receive 7.0.x patch releases without having to edit the Gemfile.

Pass original event instad of leaflet event to changeset bbox click simulation

Minor documentation tweaks

Prefer 'you' to 'one' since 'you' is used throughout the documentation and is easier for non-native speakers.

Standardise on openstreetmap-website instead of rails port

Fixes #3796

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

Fix feature request GitHub issue template

The syntax for the feature request GitHub issue template yaml file was wrong because two elements had the same ids and thus the feature request issue form did not work. This changes one that was the same.

Update bundle

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

Localisation updates from https://translatewiki.net.

clear default connections when using preauth

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

Update to iD v2.23.2

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

Update to iD v2.23.1

Localisation updates from https://translatewiki.net.

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

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

Update bundle

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

Bump eslint from 8.26.0 to 8.27.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.26.0 to 8.27.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.26.0...v8.27.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.

Bump leaflet.locatecontrol from 0.76.1 to 0.78.0

Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.76.1 to 0.78.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.76.1...v0.78.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>

Localisation updates from https://translatewiki.net.

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

Remove outdated comments

The action no longer deals with sending, and there's no display_name
param in this bit of code so it's a bit confusing.

Enable the ActionOrder cop for remaining controllers

Where actions were reordered, the rails standard actions were
also moved to the top of each controller.

Fix new rubocop warnings

Update bundle

Remove query result geometry when unloading the page

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

Replace spinners in javascript

Replace spinners in html.erb templates

Add aria-label to close buttons

Localisation updates from https://translatewiki.net.

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

Remove query spinner show delay because the spinner is not hidden

Localisation updates from https://translatewiki.net.

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

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

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

Add guidance on making pull requests easier to review

Use close button instead of geolink in sidebar content

Merge changeset <ol>s on Load more

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

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

Fix new rubocop warnings

Update bundle

Pass min/max lat/lon to notes index rss builder

Pass min/max lat/lon to notes rss feed builder

Localisation updates from https://translatewiki.net.

get_text("reversegeocode/result") returns an XML node.  We want to get the text value of that node.

This will unescape &#039; into an apostrophe.

All callers of this function will later re-encode it depending out output whether it be HTML in an email, or XML in an RSS feed.

* app/mailers/user_mail.rb
* app/helpers/geocode_helper.rb
* app/views/api/notes/feed.rss.builder
* app/views/api/notes/_note.rss.builder

Fixes openstreetmap/openstreetmap-website#3761

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

Merge pull request #3764 from openbrian/patch-3

another trailing space

another trailing space

My editor does this too.  There's one more trailing space in this area of the file.

Bump eslint from 8.25.0 to 8.26.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.25.0 to 8.26.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.25.0...v8.26.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.

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

Merge pull request #3694 from AntonKhorev/share-button-group

Use Bootstrap button group for share tabs

Merge branch 'pull/3692'

Minor tweaks to the markdown, in particular to wrap code characters in backticks

Merge pull request #3693 from lectrician1/patch-2

Clarify Windows installation bad + alternatives

Merge pull request #3700 from AntonKhorev/about

Replace <h5> used to set the font size with Bootstrap class h5

Merge pull request #3704 from AntonKhorev/leaflet-control-groups

Group map controls explicitly

Update bundle

Test that tags are included correctly in the changeset feed

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

Fix malformed tags table in changeset atom feed

Localisation updates from https://translatewiki.net.

Require multi_json for windowslive logins

The windowslive omniauth plugin needs multi_json but doesn't require
it and the omniauth core is no longer doing so.

Allow credentials to be sent with overpass calls

Localisation updates from https://translatewiki.net.

Unflip the heading text on the new user screen for RTL languages

Fixes #3752

Update bundle

Localisation updates from https://translatewiki.net.

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

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

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

Don't scroll to or show markers of notes with no data

Display icon for current note only if this icon is defined

Bump eslint from 8.24.0 to 8.25.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.24.0 to 8.25.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.24.0...v8.25.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.

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

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

Fix concurrency groups for github actions

Load note from edit link with remote control

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

Remove whitespace eslint complaining about

Remove note from edit link when it's not a current object