choo-log - npm Package Compare versions

Comparing version 1.4.0 to 1.4.1

index.js

 const deepDiff = require('deep-diff')
 const padRight = require('pad-right')
 const padLeft = require('pad-left')
+const browser = require('detect-browser')
 
@@ -50,3 +51,3 @@ module.exports = chooLog
 
-    if (console.groupCollapsed) {
+    if (groupCollapseSupported()) {
       logGroup(line)
@@ -74,3 +75,3 @@ logInner(name, data)
 
-    if (console.groupCollapsed) {
+    if (groupCollapseSupported()) {
       logGroup(line)
@@ -110,3 +111,3 @@ logInner(err)
 
-    if (console.groupCollapsed) {
+    if (groupCollapseSupported()) {
       logGroup(line)
@@ -165,10 +166,20 @@ logInner(prev, state)
 function colorify (color, line, prev) {
-  if (prev) {
-    if (!prev[0]) prev[0] = ''
-    prev[0] = prev[0] += ' %c' + line
-    prev.push('color: ' + colors[color])
+  var newLine = '%c' + line
+  var newStyle = 'color: ' + colors[color] + ';'
+
+  if (!prev) {
+    prev = [ newLine, newStyle ]
     return prev
+  }
+
+  if (!prev[0]) prev[0] = ''
+  prev[0] += ' ' + newLine
+
+  if (!prev[1]) prev[1] = ''
+  if (browser.name === 'firefox') {
+    prev[1] += ' ' + newStyle
   } else {
-    return [ '%c' + line, 'color: ' + colors[color] ]
+    prev.push(newStyle)
   }
+  return prev
 }
@@ -183,1 +194,6 @@
 }
+
+function groupCollapseSupported () {
+  return console.groupCollapsed && browser.name !== 'firefox'
+}
+

package.json

 {
   "name": "choo-log",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "Development logger for choo",
@@ -25,2 +25,3 @@ "main": "index.js",
     "deep-diff": "^0.3.4",
+    "detect-browser": "^1.5.0",
     "pad-left": "^2.1.0",
@@ -30,3 +31,3 @@ "pad-right": "^0.2.2"
   "devDependencies": {
-    "budo": "^8.3.0",
+    "budo": "^9.1.0",
     "choo": "^3.0.1",
@@ -33,0 +34,0 @@ "dependency-check": "^2.5.2",

README.md

@@ -5,3 +5,3 @@ # choo-log [![stability][0]][1]
 
-Development logger for choo.
+Development logger for [choo][12].
 
@@ -12,11 +12,10 @@ ![screen capture](./screen.gif)
 ```js
-const chooLog = require('choo-log')
+const log = require('choo-log')
 const choo = require('choo')
 
-const logger = chooLog()
-const app = choo({
-  onAction: logger.onAction,
-  onError: logger.onError,
-  onStateChange: logger.onStateChange
-})
+const app = choo()
+app.use(log())
+
+const tree = app.start()
+document.body.appendChild(tree)
 ```
@@ -29,15 +28,10 @@
 
+const app = choo()
+
 // this block of code will be eliminated by any minification if
 // NODE_ENV is set to "production"
 if (process.env.NODE_ENV !== 'production') {
-  const chooLog = require('choo-log')
-  const logger = chooLog()
-  var hooks = {
-    onAction: logger.onAction(),
-    onError: logger.onError(),
-    onStateChange: logger.onStateChange(),
-  }
+  const log = require('choo-log')
+  app.use(log())
 }
-
-const app = hooks ? choo(hooks) : choo()
 ```
@@ -47,13 +41,7 @@
 ### logger = chooLog()
-Create a new logger instance
+Create a new logger instance. Listens to:
+- `onAction()`: show the values inside of new `actions`
+- `onError()`: display errors
+- `onStateChange()`: show current state, previous state and a diff
 
-### logger.onAction
-Logger for `onAction()`
-
-### logger.onError
-Logger for `onError()`
-
-### logger.onStateChange
-Logger for `onStateChange()`
-
 ## Installation
@@ -79,1 +67,2 @@ ```sh
 [11]: https://github.com/feross/standard
+[12]: https://github.com/yoshuawuyts/choo

No alert changes

Improved metrics

Total package byte prevSize

269232

increased by0.07%

Lines of code

229

increased by5.05%

Worsened metrics

Dependency count

4

increased by33.33%

Number of lines in readme file

64

decreased by-14.67%

Dependency changes

• + Addeddetect-browser@^1.5.0

• + Addeddetect-browser@1.12.0(transitive)