Revert "Suppress errors when disabling the focus trap"

This commit duplicates commit e9ffefb4d0db61bf8e12b3824c6515ac16fe3812 which is
the cleaner fix, but was accidentally commited to `master` only. We revert this
one to allow for a clean cherry-picking of
e9ffefb4d0db61bf8e12b3824c6515ac16fe3812.

This reverts commit e7439c4f5fc7daf8d45d2de478c488798ddf463a.

Install phpcs via `shivammathur/setup-php`

Apparently the previously used actions uses an out of date phpcs.

(cherry picked from commit c5cd836751c3fb357b8ddce52a95a83f3ac46347)

Merge pull request #4805 from WoltLab/package-xsd

Added documentation of the void element

Use correct CodeMirror mode for SCSS inputs

`sass` is the indentation based SASS syntax.

Incorrect markup in the authentication dialog

Suppress errors when disabling the focus trap

Use CSS variable to apply the dialog offset

Revert "Dynamic width of code box line numbers"

This reverts commit d3853740d8304f9703876006db990c2e01642656.

Dynamic width of code box line numbers

This avoids indenting short code listings too much, especially on screens with limited space.

Do not wrap code boxes on mobile

Having long lines wrap multiple times is horrible, overflow scrolling is much better and also quite natural on touch devices.

Better accessibility of code box buttons

Newer iOS versions permit a click near the bottom

Prevent the "external url" indicator to wrap

See https://www.woltlab.com/community/thread/295487-einzelnes-umbrechen-des-externalurl-icons-verhindern/

Expand the active menu item by default

Merge pull request #4807 from WoltLab/55-desktop-notifications

Remove the option to disable desktop notifications

Align the mobile main menu to the right

See https://www.woltlab.com/community/thread/295627-mobil-men%C3%BC-links/

Suppress error messages when aborting `fetch()`

See https://www.woltlab.com/community/thread/295566-fetch-is-aborted-bei-suchergebnissen/

Remove the option to disable desktop notifications

Fixes #4806

Added documentation of the void element

Merge branch '5.4' into 5.5

Truncate error message in cronjob log if necessary

Support for meta tags during article import

Typo

Remove obsolete workaround for PHP_CodeSniffer bug

Merge branch '5.4' into 5.5

Merge pull request #4798 from pehbeh/patch-1

Update URL to WoltLab Community and Plugin-Store in acp/pageHeaderUser.tpl

Update URL to WoltLab Community and Plugin-Store in acp/pageHeaderUser.tpl

Fix incorrect use of `mb_strpos` in MysqlSearchEngine

The return value of `mb_strpos` needs to be checked type-safely, as both `0`
and `false` are falsy. In this case this likely was safe, as the inner join may
not appear at the start of the query, it was a questionable nonetheless.

Fix this by using `str_contains()` which makes the intent even clearer.

Merge branch '5.4' into 5.5

Update URL to WoltLab Ticket Support in acp/pageHeaderUser.tpl

Merge pull request #4787 from WoltLab/user-online-list

User online statistics were invisible when all active users were invisible

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Add the `required` attribute to the recipientID select in contact.tpl

see a8490749c3ba7014380d55462fc45dd635c1d71c

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Indicate that selecting a recipient is required in contact.tpl

Applied code suggestions

Show legend only when users are listed

User online statistics were invisible when all active users were invisible

Merge branch '5.4' into 5.5

Drop obsolete update_com.woltlab.wcf_5.4.15_deleteDsStore.php

Drop obsolete fileDelete_5.5.xml

Fix language items in recommended section of system requirements in WCFSetup

see 3445cbe2a005ead9843d9e17709a915631dd11b5
see e88d06dc88bc263b7424fbccfa47c13907413b8c

Release 5.5.0 Beta 2

Incorrect dialog position on smartphones

See https://www.woltlab.com/community/thread/295560-neues-thema-erstellen-au%C3%9Ferhalb-des-bildschirms/

Add the 5.5 branch to GitHub workflows

Release 5.5.0 Beta 1

Removed the upgrade instructions from 5.4.*

Updating minified JavaScript files

Merge branch '5.4'

