experiments/outline-editor/editor.js

@@ -357,9 +357,7 @@ var templates = {
 		// body...
 		// XXX only do this if we have nested elements...
 		elem.collapsed = true
-
-		elem.ignore = true
-
+		
 		// button...
 		return header },
 	// XXX focus button -- see todo...
@@ -534,7 +532,6 @@ var quoted = {
 
 // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -      
 
-// XXX add ability to skip subtree from status calculation...
 var tasks = {
 	__proto__: plugin,
 
@@ -561,8 +558,6 @@ var tasks = {
 					return res }, {})) },
 
 	// State...
-	// NOTE: this will not count checkboxes under '[ignore]', this is 
-	// 		useful for not ounting items withing templates and the like...
 	updateStatus: function(editor, node){
 		node = editor.get(node)
 		if(node == null){
@@ -571,16 +566,10 @@ var tasks = {
 			.querySelector('.view')
 				.querySelector('.completion')
 		if(state){
-			var checkboxes = 
-				node.querySelectorAll('input[type=checkbox]').length 
-				// XXX should we check ignore value here???
-				- node.querySelectorAll('[ignore] input[type=checkbox]').length
-			var checkboxes_checked = 
-				node.querySelectorAll('input[type=checkbox]:checked').length 
-				// XXX should we check ignore value here???
-				- node.querySelectorAll('[ignore] input[type=checkbox]:checked').length
 			var c = 
-				((checkboxes_checked / checkboxes) * 100)
+				((node.querySelectorAll('input[type=checkbox]:checked').length
+						/ node.querySelectorAll('input[type=checkbox]').length)
+					* 100)
 				.toFixed(0)
 			!isNaN(c)
 				&& state.setAttribute('completion', c +'%') }
@@ -1056,7 +1045,6 @@ var JSONOutline = {
 	__block_attrs__: {
 		id: 'attr',
 		collapsed: 'attr',
-		ignore: 'attr',
 		focused: 'cls',
 	},
 

experiments/outline-editor/index.html

@@ -50,7 +50,6 @@ var setup = function(){
   - 
 - ## Bugs:
   focused:: true
-  - BUG: refreshes under FF sometimes produce partial/inconsistent views...
   - BUG: need to account for hidden elements when cursoring between elements
     - to reproduce::
       - XXX move from/to this element
@@ -211,18 +210,10 @@ var setup = function(){
             - [ ] B
             - [ ] C
         - and a shorthand for todo:
-          - TODO in lists like this new items will be created with a todo 
-            template automatically (???)
+          - TODO in lists like this new items will be created with a todo template
             - [ ] A
             - [ ] B
             - [ ] C
-        - Need to ignore the template subtree in `[%]` calculation... DONE
-          collapsed:: true
-          - Example [%] (should be 50%)
-            - TEMPLATE new
-              - [ ] 
-            - [x] Done
-            - [ ] Undone
   - Time to think about a standalone client -- at least to edit own notes as a test...
     - _also this would be a nice opportunity to start the move to a newer electron version_
     - Deployment:
@@ -735,7 +726,7 @@ var setup = function(){
           - [ ] ASAP 
           - [ ] 
           - 
-        - Q: should we include button text in item???
+        - [ ] example item
     - Attributes:
 	  id:: attributes
 	  - collapsed