multi line message when "from" is set

Author: sokra

lib/ModuleBuildError.js

@@ -12,9 +12,10 @@ class ModuleBuildError extends WebpackError {
 		let message = "Module build failed";
 		let details = undefined;
 		if (from) {
-			message += ` (from ${from})`;
+			message += ` (from ${from}):\n`;
+		} else {
+			message += ": ";
 		}
-		message += ": ";
 		if (err !== null && typeof err === "object") {
 			if (typeof err.stack === "string" && err.stack) {
 				const stack = cutOffLoaderExecution(err.stack);

lib/ModuleError.js

@@ -11,12 +11,14 @@ class ModuleError extends WebpackError {
 	constructor(module, err, { from = null } = {}) {
 		let message = "Module Error";
 		if (from) {
-			message += ` (from ${from})`;
+			message += ` (from ${from}):\n`;
+		} else {
+			message += ": ";
 		}
 		if (err && typeof err === "object" && err.message) {
-			message += `:\n${err.message}`;
+			message += err.message;
 		} else if (err) {
-			message += `:\n${err}`;
+			message += err;
 		}
 		super(message);
 		this.name = "ModuleError";

lib/ModuleWarning.js

@@ -11,12 +11,14 @@ class ModuleWarning extends WebpackError {
 	constructor(module, warning, { from = null } = {}) {
 		let message = "Module Warning";
 		if (from) {
-			message += ` (from ${from})`;
+			message += ` (from ${from}):\n`;
+		} else {
+			message += ": ";
 		}
 		if (warning && typeof warning === "object" && warning.message) {
-			message += `:\n${warning.message}`;
+			message += warning.message;
 		} else if (warning) {
-			message += `:\n${warning}`;
+			message += warning;
 		}
 		super(message);
 		this.name = "ModuleWarning";