Removed an outdated webkit work-around
authorAlexander Ebert <ebert@woltlab.com>
Sat, 28 May 2022 13:05:48 +0000 (15:05 +0200)
committerAlexander Ebert <ebert@woltlab.com>
Sat, 28 May 2022 13:05:48 +0000 (15:05 +0200)
This was required in <= 5.4 to fix an issue caused by the `transform: translateX(-50%)` offset used in these versions. 5.5 uses a pixel-perfect positioning that no longer relies on this hack.

ts/WoltLabSuite/Core/Ui/Dialog.ts
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Dialog.js
wcfsetup/install/files/style/ui/dialog.scss

index 4ee1a088e6c9faf8c93e4775dea462fd8465a6c2..cebda856bfa343dd5fa25fef16edf930102e1996 100644 (file)
@@ -630,19 +630,6 @@ const UiDialog = {
     const maximumHeight = window.innerHeight * (_dialogFullHeight ? 1 : 0.8) - unavailableHeight;
     contentContainer.style.setProperty("max-height", `${~~maximumHeight}px`, "");
 
-    // Chrome and Safari use heavy anti-aliasing when the dialog's width
-    // cannot be evenly divided, causing the whole text to become blurry
-    if (Environment.browser() === "chrome" || Environment.browser() === "safari") {
-      // The new Microsoft Edge is detected as "chrome", because effectively we're detecting
-      // Chromium rather than Chrome specifically. The workaround for fractional pixels does
-      // not work well in Edge, there seems to be a different logic for fractional positions,
-      // causing the text to be blurry.
-      //
-      // We can use `backface-visibility: hidden` to prevent the anti aliasing artifacts in
-      // WebKit/Blink, which will also prevent some weird font rendering issues when resizing.
-      contentContainer.classList.add("jsWebKitFractionalPixelFix");
-    }
-
     const callbackObject = _dialogToObject.get(id);
     //noinspection JSUnresolvedVariable
     if (callbackObject !== undefined && typeof callbackObject._dialogSubmit === "function") {
index 33c240dbcdd50ed7d57f912d3094a7ea6d68e3a3..b7600c94bbceb1113260e8baff9d29947ce42b7a 100644 (file)
@@ -520,18 +520,6 @@ define(["require", "exports", "tslib", "../Core", "../Dom/Change/Listener", "./S
             unavailableHeight += Util_1.default.outerHeight(data.header);
             const maximumHeight = window.innerHeight * (_dialogFullHeight ? 1 : 0.8) - unavailableHeight;
             contentContainer.style.setProperty("max-height", `${~~maximumHeight}px`, "");
-            // Chrome and Safari use heavy anti-aliasing when the dialog's width
-            // cannot be evenly divided, causing the whole text to become blurry
-            if (Environment.browser() === "chrome" || Environment.browser() === "safari") {
-                // The new Microsoft Edge is detected as "chrome", because effectively we're detecting
-                // Chromium rather than Chrome specifically. The workaround for fractional pixels does
-                // not work well in Edge, there seems to be a different logic for fractional positions,
-                // causing the text to be blurry.
-                //
-                // We can use `backface-visibility: hidden` to prevent the anti aliasing artifacts in
-                // WebKit/Blink, which will also prevent some weird font rendering issues when resizing.
-                contentContainer.classList.add("jsWebKitFractionalPixelFix");
-            }
             const callbackObject = _dialogToObject.get(id);
             //noinspection JSUnresolvedVariable
             if (callbackObject !== undefined && typeof callbackObject._dialogSubmit === "function") {
index 648a923ea816ffd3ebcea17f67f476c9815e2af9..1636bb84ef90ee424ee8d4942b14a6b3b13a8be3 100644 (file)
                                }
                        }
                }
-
-               /* Chrome and Safari use heavy anti-aliasing when the dialog's width
-                  cannot be evenly divided, causing the whole text to become blurry */
-               &.jsWebKitFractionalPixelFix {
-                       backface-visibility: hidden;
-               }
        }
 }