boxen - npm Package Compare versions

Comparing version 0.7.0 to 0.8.0

index.js

 'use strict';
-var stringWidth = require('string-width');
-var repeating = require('repeating');
-var chalk = require('chalk');
-var objectAssign = require('object-assign');
-var widestLine = require('widest-line');
-var filledArray = require('filled-array');
-var cliBoxes = require('cli-boxes');
-var camelCase = require('camelcase');
-var ansiAlign = require('ansi-align');
+const stringWidth = require('string-width');
+const chalk = require('chalk');
+const widestLine = require('widest-line');
+const cliBoxes = require('cli-boxes');
+const camelCase = require('camelcase');
+const ansiAlign = require('ansi-align');
 
-var getObject = function (detail) {
-	var obj;
+const getObject = detail => {
+	let obj;
 
@@ -23,3 +20,3 @@ if (typeof detail === 'number') {
 	} else {
-		obj = objectAssign({
+		obj = Object.assign({
 			top: 0,
@@ -35,4 +32,4 @@ right: 0,
 
-var getBorderChars = function (borderStyle) {
-	var sides = [
+const getBorderChars = borderStyle => {
+	const sides = [
 		'topLeft',
@@ -46,3 +43,3 @@ 'topRight',
 
-	var chars;
+	let chars;
 
@@ -53,8 +50,8 @@ if (typeof borderStyle === 'string') {
 		if (!chars) {
-			throw new TypeError('Invalid border style: ' + borderStyle);
+			throw new TypeError(`Invalid border style: ${borderStyle}`);
 		}
 	} else {
-		sides.forEach(function (key) {
+		sides.forEach(key => {
 			if (!borderStyle[key] || typeof borderStyle[key] !== 'string') {
-				throw new TypeError('Invalid border style: ' + key);
+				throw new TypeError(`Invalid border style: ${key}`);
 			}
@@ -69,8 +66,6 @@ });
 
-var getBackgroundColorName = function (x) {
-	return camelCase('bg', x);
-};
+const getBackgroundColorName = x => camelCase('bg', x);
 
-module.exports = function (text, opts) {
-	opts = objectAssign({
+module.exports = (text, opts) => {
+	opts = Object.assign({
 		padding: 0,
@@ -88,67 +83,56 @@ borderStyle: 'single',
 	if (opts.borderColor && !chalk[opts.borderColor]) {
-		throw new Error(opts.borderColor + ' is not a valid borderColor');
+		throw new Error(`${opts.borderColor} is not a valid borderColor`);
 	}
 
 	if (opts.backgroundColor && !chalk[opts.backgroundColor]) {
-		throw new Error(opts.backgroundColor + ' is not a valid backgroundColor');
+		throw new Error(`${opts.backgroundColor} is not a valid backgroundColor`);
 	}
 
-	var chars = getBorderChars(opts.borderStyle);
-	var padding = getObject(opts.padding);
-	var margin = getObject(opts.margin);
+	const chars = getBorderChars(opts.borderStyle);
+	const padding = getObject(opts.padding);
+	const margin = getObject(opts.margin);
 
-	var colorizeBorder = function (x) {
-		var ret = opts.borderColor ? chalk[opts.borderColor](x) : x;
+	const colorizeBorder = x => {
+		const ret = opts.borderColor ? chalk[opts.borderColor](x) : x;
 		return opts.dimBorder ? chalk.dim(ret) : ret;
 	};
 
-	var colorizeContent = function (x) {
-		return opts.backgroundColor ? chalk[opts.backgroundColor](x) : x;
-	};
+	const colorizeContent = x => opts.backgroundColor ? chalk[opts.backgroundColor](x) : x;
 
 	text = ansiAlign(text, {align: opts.align});
 
-	var NL = '\n';
-	var PAD = ' ';
-	var lines = text.split(NL);
+	const NL = '\n';
+	const PAD = ' ';
 
+	let lines = text.split(NL);
+
 	if (padding.top > 0) {
-		lines = filledArray('', padding.top).concat(lines);
+		lines = Array(padding.top).fill('').concat(lines);
 	}
 
 	if (padding.bottom > 0) {
-		lines = lines.concat(filledArray('', padding.bottom));
+		lines = lines.concat(Array(padding.bottom).fill(''));
 	}
 
-	var contentWidth = widestLine(text) + padding.left + padding.right;
-	var paddingLeft = repeating(PAD, padding.left);
+	const contentWidth = widestLine(text) + padding.left + padding.right;
+	const paddingLeft = PAD.repeat(padding.left);
+	const columns = process.stdout.columns;
+	let marginLeft = PAD.repeat(margin.left);
 
-	var marginLeft;
-	var padWidth;
-	var currentColumns = process.stdout.columns;
-	switch (opts.float) {
-		case 'center':
-			padWidth = (currentColumns - contentWidth) / 2;
-			marginLeft = repeating(PAD, padWidth);
-			break;
-
-		case 'right':
-			padWidth = Math.max(currentColumns - contentWidth - 2, 0);
-			padWidth = padWidth < 0 ? 0 : padWidth;
-			marginLeft = repeating(PAD, padWidth);
-			break;
-
-		default:
-			marginLeft = repeating(PAD, margin.left);
-			break;
+	if (opts.float === 'center') {
+		const padWidth = (columns - contentWidth) / 2;
+		marginLeft = PAD.repeat(padWidth);
+	} else if (opts.float === 'right') {
+		let padWidth = Math.max(columns - contentWidth - 2, 0);
+		padWidth = padWidth < 0 ? 0 : padWidth;
+		marginLeft = PAD.repeat(padWidth);
 	}
 
-	var horizontal = repeating(chars.horizontal, contentWidth);
-	var top = colorizeBorder(repeating(NL, margin.top) + marginLeft + chars.topLeft + horizontal + chars.topRight);
-	var bottom = colorizeBorder(marginLeft + chars.bottomLeft + horizontal + chars.bottomRight + repeating(NL, margin.bottom));
-	var side = colorizeBorder(chars.vertical);
+	const horizontal = chars.horizontal.repeat(contentWidth);
+	const top = colorizeBorder(NL.repeat(margin.top) + marginLeft + chars.topLeft + horizontal + chars.topRight);
+	const bottom = colorizeBorder(marginLeft + chars.bottomLeft + horizontal + chars.bottomRight + NL.repeat(margin.bottom));
+	const side = colorizeBorder(chars.vertical);
 
-	var middle = lines.map(function (line) {
-		var paddingRight = repeating(PAD, contentWidth - stringWidth(line) - padding.left);
-
+	const middle = lines.map(line => {
+		const paddingRight = PAD.repeat(contentWidth - stringWidth(line) - padding.left);
 		return marginLeft + side + colorizeContent(paddingLeft + line + paddingRight) + side;
@@ -155,0 +139,0 @@ }).join(NL);

package.json

 {
   "name": "boxen",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "Create boxes in the terminal",
@@ -13,3 +13,3 @@ "license": "MIT",
   "engines": {
-    "node": ">=0.10.0"
+    "node": ">=4"
   },
@@ -36,8 +36,5 @@ "scripts": {
     "ansi-align": "^1.1.0",
-    "camelcase": "^2.1.0",
+    "camelcase": "^3.0.0",
     "chalk": "^1.1.1",
     "cli-boxes": "^1.0.0",
-    "filled-array": "^1.0.0",
-    "object-assign": "^4.0.1",
-    "repeating": "^2.0.0",
     "string-width": "^1.0.1",
@@ -49,3 +46,6 @@ "widest-line": "^1.0.0"
     "xo": "*"
+  },
+  "xo": {
+    "esnext": true
   }
 }

readme.md

@@ -175,2 +175,2 @@ # <img src="screenshot.png" width="400" alt="boxen">
 
-MIT © [Sindre Sorhus](http://sindresorhus.com)
+MIT © [Sindre Sorhus](https://sindresorhus.com)

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Dependency count

6

decreased by-33.33%

Worsened metrics

Total package byte prevSize

8535

decreased by-3.69%

Lines of code

110

decreased by-11.29%

Dependency changes

• + Addedcamelcase@3.0.0(transitive)

• - Removedfilled-array@^1.0.0

• - Removedobject-assign@^4.0.1

• - Removedrepeating@^2.0.0

• - Removedcamelcase@2.1.1(transitive)

• - Removedfilled-array@1.1.0(transitive)

• - Removedis-finite@1.1.0(transitive)

• - Removedobject-assign@4.1.1(transitive)

• - Removedrepeating@2.0.1(transitive)

• Updatedcamelcase@^3.0.0