Browse Source

Move files_sharing additionalscripts over to webpack

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Roeland Jago Douma 5 years ago
parent
commit
0a4569dff7

+ 2 - 0
.gitattributes

@@ -3,6 +3,8 @@
 
 /apps/accessibility/js/accessibility.js binary
 /apps/accessibility/js/accessibility.js.map binary
+/apps/files_sharing/js/additionalScripts.js binary
+/apps/files_sharing/js/additionalScripts.js.map binary
 /apps/files_versions/js/files_versions.js binary
 /apps/files_versions/js/files_versions.js.map binary
 /apps/oauth2/js/oauth2.js binary

+ 2 - 0
apps/files_sharing/.l10nignore

@@ -0,0 +1,2 @@
+#webpack bundled files
+js/additionalScripts.js

+ 0 - 1
apps/files_sharing/appinfo/app.php

@@ -42,7 +42,6 @@ $eventDispatcher = \OC::$server->getEventDispatcher();
 $eventDispatcher->addListener(
 	'OCA\Files::loadAdditionalScripts',
 	function() {
-		\OCP\Util::addStyle('files_sharing', 'mergedAdditionalStyles');
 		\OCP\Util::addScript('files_sharing', 'additionalScripts');
 	}
 );

+ 0 - 2
apps/files_sharing/css/mergedAdditionalStyles.scss

@@ -1,2 +0,0 @@
-@import 'sharetabview.scss';
-@import 'sharebreadcrumb.scss';

File diff suppressed because it is too large
+ 0 - 0
apps/files_sharing/js/additionalScripts.js


File diff suppressed because it is too large
+ 0 - 0
apps/files_sharing/js/additionalScripts.js.map


+ 0 - 5
apps/files_sharing/js/additionalScripts.json

@@ -1,5 +0,0 @@
-[
-	"share.js",
-	"sharetabview.js",
-	"sharebreadcrumbview.js"
-]

+ 6 - 0
apps/files_sharing/src/additionalScripts.js

@@ -0,0 +1,6 @@
+import './share'
+import './sharetabview'
+import './sharebreadcrumbview'
+
+import './style/sharetabview.scss'
+import './style/sharebreadcrumb.scss'

+ 0 - 0
apps/files_sharing/js/share.js → apps/files_sharing/src/share.js


+ 0 - 0
apps/files_sharing/js/sharebreadcrumbview.js → apps/files_sharing/src/sharebreadcrumbview.js


+ 0 - 0
apps/files_sharing/js/sharetabview.js → apps/files_sharing/src/sharetabview.js


+ 0 - 0
apps/files_sharing/css/sharebreadcrumb.scss → apps/files_sharing/src/style/sharebreadcrumb.scss


+ 0 - 0
apps/files_sharing/css/sharetabview.scss → apps/files_sharing/src/style/sharetabview.scss


+ 37 - 0
apps/files_sharing/webpack.js

@@ -0,0 +1,37 @@
+const path = require('path');
+
+module.exports = {
+	entry: path.join(__dirname, 'src', 'additionalScripts.js'),
+	output: {
+		path: path.resolve(__dirname, './js'),
+		publicPath: '/js/',
+		filename: 'additionalScripts.js'
+	},
+	module: {
+		rules: [
+			{
+				test: /\.css$/,
+				use: ['style-loader', 'css-loader']
+			},
+			{
+				test: /\.scss$/,
+				use: ['style-loader', 'css-loader', 'sass-loader']
+			},
+			{
+				test: /\.js$/,
+				loader: 'babel-loader',
+				exclude: /node_modules/
+			},
+			{
+				test: /\.(png|jpg|gif|svg)$/,
+				loader: 'file-loader',
+				options: {
+					name: '[name].[ext]?[hash]'
+				}
+			}
+		]
+	},
+	resolve: {
+		extensions: ['*', '.js']
+	}
+};

+ 1 - 3
tests/karma.config.js

@@ -54,10 +54,8 @@ module.exports = function(config) {
 					// up with the global namespace/classes/state
 					'apps/files_sharing/js/app.js',
 					'apps/files_sharing/js/sharedfilelist.js',
-					'apps/files_sharing/js/share.js',
-					'apps/files_sharing/js/sharebreadcrumbview.js',
+					'apps/files_sharing/js/additionalScripts.js',
 					'apps/files_sharing/js/public.js',
-					'apps/files_sharing/js/sharetabview.js',
 					'apps/files_sharing/js/files_drop.js',
 					'apps/files_sharing/js/templates.js',
 				],

+ 2 - 0
webpack.common.js

@@ -1,4 +1,5 @@
 const core = require('./core/webpack');
+const files_sharing = require('./apps/files_sharing/webpack')
 const files_trashbin = require('./apps/files_trashbin/webpack')
 const files_versions = require('./apps/files_versions/webpack');
 const oauth2 = require('./apps/oauth2/webpack')
@@ -6,6 +7,7 @@ const systemtags = require('./apps/systemtags/webpack')
 
 module.exports = [].concat(
 	core,
+	files_sharing,
 	files_trashbin,
 	files_versions,
 	oauth2,

Some files were not shown because too many files changed in this diff