Fdans has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/390551 )
Change subject: Remove download of font from google endpoint ...................................................................... Remove download of font from google endpoint This change splits off the css outside of the bundle for development. Spliting CSS and JS makes a lot easier the configuration of the fonts. There is a new file (lato.css) that stores the references to the fonts we are using so they are managed by our code rather than referencing google. Font files are embeded or downloaded depending on size. Bug: T178317 Change-Id: Ifdec8c07f3eb578112b779105f51d0e2781d5965 --- M package.json A src/lato/1KWMyx7m-L0fkQGwYhWwuuvvDin1pK8aKteLpeZ5c0A.woff2 A src/lato/8qcEw_nrk_5HEcCpYdJu8BTbgVql8nDJpwnrE27mub0.woff2 A src/lato/AcvTq8Q0lyKKNxRlL28Rn4X0hVgzZQUfRDuZrPvH3D8.woff2 A src/lato/HkF_qI1x_noxlxhrhMQYEJBw1xU1rKptJj_0jans920.woff2 A src/lato/MDadn8DQ_3oT6kvnUq_2r_esZW2xOQ-xsNqO47m55DA.woff2 A src/lato/MgNNr5y1C_tIEuLEmicLmwLUuEpTyoUstqEm5AMlJo4.woff2 A src/lato/cT2GN3KRBUX69GVJ2b2hxn-_kf6ByYO6CLYdB4HQE-Y.woff2 A src/lato/lato.css A src/lato/rZPI2gHXi8zxUjnybc2ZQFKPGs1ZzpMvnHX-7fPOuAc.woff2 M src/main.js M webpack/base.config.js M webpack/prod.config.js 13 files changed, 95 insertions(+), 5 deletions(-) Approvals: Fdans: Verified; Looks good to me, approved diff --git a/package.json b/package.json index 8b3a0ce..da04355 100644 --- a/package.json +++ b/package.json @@ -44,6 +44,7 @@ "karma-phantomjs-launcher": "^1.0.4", "karma-webpack": "^2.0.3", "style-loader": "^0.18.1", + "string-replace-loader":"^1.3.0", "uglifyjs-webpack-plugin": "^0.4.6", "url-loader": "^0.5.8", "vue-loader": "^12.0.4", diff --git a/src/lato/1KWMyx7m-L0fkQGwYhWwuuvvDin1pK8aKteLpeZ5c0A.woff2 b/src/lato/1KWMyx7m-L0fkQGwYhWwuuvvDin1pK8aKteLpeZ5c0A.woff2 new file mode 100644 index 0000000..3c03843 --- /dev/null +++ b/src/lato/1KWMyx7m-L0fkQGwYhWwuuvvDin1pK8aKteLpeZ5c0A.woff2 Binary files differ diff --git a/src/lato/8qcEw_nrk_5HEcCpYdJu8BTbgVql8nDJpwnrE27mub0.woff2 b/src/lato/8qcEw_nrk_5HEcCpYdJu8BTbgVql8nDJpwnrE27mub0.woff2 new file mode 100644 index 0000000..c5f3b20 --- /dev/null +++ b/src/lato/8qcEw_nrk_5HEcCpYdJu8BTbgVql8nDJpwnrE27mub0.woff2 Binary files differ diff --git a/src/lato/AcvTq8Q0lyKKNxRlL28Rn4X0hVgzZQUfRDuZrPvH3D8.woff2 b/src/lato/AcvTq8Q0lyKKNxRlL28Rn4X0hVgzZQUfRDuZrPvH3D8.woff2 new file mode 100644 index 0000000..a7839e1 --- /dev/null +++ b/src/lato/AcvTq8Q0lyKKNxRlL28Rn4X0hVgzZQUfRDuZrPvH3D8.woff2 Binary files differ diff --git a/src/lato/HkF_qI1x_noxlxhrhMQYEJBw1xU1rKptJj_0jans920.woff2 b/src/lato/HkF_qI1x_noxlxhrhMQYEJBw1xU1rKptJj_0jans920.woff2 new file mode 100644 index 0000000..623e9f9 --- /dev/null +++ b/src/lato/HkF_qI1x_noxlxhrhMQYEJBw1xU1rKptJj_0jans920.woff2 Binary files differ diff --git a/src/lato/MDadn8DQ_3oT6kvnUq_2r_esZW2xOQ-xsNqO47m55DA.woff2 b/src/lato/MDadn8DQ_3oT6kvnUq_2r_esZW2xOQ-xsNqO47m55DA.woff2 new file mode 100644 index 0000000..597b855 --- /dev/null +++ b/src/lato/MDadn8DQ_3oT6kvnUq_2r_esZW2xOQ-xsNqO47m55DA.woff2 Binary files differ diff --git a/src/lato/MgNNr5y1C_tIEuLEmicLmwLUuEpTyoUstqEm5AMlJo4.woff2 b/src/lato/MgNNr5y1C_tIEuLEmicLmwLUuEpTyoUstqEm5AMlJo4.woff2 new file mode 100644 index 0000000..e9d0b3f --- /dev/null +++ b/src/lato/MgNNr5y1C_tIEuLEmicLmwLUuEpTyoUstqEm5AMlJo4.woff2 Binary files differ diff --git a/src/lato/cT2GN3KRBUX69GVJ2b2hxn-_kf6ByYO6CLYdB4HQE-Y.woff2 b/src/lato/cT2GN3KRBUX69GVJ2b2hxn-_kf6ByYO6CLYdB4HQE-Y.woff2 new file mode 100644 index 0000000..e3537ee --- /dev/null +++ b/src/lato/cT2GN3KRBUX69GVJ2b2hxn-_kf6ByYO6CLYdB4HQE-Y.woff2 Binary files differ diff --git a/src/lato/lato.css b/src/lato/lato.css new file mode 100644 index 0000000..9c366d3 --- /dev/null +++ b/src/lato/lato.css @@ -0,0 +1,64 @@ +/* latin-ext */ +@font-face { + font-family: 'Lato'; + font-style: italic; + font-weight: 400; + src: local('Lato Italic'), local('Lato-Italic'), url(./cT2GN3KRBUX69GVJ2b2hxn-_kf6ByYO6CLYdB4HQE-Y.woff2) format('woff2'); + unicode-range: U+0100-024F, U+1E00-1EFF, U+20A0-20AB, U+20AD-20CF, U+2C60-2C7F, U+A720-A7FF; +} +/* latin */ +@font-face { + font-family: 'Lato'; + font-style: italic; + font-weight: 400; + src: local('Lato Italic'), local('Lato-Italic'), url(./1KWMyx7m-L0fkQGwYhWwuuvvDin1pK8aKteLpeZ5c0A.woff2) format('woff2'); + unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215; +} +/* latin-ext */ +@font-face { + font-family: 'Lato'; + font-style: italic; + font-weight: 700; + src: local('Lato Bold Italic'), local('Lato-BoldItalic'), url(./AcvTq8Q0lyKKNxRlL28Rn4X0hVgzZQUfRDuZrPvH3D8.woff2) format('woff2'); + unicode-range: U+0100-024F, U+1E00-1EFF, U+20A0-20AB, U+20AD-20CF, U+2C60-2C7F, U+A720-A7FF; +} +/* latin */ +@font-face { + font-family: 'Lato'; + font-style: italic; + font-weight: 700; + src: local('Lato Bold Italic'), local('Lato-BoldItalic'), url(./HkF_qI1x_noxlxhrhMQYEJBw1xU1rKptJj_0jans920.woff2) format('woff2'); + unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215; +} +/* latin-ext */ +@font-face { + font-family: 'Lato'; + font-style: normal; + font-weight: 400; + src: local('Lato Regular'), local('Lato-Regular'), url(./8qcEw_nrk_5HEcCpYdJu8BTbgVql8nDJpwnrE27mub0.woff2) format('woff2'); + unicode-range: U+0100-024F, U+1E00-1EFF, U+20A0-20AB, U+20AD-20CF, U+2C60-2C7F, U+A720-A7FF; +} +/* latin */ +@font-face { + font-family: 'Lato'; + font-style: normal; + font-weight: 400; + src: local('Lato Regular'), local('Lato-Regular'), url(./MDadn8DQ_3oT6kvnUq_2r_esZW2xOQ-xsNqO47m55DA.woff2) format('woff2'); + unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215; +} +/* latin-ext */ +@font-face { + font-family: 'Lato'; + font-style: normal; + font-weight: 700; + src: local('Lato Bold'), local('Lato-Bold'), url(./rZPI2gHXi8zxUjnybc2ZQFKPGs1ZzpMvnHX-7fPOuAc.woff2) format('woff2'); + unicode-range: U+0100-024F, U+1E00-1EFF, U+20A0-20AB, U+20AD-20CF, U+2C60-2C7F, U+A720-A7FF; +} +/* latin */ +@font-face { + font-family: 'Lato'; + font-style: normal; + font-weight: 700; + src: local('Lato Bold'), local('Lato-Bold'), url(./MgNNr5y1C_tIEuLEmicLmwLUuEpTyoUstqEm5AMlJo4.woff2) format('woff2'); + unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215; +} diff --git a/src/lato/rZPI2gHXi8zxUjnybc2ZQFKPGs1ZzpMvnHX-7fPOuAc.woff2 b/src/lato/rZPI2gHXi8zxUjnybc2ZQFKPGs1ZzpMvnHX-7fPOuAc.woff2 new file mode 100644 index 0000000..4f778f9 --- /dev/null +++ b/src/lato/rZPI2gHXi8zxUjnybc2ZQFKPGs1ZzpMvnHX-7fPOuAc.woff2 Binary files differ diff --git a/src/main.js b/src/main.js index 11ff33f..2a8c873 100644 --- a/src/main.js +++ b/src/main.js @@ -4,6 +4,7 @@ import Vue from 'vue'; import App from './App'; import '../semantic/dist/semantic.css'; +import './lato/lato.css'; import store from './store'; import router from './router'; diff --git a/webpack/base.config.js b/webpack/base.config.js index 44e621b..c4a6952 100644 --- a/webpack/base.config.js +++ b/webpack/base.config.js @@ -3,6 +3,9 @@ const utils = require('./utils'); const webpack = require('webpack'); const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin; +const ExtractTextPlugin = require("extract-text-webpack-plugin"); + + module.exports = { entry: './src/main.js', @@ -34,13 +37,35 @@ name: utils.assetsPath('fonts/[name].[hash:7].[ext]') } }, + { + test: /\.css$/, + use: ExtractTextPlugin.extract({ + fallback: "style-loader", + use: "css-loader" + }) + }, + // Replacing google imports in semantic with ...ahem nothing so they do not run + // the css needed is included in lato.css { - test: /\.css$/, - use: [ 'style-loader', 'css-loader' ] + test: /semantic\.css$/, + use : + [ + {loader: 'string-replace-loader', + query: { + search: 'https://fonts.googleapis.com/css?family=Lato:400,700,400italic,700italic&subset=latin', + replace: '', + strict: true + } + } + ] } - ] + ] }, + + plugins: [ + // Extract imported CSS into own file + new ExtractTextPlugin('[name].bundle.[chunkhash].css'), new HtmlWebpackPlugin({ filename: 'index.html', template: 'src/index.html', @@ -60,6 +85,7 @@ ), // uncomment to see bundle size composition when running webpack // new BundleAnalyzerPlugin() + ], devServer: { contentBase: utils.resolve('dist'), diff --git a/webpack/prod.config.js b/webpack/prod.config.js index 6626bf8..6a8c5fb 100644 --- a/webpack/prod.config.js +++ b/webpack/prod.config.js @@ -23,8 +23,6 @@ }, plugins: [ - // Extract imported CSS into own file - new ExtractTextPlugin('[name].bundle.[chunkhash].css'), // Minify JS new UglifyJSPlugin(), // Minify CSS -- To view, visit https://gerrit.wikimedia.org/r/390551 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdec8c07f3eb578112b779105f51d0e2781d5965 Gerrit-PatchSet: 2 Gerrit-Project: analytics/wikistats2 Gerrit-Branch: master Gerrit-Owner: Nuria <nu...@wikimedia.org> Gerrit-Reviewer: Fdans <fd...@wikimedia.org> Gerrit-Reviewer: Milimetric <dandree...@wikimedia.org> Gerrit-Reviewer: Nuria <nu...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits