From 960d0d1242880222b18ed17487003a499caf5a64 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Wed, 17 Aug 2016 13:36:17 +0200 Subject: [PATCH 1/6] Filter can't be named invert in FF --- settings/js/apps.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/settings/js/apps.js b/settings/js/apps.js index f8ad9c7918..9a68ebf90a 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -237,8 +237,8 @@ OC.Settings.Apps = OC.Settings.Apps || { if (appfromstore) { img += ''; } else { - img += ''; - img += ''; + img += ''; + img += ''; } return img; }, From 1444c1d3456b1f9b7179d1be2c7496a84bbc8fdd Mon Sep 17 00:00:00 2001 From: Morris Jobke Date: Wed, 17 Aug 2016 14:45:08 +0200 Subject: [PATCH 2/6] Introduce weird hack for Firefox * everything is 4 times as big (doubled width and doubled height) --- settings/js/apps.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/settings/js/apps.js b/settings/js/apps.js index 9a68ebf90a..5a94a78381 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -234,6 +234,11 @@ OC.Settings.Apps = OC.Settings.Apps || { imageUrl : function (url, appfromstore) { var img = ''; + // weird hack because in Firefox everything is 4 times as big (doubled in width and doubled in height) + // TODO: replace this with a proper solution + if($.browser.mozilla) { + img = ''; + } if (appfromstore) { img += ''; } else { From 314b4005bb2c9d2f82991d2d476f0fc6b899df4b Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Wed, 17 Aug 2016 15:16:48 +0200 Subject: [PATCH 3/6] Remove hack --- settings/js/apps.js | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/settings/js/apps.js b/settings/js/apps.js index 5a94a78381..2e2bf6830c 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -234,11 +234,7 @@ OC.Settings.Apps = OC.Settings.Apps || { imageUrl : function (url, appfromstore) { var img = ''; - // weird hack because in Firefox everything is 4 times as big (doubled in width and doubled in height) - // TODO: replace this with a proper solution - if($.browser.mozilla) { - img = ''; - } + if (appfromstore) { img += ''; } else { From ba4633b165af3bc2a4c732d628fbcf5a3263b986 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Wed, 17 Aug 2016 15:23:51 +0200 Subject: [PATCH 4/6] Add viewBox to app icons If we don't add a viewbox a lot of browsers will just say NOPE!!! when we ask them to scale. --- apps/comments/img/comments.svg | 2 +- apps/files_external/img/app.svg | 2 +- apps/files_sharing/img/app.svg | 1 + apps/files_trashbin/img/app.svg | 1 + apps/files_versions/img/app.svg | 1 + apps/systemtags/img/app.svg | 2 +- apps/testing/img/app.svg | 2 +- apps/user_ldap/img/app.svg | 2 +- core/img/default-app-icon.svg | 2 +- 9 files changed, 9 insertions(+), 6 deletions(-) diff --git a/apps/comments/img/comments.svg b/apps/comments/img/comments.svg index 5cdc84a25d..1eabeec334 100644 --- a/apps/comments/img/comments.svg +++ b/apps/comments/img/comments.svg @@ -1,4 +1,4 @@ - + diff --git a/apps/files_external/img/app.svg b/apps/files_external/img/app.svg index 6cafe174f6..f1fd591ca7 100644 --- a/apps/files_external/img/app.svg +++ b/apps/files_external/img/app.svg @@ -1,4 +1,4 @@ - + diff --git a/apps/files_sharing/img/app.svg b/apps/files_sharing/img/app.svg index d64e44b70b..11a1a37a29 100644 --- a/apps/files_sharing/img/app.svg +++ b/apps/files_sharing/img/app.svg @@ -9,6 +9,7 @@ xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" height="16" width="16" + viewBox="0 0 16 16" version="1.1" id="svg2" inkscape:version="0.48.5 r10040" diff --git a/apps/files_trashbin/img/app.svg b/apps/files_trashbin/img/app.svg index b749f9879e..cd169009e7 100644 --- a/apps/files_trashbin/img/app.svg +++ b/apps/files_trashbin/img/app.svg @@ -9,6 +9,7 @@ xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" height="16" width="16" + viewBox="0 0 16 16" version="1.1" id="svg2" inkscape:version="0.48.5 r10040" diff --git a/apps/files_versions/img/app.svg b/apps/files_versions/img/app.svg index 862b0a6885..831ef7fedd 100644 --- a/apps/files_versions/img/app.svg +++ b/apps/files_versions/img/app.svg @@ -9,6 +9,7 @@ xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" height="16" width="16" + viewBox="0 0 16 16" version="1.0" id="svg2" inkscape:version="0.48.5 r10040" diff --git a/apps/systemtags/img/app.svg b/apps/systemtags/img/app.svg index 8bfd246078..4f201c6c76 100644 --- a/apps/systemtags/img/app.svg +++ b/apps/systemtags/img/app.svg @@ -1,4 +1,4 @@ - + diff --git a/apps/testing/img/app.svg b/apps/testing/img/app.svg index b6ae35211a..ad789050c9 100644 --- a/apps/testing/img/app.svg +++ b/apps/testing/img/app.svg @@ -1,4 +1,4 @@ - + diff --git a/apps/user_ldap/img/app.svg b/apps/user_ldap/img/app.svg index 0ce7ed867b..5642ad9c9e 100644 --- a/apps/user_ldap/img/app.svg +++ b/apps/user_ldap/img/app.svg @@ -1,5 +1,5 @@ - + diff --git a/core/img/default-app-icon.svg b/core/img/default-app-icon.svg index b6ae35211a..ad789050c9 100644 --- a/core/img/default-app-icon.svg +++ b/core/img/default-app-icon.svg @@ -1,4 +1,4 @@ - + From bded787d0c4a4c09d6134dec1f4f05751e1d1ec1 Mon Sep 17 00:00:00 2001 From: Morris Jobke Date: Wed, 17 Aug 2016 15:59:30 +0200 Subject: [PATCH 5/6] Empty tags are not allowed for image and feColorMatrix in IE11 and below --- core/templates/layout.user.php | 8 ++++---- settings/js/apps.js | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php index be5c769ab7..2c0d3f0529 100644 --- a/core/templates/layout.user.php +++ b/core/templates/layout.user.php @@ -114,8 +114,8 @@ class="active"> - - + + @@ -132,8 +132,8 @@ class="active"> - - + + diff --git a/settings/js/apps.js b/settings/js/apps.js index 2e2bf6830c..89b446ed3b 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -238,8 +238,8 @@ OC.Settings.Apps = OC.Settings.Apps || { if (appfromstore) { img += ''; } else { - img += ''; - img += ''; + img += ''; + img += ''; } return img; }, From 28c192c9f5a5279482eaabbcaf2d24e78379c5bd Mon Sep 17 00:00:00 2001 From: Morris Jobke Date: Wed, 17 Aug 2016 16:04:53 +0200 Subject: [PATCH 6/6] Fix null reference in IE --- settings/js/apps.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/settings/js/apps.js b/settings/js/apps.js index 89b446ed3b..ac259dd798 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -196,13 +196,13 @@ OC.Settings.Apps = OC.Settings.Apps || { if (app.preview && !OC.Util.isIE()) { var currentImage = new Image(); currentImage.src = app.preview; - } - currentImage.onload = function() { - page.find('.app-image') - .append(OC.Settings.Apps.imageUrl(app.preview, app.detailpage)) - .fadeIn(); - }; + currentImage.onload = function() { + page.find('.app-image') + .append(OC.Settings.Apps.imageUrl(app.preview, app.detailpage)) + .fadeIn(); + }; + } // set group select properly if(OC.Settings.Apps.isType(app, 'filesystem') || OC.Settings.Apps.isType(app, 'prelogin') ||