Added polyfills for unused functions

AltspaceVR · Oct 9, 2017 · 58ae469 · 58ae469
1 parent 162e5dd
commit 58ae469
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 16 deletions.
diff --git a/src/components/AltspaceComponent.js b/src/components/AltspaceComponent.js
@@ -38,7 +38,7 @@ class AltspaceComponent extends AFrameComponent
 			* @default false
 			* @memberof module:altspace/components.altspace
 			*/
-			usePixelScale: { type: 'boolean', default: 'false'},
+			usePixelScale: { type: 'boolean', default: false},
 
 			/**
 			* Puts the origin at the `bottom`, `middle` (default), or `top` of the Altspace enclosure.
@@ -56,7 +56,7 @@ class AltspaceComponent extends AFrameComponent
 			* @default true
 			* @memberof module:altspace/components.altspace
 			*/
-			enclosuresOnly: { type: 'boolean', default: 'true'},
+			enclosuresOnly: { type: 'boolean', default: true},
 
 			/**
 			* Puts the app into fullspace mode.
@@ -65,7 +65,7 @@ class AltspaceComponent extends AFrameComponent
 			* @default false
 			* @memberof module:altspace/components.altspace
 			*/
-			fullspace: { type: 'boolean', default: 'false'}
+			fullspace: { type: 'boolean', default: false}
 		}
 	}
 
@@ -160,8 +160,10 @@ class AltspaceComponent extends AFrameComponent
 		renderer.getPixelRatio = noop;
 		renderer.getMaxAnisotropy = noop;
 		renderer.setFaceCulling = noop;
+		renderer.submitFrame = noop;
 		renderer.context = {canvas: {}};
 		renderer.shadowMap = {};
+		renderer.requestAnimationFrame = window.requestAnimationFrame;
 	}
 
 	/*

diff --git a/src/components/AltspaceCursorCollider.js b/src/components/AltspaceCursorCollider.js
@@ -22,7 +22,7 @@ class AltspaceCursorCollider extends AFrameComponent
 			* @default true
 			* @memberof module:altspace/components.altspace-cursor-collider
 			*/
-			enabled: {type: 'boolean', default: 'true'}
+			enabled: {type: 'boolean', default: true}
 		};
 	}
 

diff --git a/src/components/NColliders.js b/src/components/NColliders.js
@@ -101,7 +101,7 @@ class NBoxCollider extends NCollider {
 			* @default [1, 1, 1]
 			* @memberof module:altspace/components.n-box-collider
 			*/
-			size: { type: 'vec3', default: '1 1 1' }
+			size: { type: 'vec3', default: {x:1,y:1,z:1} }
 		};
 	}
 }
@@ -125,7 +125,7 @@ class NCapsuleCollider extends NCollider {
 			* @default 1
 			* @memberof module:altspace/components.n-capsule-collider
 			*/
-			radius: { default: '0', type: 'number' },
+			radius: { default: 0, type: 'number' },
 
 			/**
 			* The height of the shaft of the capsule in meters.
@@ -134,7 +134,7 @@ class NCapsuleCollider extends NCollider {
 			* @default 0
 			* @memberof module:altspace/components.n-capsule-collider
 			*/
-			height: { default: '0', type: 'number' },
+			height: { default: 0, type: 'number' },
 
 			/**
 			* The axis with which the capsule is aligned. Must be one of 'x', 'y' or 'z'.
@@ -170,7 +170,7 @@ class NMeshCollider extends NCollider {
 			* @default true
 			* @memberof module:altspace/components.n-mesh-collider
 			*/
-			convex: { type: 'boolean', default: 'true' }
+			convex: { type: 'boolean', default: true }
 		};
 	}
 

diff --git a/src/components/NSmallComponents.js b/src/components/NSmallComponents.js
@@ -76,7 +76,7 @@ class NText extends NativeComponent {
 			* @default 10
 			* @memberof module:altspace/components.n-text
 			*/
-			fontSize: { default: '10', type: 'int' },//roughly a meter tall
+			fontSize: { default: 10, type: 'int' },//roughly a meter tall
 
 			/**
 			* The width of the text area in meters. If the
@@ -86,7 +86,7 @@ class NText extends NativeComponent {
 			* @default 10
 			* @memberof module:altspace/components.n-text
 			*/
-			width: { default: '10', type: 'number' },//in meters
+			width: { default: 10, type: 'number' },//in meters
 
 			/**
 			* The height of the text area in meters. If the
@@ -96,7 +96,7 @@ class NText extends NativeComponent {
 			* @default 1
 			* @memberof module:altspace/components.n-text
 			*/
-			height: { default: '1', type: 'number' },//in meters
+			height: { default: 1, type: 'number' },//in meters
 
 			/**
 			* The horizontal anchor point for the text. Can be `left`, `middle`, or `right`.

diff --git a/src/components/SyncSystem.js b/src/components/SyncSystem.js
@@ -21,31 +21,31 @@ class SyncSystem extends AFrameSystem
 			* @member {string} author
 			* @memberof module:altspace/components.sync-system
 			*/
-			author: { type: 'string', default: null },
+			author: { type: 'string' },
 
 			/**
 			* The name of the app.
 			* @instance
 			* @member {string} app
 			* @memberof module:altspace/components.sync-system
 			*/
-			app: { type: 'string', default: null },
+			app: { type: 'string' },
 
 			/**
 			* Override the instance ID. Can also be overridden with a URL parameter.
 			* @instance
 			* @member {string} instance
 			* @memberof module:altspace/components.sync-system
 			*/
-			instance: { type: 'string', default: null },
+			instance: { type: 'string' },
 
 			/**
 			* Override the base reference. Set this to use your own Firebase.
 			* @instance
 			* @member {string} refUrl
 			* @memberof module:altspace/components.sync-system
 			*/
-			refUrl: { type: 'string', default: null }
+			refUrl: { type: 'string' }
 		};
 	}
 

diff --git a/src/components/instantiator.js b/src/components/instantiator.js
@@ -54,7 +54,7 @@ class Instantiator extends AFrameComponent
 		* @default true
 		* @memberof module:altspace/components.instantiator
 		*/
-		removeLast: {type: 'boolean', default: 'true'},
+		removeLast: {type: 'boolean', default: true},
 	}; }
 
 	init() {