annotation: assign dflt pos if category IS NOT present

etpinard · etpinard · commit 9583cfaca0c4 · 2016-11-16T12:47:13.000-05:00
diff --git a/src/plots/cartesian/axes.js b/src/plots/cartesian/axes.js
@@ -116,7 +116,7 @@ axes.coercePosition = function(containerOut, gd, coerce, axRef, attr, dflt) {
             // if position is given as a category name, convert it to a number
             if(typeof pos === 'string' && (ax._categories || []).length) {
                 newPos = ax._categories.indexOf(pos);
-                containerOut[attr] = (newPos !== -1) ? dflt : newPos;
+                containerOut[attr] = (newPos === -1) ? dflt : newPos;
                 return;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,7 @@ axes.coercePosition = function(containerOut, gd, coerce, axRef, attr, dflt) {`
`116`	`116`	`// if position is given as a category name, convert it to a number`
`117`	`117`	`if(typeof pos === 'string' && (ax._categories \|\| []).length) {`
`118`	`118`	`newPos = ax._categories.indexOf(pos);`
`119`		`- containerOut[attr] = (newPos !== -1) ? dflt : newPos;`
	`119`	`+ containerOut[attr] = (newPos === -1) ? dflt : newPos;`
`120`	`120`	`return;`
`121`	`121`	`}`
`122`	`122`	`}`