adding _styles storage to p5.graphics elements closes #387

Author: lauren mccarthy

lib/p5.js

@@ -1,4 +1,4 @@
-/*! p5.js v0.3.8 October 04, 2014 */
+/*! p5.js v0.3.8 October 07, 2014 */
 var shim = function (require) {
     window.requestDraw = function () {
       return window.requestAnimationFrame || window.webkitRequestAnimationFrame || window.mozRequestAnimationFrame || window.oRequestAnimationFrame || window.msRequestAnimationFrame || function (callback, element) {
@@ -98,7 +98,7 @@ var core = function (require, shim, constants) {
       this._updateInterval = 0;
       this._isGlobal = false;
       this._loop = true;
-      this.styles = [];
+      this._styles = [];
       this._defaultCanvasSize = {
         width: 100,
         height: 100
@@ -597,6 +597,7 @@ var p5Graphics = function (require, core, constants) {
         this._pInst._setProperty('height', this.height);
       } else {
         this.canvas.style.display = 'none';
+        this._styles = [];
       }
     };
     p5.Graphics.prototype = Object.create(p5.Element.prototype);
@@ -1807,7 +1808,7 @@ var dataconversion = function (require, core) {
         radix = radix | 10;
         return parseInt(n, radix);
       } else if (typeof n === 'number') {
-        return Math.floor(n);
+        return n | 0;
       } else if (typeof n === 'boolean') {
         return n ? 1 : 0;
       } else if (n instanceof Array) {
@@ -4442,7 +4443,7 @@ var structure = function (require, core) {
     };
     p5.prototype.push = function () {
       this.drawingContext.save();
-      this.styles.push({
+      this._styles.push({
         doStroke: this._doStroke,
         doFill: this._doFill,
         tint: this._tint,
@@ -4458,7 +4459,7 @@ var structure = function (require, core) {
     };
     p5.prototype.pop = function () {
       this.drawingContext.restore();
-      var lastS = this.styles.pop();
+      var lastS = this._styles.pop();
       this._doStroke = lastS.doStroke;
       this._doFill = lastS.doFill;
       this._tint = lastS.tint;

lib/p5.min.js

@@ -146,7 +146,7 @@ define(function (require) {
     this._updateInterval = 0;
     this._isGlobal = false;
     this._loop = true;
-    this.styles = [];
+    this._styles = [];
     this._defaultCanvasSize = {
       width: 100,
       height: 100

src/core/core.js

@@ -53,7 +53,7 @@ define(function (require) {
       radix = radix | 10;
       return parseInt(n, radix);
     } else if (typeof n === 'number') {
-      return Math.floor(n);
+      return n | 0;
     } else if (typeof n === 'boolean') {
       return n ? 1 : 0;
     } else if (n instanceof Array) {

src/data/conversion.js

@@ -54,6 +54,7 @@ define(function(require) {
       this._pInst._setProperty('height', this.height);
     } else { // hide if offscreen buffer by default
       this.canvas.style.display = 'none';
+      this._styles = []; // non-main elt styles stored in p5.Graphics
     }
   };
 

src/objects/p5.Graphics.js

@@ -102,7 +102,7 @@ define(function (require) {
    */
   p5.prototype.push = function () {
     this.drawingContext.save();
-    this.styles.push({
+    this._styles.push({
       doStroke: this._doStroke,
       doFill: this._doFill,
       tint: this._tint,
@@ -170,7 +170,7 @@ define(function (require) {
    */
   p5.prototype.pop = function () {
     this.drawingContext.restore();
-    var lastS = this.styles.pop();
+    var lastS = this._styles.pop();
     this._doStroke = lastS.doStroke;
     this._doFill = lastS.doFill;
     this._tint = lastS.tint;