From dccb3ab5b2c8f8029823d20ee3e6dd14a5098d87 Mon Sep 17 00:00:00 2001 From: Christoph Wurst Date: Mon, 8 Oct 2018 16:33:41 +0200 Subject: [PATCH] Remove deprecated SVG helpers for old IEs Signed-off-by: Christoph Wurst --- apps/files/js/breadcrumb.js | 5 ----- apps/files_sharing/js/public.js | 2 +- core/js/js.js | 27 --------------------------- 3 files changed, 1 insertion(+), 33 deletions(-) diff --git a/apps/files/js/breadcrumb.js b/apps/files/js/breadcrumb.js index 319425b67b..e1193e79b2 100644 --- a/apps/files/js/breadcrumb.js +++ b/apps/files/js/breadcrumb.js @@ -178,11 +178,6 @@ $crumb.append(view.$el); }, this); - // in case svg is not supported by the browser we need to execute the fallback mechanism - if (!OC.Util.hasSVGSupport()) { - OC.Util.replaceSVG(this.$el); - } - // setup drag and drop if (this.onDrop) { this.$el.find('.crumb:not(:last-child):not(.crumbmenu), .crumblist:not(:last-child)').droppable({ diff --git a/apps/files_sharing/js/public.js b/apps/files_sharing/js/public.js index 64ffe8bbf2..3328b1663d 100644 --- a/apps/files_sharing/js/public.js +++ b/apps/files_sharing/js/public.js @@ -168,7 +168,7 @@ OCA.Sharing.PublicApp = { img.attr('src', OC.generateUrl('/apps/files_sharing/publicpreview/' + token + '?' + OC.buildQueryString(params))); imgcontainer.appendTo('#imgframe'); } else if (mimetype.substr(0, mimetype.indexOf('/')) !== 'video') { - img.attr('src', OC.Util.replaceSVGIcon(mimetypeIcon)); + img.attr('src', mimetypeIcon); img.attr('width', 128); imgcontainer.appendTo('#imgframe'); } diff --git a/core/js/js.js b/core/js/js.js index 8e7796143d..8cfb31fcae 100644 --- a/core/js/js.js +++ b/core/js/js.js @@ -1981,33 +1981,6 @@ OC.Util = { } return moment(timestamp).fromNow(); }, - /** - * Returns whether the browser supports SVG - * @deprecated SVG is always supported (since 9.0) - * @return {boolean} true if the browser supports SVG, false otherwise - */ - hasSVGSupport: function(){ - return true; - }, - /** - * If SVG is not supported, replaces the given icon's extension - * from ".svg" to ".png". - * If SVG is supported, return the image path as is. - * @param {string} file image path with svg extension - * @deprecated SVG is always supported (since 9.0) - * @return {string} fixed image path with png extension if SVG is not supported - */ - replaceSVGIcon: function(file) { - return file; - }, - /** - * Replace SVG images in all elements that have the "svg" class set - * with PNG images. - * - * @param $el root element from which to search, defaults to $('body') - * @deprecated SVG is always supported (since 9.0) - */ - replaceSVG: function($el) {}, /** * Fix image scaling for IE8, since background-size is not supported.