Merge pull request #5592 from WoltLab/numberformat-php

Fix number formatting in StringUtil

Fix PHP codestyle

Fix StringUtil::formatDouble() for $maxDecimals != 2

NumberFormatter performs rounding itself, thus there is no need to round
manually. This also changes the recommended alternative for this use case that
is mentioned in the doc comment.

Limit the NumberFormatter to 2 fraction digits in StringUtil::getNumberFormatter()

see https://www.woltlab.com/community/thread/300795-startseite-beitr%C3%A4ge-pro-tag-ist-%C3%BCbertrieben-genau/

Fix handling of negative numbers in StringUtil::formatInteger()

The formatted value was compared against `0`, not the original integer. Just
delegate to `formatNumeric()`.

Add missing `.button` class in PaypalPaymentMethod

Fix the position of the quote selection

It shares the `balloonTooltip` class but is not part of the page overlay container.

See https://www.woltlab.com/community/thread/300757-quote-by-marking-text-tooltip-is-misplaced-when-page-is-scrolled/

Release 6.0.0 Alpha 7

Updating minified JavaScript files

Update the CKEditor 5 bundle

Improve the readability of mentions in the editor

See https://www.woltlab.com/community/thread/300743-vorschl%C3%A4ge-f%C3%BCr-erw%C3%A4hnungen-etwas-vergr%C3%B6%C3%9Fern/

Disable image resizing

See #5591

Merge pull request #5590 from WoltLab/better-scroll-disable

Improve the behavior of visible scrollbars

Closing dialogs with Esc should close and detach the dialog

See https://www.woltlab.com/community/thread/300742-seite-nicht-mehr-scrollbar-wenn-dialog-mittels-esc-geschlossen-wird/

Fix margins in embedded content widget

Add a default value for the x offset of legacy dialogs

The lack of a valid value sometimes caused the dialog to be squished against the side of the document, causing an incorrect width calculation.

Simulate the visible sidebar when a dialog is open

Always force the vertical scrollbar to be visible

Remove the outdated module import

Fix the scroll freeze handling on mobile

See https://www.woltlab.com/community/thread/300714-page-jumps-on-mobile-when-opening-reaction-dialog/

Add a workaround for the collapsing borders in Firefox

See https://www.woltlab.com/community/thread/300725-broken-hover-effect-for-tables-in-ckeditor/

Merge pull request #5589 from WoltLab/editor-align-order

Use “standard” order for alignment options in CKEditor toolbar

Limit the ToC to a maximum of 50% width on large devices

See https://www.woltlab.com/community/thread/300544-maximalbreite-f%C3%BCr-inhaltsverzeichnis-setzen/

Fix the position of tooltips

See #5575

Fix the color of the balloon tooltip pointer

See https://www.woltlab.com/community/thread/300637-incorrectly-colored-triangle-pointer-for-image-controls-in-ckeditor-in-dark-mode/

Restore the mobile search icon when closing the search

See https://www.woltlab.com/community/thread/300653-mobiles-such-icon-%C3%A4ndert-seinen-status-nicht/

Suppress button hover states on touch devices

See https://www.woltlab.com/community/thread/300684-mobil-ist-es-nicht-erkennbar-wenn-man-eine-formatierung-deaktiviert/

Use “standard” order for alignment options in CKEditor toolbar

It might be arguable whether it should be "left, center, right, justify" or
"left, right, center, justify", but in no case "center" should come first.

"left, center, right, justify" is used by Google Docs and LibreOffice Writer.

Provide a sane width attribute for attachments

Report image attachments to the editor

See https://www.woltlab.com/community/thread/300722-broken-image-icon-when-editing-a-post-with-a-video-file-attached/

Move popover into the body element

These elements do not exist within dialogs and their behavior does not play well with fixed positioning.

Fix the position of balloon tooltips

Fix the handling of the width attributes of `[attach]`

Merge branch 'master' of https://github.com/WoltLab/WCF

Make explanation of notification setting more precise

Use fixed positioning for drop-down menus

Merge branch 'master' of https://github.com/WoltLab/WCF

Fix some logic errors when sending comment notifications

The old code assumed that the active user was also the author of the respective comment, but this is not guaranteed.

Fix typo

Fix a typo in the style variable value

Release 6.0.0 Alpha 6

Updating minified JavaScript files

Merge pull request #5588 from WoltLab/ckeditor-image-resize

Add image resize support in CKEditor

Update the CKEditor 5 bundle

Fix the positioning of tooltips

Fixes #5575

Fix the installation through the projects

Fix the image resizing of aligned images

Fix a typo

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Clean up the code

