summaryrefslogtreecommitdiffstats
path: root/web_src
diff options
context:
space:
mode:
authorwxiaoguang <wxiaoguang@gmail.com>2023-06-12 23:44:53 +0800
committerGitHub <noreply@github.com>2023-06-12 15:44:53 +0000
commitd8e45608d60c4d00859f0531addf5b593b0dfd2b (patch)
treedbb8ffe63753c220f41a18cdb4c3b3c9f6b1c94d /web_src
parenteaf551b3c19a37e550844828647960f9a2906ab6 (diff)
downloadgitea-d8e45608d60c4d00859f0531addf5b593b0dfd2b.tar.gz
gitea-d8e45608d60c4d00859f0531addf5b593b0dfd2b.zip
Remove hacky patch for "safari emoji glitch fix" (#25208)
According to my test, the UI (emoji) is fine in Safari And actually the code is just dead code, because the "resize" event is never fired on page loading. So for most cases users just view the pages without this hacky patch, nobody ever complains.
Diffstat (limited to 'web_src')
-rw-r--r--web_src/css/base.css13
-rw-r--r--web_src/js/features/common-global.js14
-rw-r--r--web_src/js/utils.js13
3 files changed, 0 insertions, 40 deletions
diff --git a/web_src/css/base.css b/web_src/css/base.css
index f9b2ca99ba..38faf2287c 100644
--- a/web_src/css/base.css
+++ b/web_src/css/base.css
@@ -2199,19 +2199,6 @@ table th[data-sortt-desc] .svg {
vertical-align: -0.075em;
}
-@supports (-webkit-hyphens:none) {
- body:not(.safari-above125) .emoji,
- body:not(.safari-above125) .reaction {
- font-size: inherit;
- vertical-align: inherit;
- }
- body:not(.safari-above125) .emoji img,
- body:not(.safari-above125) .reaction img {
- font-size: 1.25em;
- vertical-align: -0.225em !important;
- }
-}
-
.emoji img,
.reaction img {
border-width: 0 !important;
diff --git a/web_src/js/features/common-global.js b/web_src/js/features/common-global.js
index cb50ebcae0..46a80beb51 100644
--- a/web_src/js/features/common-global.js
+++ b/web_src/js/features/common-global.js
@@ -1,6 +1,5 @@
import $ from 'jquery';
import 'jquery.are-you-sure';
-import {mqBinarySearch} from '../utils.js';
import {createDropzone} from './dropzone.js';
import {initCompColorPicker} from './comp/ColorPicker.js';
import {showGlobalErrorMessage} from '../bootstrap.js';
@@ -62,19 +61,6 @@ export function initGlobalButtonClickOnEnter() {
}
export function initGlobalCommon() {
- // Undo Safari emoji glitch fix at high enough zoom levels
- if (navigator.userAgent.match('Safari')) {
- $(window).on('resize', () => {
- const px = mqBinarySearch('width', 0, 4096, 1, 'px');
- const em = mqBinarySearch('width', 0, 1024, 0.01, 'em');
- if (em * 16 * 1.25 - px <= -1) {
- $('body').addClass('safari-above125');
- } else {
- $('body').removeClass('safari-above125');
- }
- });
- }
-
// Semantic UI modules.
const $uiDropdowns = $('.ui.dropdown');
diff --git a/web_src/js/utils.js b/web_src/js/utils.js
index 4f9ad452f6..4655b8eacc 100644
--- a/web_src/js/utils.js
+++ b/web_src/js/utils.js
@@ -37,19 +37,6 @@ export function stripTags(text) {
return text.replace(/<[^>]*>?/g, '');
}
-// searches the inclusive range [minValue, maxValue].
-// credits: https://matthiasott.com/notes/write-your-media-queries-in-pixels-not-ems
-export function mqBinarySearch(feature, minValue, maxValue, step, unit) {
- if (maxValue - minValue < step) {
- return minValue;
- }
- const mid = Math.ceil((minValue + maxValue) / 2 / step) * step;
- if (matchMedia(`screen and (min-${feature}:${mid}${unit})`).matches) {
- return mqBinarySearch(feature, mid, maxValue, step, unit); // feature is >= mid
- }
- return mqBinarySearch(feature, minValue, mid - step, step, unit); // feature is < mid
-}
-
export function parseIssueHref(href) {
const path = (href || '').replace(/[#?].*$/, '');
const [_, owner, repo, type, index] = /([^/]+)\/([^/]+)\/(issues|pulls)\/([0-9]+)/.exec(path) || [];