From 3a2ab5a764b5ea0f06b86209837440f2ffdeb24e Mon Sep 17 00:00:00 2001 From: Matthew Irish Date: Thu, 7 Jun 2018 15:08:53 -0500 Subject: [PATCH] fix inlined SVG fill for Firefox (#4722) * fix inlined SVG fill for Firefox * rename bulma in-repo-addon to css * add sass-svg-uri * get rid of bespoke variable and use sass-svg-uri module --- ui/app/styles/core/bulma-radio-checkboxes.scss | 4 +++- ui/lib/{bulma => css}/index.js | 9 ++++++++- ui/lib/{bulma => css}/package.json | 2 +- ui/package.json | 3 ++- ui/yarn.lock | 4 ++++ 5 files changed, 18 insertions(+), 4 deletions(-) rename ui/lib/{bulma => css}/index.js (79%) rename ui/lib/{bulma => css}/package.json (68%) diff --git a/ui/app/styles/core/bulma-radio-checkboxes.scss b/ui/app/styles/core/bulma-radio-checkboxes.scss index 8d7f944cb6..1f9f2d5f4a 100644 --- a/ui/app/styles/core/bulma-radio-checkboxes.scss +++ b/ui/app/styles/core/bulma-radio-checkboxes.scss @@ -1,3 +1,5 @@ +@import "../sass-svg-uri/svg-uri"; + .b-checkbox input[type="checkbox"]:checked + label::before { border-color: $blue; } @@ -6,7 +8,7 @@ .b-checkbox input[type="radio"]:checked + label::after { font-family: $family-monospace; /*checkmark from ionicons*/ - content: url('data:image/svg+xml; utf8, '); + content: svg-uri(''); } .b-checkbox.no-label input[type="checkbox"] + label { diff --git a/ui/lib/bulma/index.js b/ui/lib/css/index.js similarity index 79% rename from ui/lib/bulma/index.js rename to ui/lib/css/index.js index 075b56d4e9..6323f6033b 100644 --- a/ui/lib/bulma/index.js +++ b/ui/lib/css/index.js @@ -23,6 +23,7 @@ module.exports = { this.bulmaPath = path.dirname(require.resolve('bulma')); this.bulmaSwitchPath = path.dirname(require.resolve('bulma-switch/switch.sass')); this.bulmaCheckPath = path.dirname(require.resolve('cool-checkboxes-for-bulma.io')); + this.sassSVGURIPath = path.dirname(require.resolve('sass-svg-uri')); return app; }, @@ -43,7 +44,13 @@ module.exports = { destDir: 'app/styles/bulma', annotation: 'Funnel (bulma-check)', }); + var sassSVGURI = new Funnel(this.sassSVGURIPath, { + srcDir: '/', + destDir: 'app/styles/sass-svg-uri', + annotation: 'Sass SVG URI', + }); - return mergeTrees([bulmaCheck, bulmaSwitch, bulma], { overwrite: true }); + + return mergeTrees([bulmaCheck, bulmaSwitch, bulma, sassSVGURI], { overwrite: true }); }, }; diff --git a/ui/lib/bulma/package.json b/ui/lib/css/package.json similarity index 68% rename from ui/lib/bulma/package.json rename to ui/lib/css/package.json index 5dc02af6c9..2cc1795e14 100644 --- a/ui/lib/bulma/package.json +++ b/ui/lib/css/package.json @@ -1,5 +1,5 @@ { - "name": "bulma", + "name": "css", "keywords": [ "ember-addon" ] diff --git a/ui/package.json b/ui/package.json index fef9ed28ee..061addb8e3 100644 --- a/ui/package.json +++ b/ui/package.json @@ -96,6 +96,7 @@ "prettier": "^1.5.3", "prettier-eslint-cli": "^4.2.1", "qunit-dom": "^0.6.2", + "sass-svg-uri": "^1.0.0", "string.prototype.startswith": "mathiasbynens/String.prototype.startsWith", "text-encoder-lite": "1.0.0", "yargs-parser": "^10.0.0" @@ -106,7 +107,7 @@ "private": true, "ember-addon": { "paths": [ - "lib/bulma" + "lib/css" ] } } diff --git a/ui/yarn.lock b/ui/yarn.lock index 14b9803425..1dee7aa321 100644 --- a/ui/yarn.lock +++ b/ui/yarn.lock @@ -7508,6 +7508,10 @@ sass-graph@^2.2.4: scss-tokenizer "^0.2.3" yargs "^7.0.0" +sass-svg-uri@^1.0.0: + version "1.0.0" + resolved "https://registry.yarnpkg.com/sass-svg-uri/-/sass-svg-uri-1.0.0.tgz#01e992e4e3ce8d1ec4eac4c8280c0f2ef45c6be8" + sax@>=0.6.0: version "1.2.4" resolved "https://registry.yarnpkg.com/sax/-/sax-1.2.4.tgz#2816234e2378bddc4e5354fab5caa895df7100d9"