Add resize support for attachments

Add support for resizable attachments

Merge pull request #5587 from WoltLab/attachment-bbcode-refactor

Refactor `AttachmentBBCode`

Invert the logic by short-circuiting for thumbnails

Simplify the handling of the alignment CSS classes

Slightly improve the consistency of HTML generation

Use short circuits and replace large string concats with `\sprintf()`

Split the `getParsedTag()` method into specific functions

Remove long deprecated support for external object lists

This features was deprecated 9 years ago in https://github.com/WoltLab/WCF/commit/0a2b09a2d0d914887eedf43d02328add19ec4216

Merge pull request #5586 from WoltLab/package-installation-submit-ux

Improve the UX around submitting package installation requests

Show the regular AJAX status in package installation until the first request goes through

Until the first request goes through, nothing is visible, as the dialog is not
yet spawned. Show the regular AJAX status to improve the UX.

Disable submit button in package installation confirmation after clicking

It might take a while for the request to go through and the admin should not
click twice.

Merge pull request #5584 from WoltLab/reset-date-selection-ui

Improve UI of the reset button in the date picker

Merge remote-tracking branch 'origin/master'

Merge branch '5.5'

Fix truncation of Unicode string query parameters in Benchmark

This needs to use `mb_substr()`, as we checked UTF-8 validity before to use
`UNHEX()` with binary strings. Previously UTF-8 sequences might've been cut
short and the resulting invalid sequence cannot be JSON encoded, as JSON
requires strings to be valid UTF-8.

see https://www.woltlab.com/community/thread/298853-schwer-reproduzierbar-fehlermeldungen-beim-importieren-von-sprachdateien/

Remove obsolete code

Improve UI of the reset button in the date picker

Merge pull request #5577 from WoltLab/unbundle-ckeditor

Defer loading of CKEditor and unbundle its components for the accelerated guest view

Merge pull request #5581 from WoltLab/newsfeed-loading

Improve the loading experience of the newsfeed in dark mode

Improve the behavior on mobile, use the loading indicator component

Merge pull request #5578 from WoltLab/light-mode-contrast

Improve the contrast of colors in the light mode

Slightly increase the size of buttons and inputs on touch devices

See https://www.woltlab.com/community/thread/300621-input-height-mobile-device/

Merge pull request #5583 from WoltLab/url-detection

Improve detection of URL in texts

Limit URL detection to http(s) protokoll

Apply suggestions from code review

Improve detection of URL in texts

Improve the loading experience of the newsfeed in dark mode

The default page of an `<iframe>` is `about:blank` which has a light background
color. In case the newsfeed loads slowly or fails to load entirely, there will
be a big white block in a dark mode ACP which looks odd.

Fix this issue by setting the `<iframe>` to be `visibility: hidden` until the
contents are available. Additionally a spinner is shown while the iframe is
hidden.

Merge pull request #5579 from WoltLab/return-symbol

Use correct character for return key in phrases

Use correct character for return key in phrases

Per https://en.wikipedia.org/wiki/Enter_key#Keyboard_symbols:

> The return key symbol is U+23CE ⏎ RETURN SYMBOL, an arrow pointing down and
> leftward;

Add dummy definition file for `ckeditor5-bundle` module

This prevents TypeScript from complaining about a non-existent module when
dynamically loading it for the definitions of the globals.

Change icon for the notification to solid version

This will bring the icon more in line with the other icons in the user panel.

Update the default colors

Improve the color contrast of badges

Improve the color contrast in the light mode

Remove CKEditor components from the tiny bundle

Defer loading of the CKEditor bundle until it is first requested

Restructure the toolbar a bit

Force the button separators to cover the entire height

Improve the appearance of split buttons

Add PHP 8.3 to the php.yml workflow

Add some margin around spoilers

Increase the top gap of quotes to separate them from preceding content

Merge pull request #5574 from WoltLab/language-selector-icons

Add icons to the mobile version of the language selector

Fix javascript error when you click on the language selector in the mobile menu

Add icon to the mobile version of the language selector

Use `overflow-x: clip` instead of `hidden`

The `hidden` property breaks `position: sticky`, but the `clip` value restores the behavior.

This value is not supported in Safari <16 in which case it falls back to the previous behavior of permitting horizontal overflow. This is an acceptable behavior.

Fix icon colors in message header / thread list

The link color has affected the icons, which was not intended in these locations.

Preventing the body scrolling no longer requires a hack on non-iOS devices

Fix the position of non-fixed elements

See https://www.woltlab.com/community/thread/300599-reaktionsauswahl-scrollt-nach-unten/

Release 6.0.0 Alpha 5