Browse Source

Editor: Added confirm() to delete action.

Mr.doob 12 years ago
parent
commit
32ffefa50a
1 changed files with 19 additions and 6 deletions
  1. 19 6
      editor/index.html

+ 19 - 6
editor/index.html

@@ -86,13 +86,17 @@
 
 			document.addEventListener( 'keydown', function ( event ) {
 
-				// delete
+				switch ( event.keyCode ) {
 
-				if ( event.which === 46 ) {
+					case 46: // delete
 
-					// here should come confirmation pop-up modal dialog
+						if ( confirm( 'Delete selected object?' ) === true ) {
 
-					signals.objectRemoved.dispatch();
+							signals.objectRemoved.dispatch();
+
+						}
+
+						break;
 
 				}
 
@@ -203,8 +207,17 @@
 
 							var data = JSON.parse( contents );
 
-							if ( data.metadata === undefined ) data.metadata = { type: 'geometry' }; // 2.0
-							if ( data.metadata.type === undefined ) data.metadata.type = 'geometry'; // 3.0
+							if ( data.metadata === undefined ) { // 2.0
+
+								data.metadata = { type: 'geometry' };
+
+							}
+
+							if ( data.metadata.type === undefined ) { // 3.0
+
+								data.metadata.type = 'geometry';
+
+							}
 
 							if ( data.metadata.type === 'geometry' ) {