Fix English versions of `wcf.user.security.multifactor.backup.authenticationEmail.body.*`

The phrases contained broken template scripting, due to the use of the `'`
apostroph within a single quoted string.

Dialogs could become too wide with lots of text

Suppress the redundant loading indicator

See https://www.woltlab.com/community/thread/295539-doppelte-ladebalken/

Merge pull request #4781 from WoltLab/ci-check

Flip only the file name casing on AbstractFileDeletePackageInstallationPlugin::isFilesystemCaseSensitive()

Add fileDelete_5.5.xml

These files never were part of the git repository, but were accidentally
included in an upgrade package, because an unclean source directory was used
when building the upgrade.

Flip only the file name casing on AbstractFileDeletePackageInstallationPlugin::isFilesystemCaseSensitive()

Flipping the whole path does not provide any real benefit, because we only care
about whether the file system where WoltLab Suite resides is case sensitive or
not. It does however break when `open_basedir` is configured.

Merge branch '5.4'

Remove the codestyle workflow from branches that are not master

Lazy loading the user’s avatar causes flashes

Safari (macOS and iOS) does not handle lazy loaded images above the fold gracefully. This causes the user’s avatar to flicker noticeably on ever page navigation.

Merge pull request #4779 from WoltLab/search-result-pagination

Tracked the result page number in the url

Merge pull request #4777 from WoltLab/article-comment-like-notifications

Notifications about reactions to article comments

Merge pull request #4776 from WoltLab/scroll-to-comments

Scrolling to a comment not visible by default did not work

Proper prototype creation

Tracked the result page number in the url

Proper history navigation for searches

Track the page number of search result in the url

Visual separator for specialized search filters

See https://www.woltlab.com/community/thread/295502-kategorie-auswahl-ist-verrutscht/

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Revert "Show always an no selection option in custom select options build with the OptionHandler"

This reverts commit 6fef8b82e15794eee5317e6b15bb0670f137315c.

Incorrect margin of `.formSubmit` in dialogs

This was caused by the negative margin introduced in 1d224f10cdd05e7f5d09f869ee1fbbbdff025749 which caused the calculation to be off by 10px, causing a visible gap.

See https://www.woltlab.com/community/thread/295497-fehlerhafte-mobile-darstellung-von-dialogen/

Removed obsolete imports

Notifications about reactions to article comments

Use generic trait in existing notifications for article comments

Added generic trait for article comment tests

Notification type was not hidden when modules were disabled

Scrolling to a comment not visible by default did not work

Inline editors had been not initialized properly

Unified phrases

Merge pull request #4774 from WoltLab/focus-trap-tabbable

Update focus-trap and tabbable npm dependencies

Fix dependency sync check in javascript.yml

Add check that dependencies are in sync to javascript.yml

Update focus-trap and tabbable npm dependencies

Merge branch '5.4'

Update npm dependencies

Add PHP 8.1 to the php.yml workflow

Drop obsolete exclude in php.yml workflow

This should no longer be required since 123aedf297e0402e6bd3562fe08e2d74e24760d1.

Update version constraints in composer.json

Require the currently locked versions as the minimum versions.

Merge pull request #4772 from WoltLab/htmlpurifier-4.14

Update to ezyang/htmlpurifier 4.14.*

Labeling for buttons (trash and delete) was identical

Update to ezyang/htmlpurifier 4.14.*

Sort dependencies in composer.json

Add explicit composer dependency for guzzlehttp/psr7

Remove validation attributes from __sourceCodeFormField.tpl

These are not supported with CodeMirror, because it only syncs the entered data
upon `submit()` which notably runs *after* validation.

see codemirror/CodeMirror#5092
Fixes #4732

Fix typo in function name in AbstractFileDeletePackageInstallationPlugin

This must be `strtr` not `strstr`.

see #4714

Load the page logo using the `eager` policy

Using `loading="lazy"` for the page logo causes the image to flicker noticeably in Safari on both macOS and iOS.

See https://www.woltlab.com/community/thread/295403-logo-auf-der-startseite-beim-aktualisieren-ein-rechteck/

Merge branch '5.4'

Merge pull request #4768 from WoltLab/sitemap-page

Fix handling of CMS pages in sitemap