mirror of
				https://github.com/flynx/pWiki.git
				synced 2025-10-31 11:00:08 +00:00 
			
		
		
		
	notes and cleanup...
Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
This commit is contained in:
		
							parent
							
								
									8699e47376
								
							
						
					
					
						commit
						c5c0ac4611
					
				| @ -886,6 +886,7 @@ var JSONOutline = { | |||||||
| 	__styles: undefined, | 	__styles: undefined, | ||||||
| 
 | 
 | ||||||
| 	// block render...
 | 	// block render...
 | ||||||
|  | 	// XXX PRE_POST_NEWLINE can we avoid explicitly patching for empty lines after pre???
 | ||||||
| 	__code2html__: function(code, elem={}){ | 	__code2html__: function(code, elem={}){ | ||||||
| 		var that = this | 		var that = this | ||||||
| 
 | 
 | ||||||
| @ -937,7 +938,10 @@ var JSONOutline = { | |||||||
| 		// patch for showing trailing empty lines in dom...
 | 		// patch for showing trailing empty lines in dom...
 | ||||||
| 		elem.text =  | 		elem.text =  | ||||||
| 			(elem.text == ''  | 			(elem.text == ''  | ||||||
| 					// XXX adding a </pre> exception here feels hacky...
 | 					// XXX PRE_POST_NEWLINE can we avoid this??
 | ||||||
|  | 					//		...simply .replace(/\n$/, '\n ') does not solve
 | ||||||
|  | 					// 		this -- doubles the single trailing empty line after pre...
 | ||||||
|  | 					// 		...this will require a test for all block elements eventually (???)
 | ||||||
| 					|| elem.text.trim().endsWith('</pre>')) ? | 					|| elem.text.trim().endsWith('</pre>')) ? | ||||||
| 				elem.text  | 				elem.text  | ||||||
| 				// NOTE: adding a space here is done to prevent the browser 
 | 				// NOTE: adding a space here is done to prevent the browser 
 | ||||||
|  | |||||||
| @ -111,7 +111,10 @@ var setup = function(){ | |||||||
|   - |   - | ||||||
| - ## ToDo: | - ## ToDo: | ||||||
|   - add loading spinner -- `.loading` class set on `.load(..)` and unset when done... |   - add loading spinner -- `.loading` class set on `.load(..)` and unset when done... | ||||||
|  |     - DONE set/unset class | ||||||
|  |     - add animation | ||||||
|   - clicking left of the block can select one of parent blocks -- bug or feature?? |   - clicking left of the block can select one of parent blocks -- bug or feature?? | ||||||
|  |     - _can we extend the left click zone all the way to the left edge???_ | ||||||
|   - `.json(..)`, `.data(..)`, ... should be signature compatible with `.get(..)` (???) |   - `.json(..)`, `.data(..)`, ... should be signature compatible with `.get(..)` (???) | ||||||
|     - _current state causes constant confusion..._ |     - _current state causes constant confusion..._ | ||||||
|   - Time to think about a standalone client -- at least to edit own notes as a test... |   - Time to think about a standalone client -- at least to edit own notes as a test... | ||||||
| @ -121,7 +124,6 @@ var setup = function(){ | |||||||
|   - Q: When moving up/down between nodes do we want to place the caret relative to markdown (current) or to view? (see: `CARET_V_MOVE`) |   - Q: When moving up/down between nodes do we want to place the caret relative to markdown (current) or to view? (see: `CARET_V_MOVE`) | ||||||
|   - Q: Should we use `HTMLTextAreaElement.autoUpdateSize(..)` or handle it globally in setup??? |   - Q: Should we use `HTMLTextAreaElement.autoUpdateSize(..)` or handle it globally in setup??? | ||||||
|     - _...I'm leaning towards the later..._ |     - _...I'm leaning towards the later..._ | ||||||
|   - Q: Can we place a cursor in a table correctly??? |  | ||||||
|   - Q: Should tables be text-based markdown or higher-level? |   - Q: Should tables be text-based markdown or higher-level? | ||||||
|     collapsed:: true |     collapsed:: true | ||||||
|     - for reference a normal table |     - for reference a normal table | ||||||
| @ -185,9 +187,8 @@ var setup = function(){ | |||||||
|     - delete |     - delete | ||||||
|     - copy/cut/past (???) |     - copy/cut/past (???) | ||||||
|     - touch/mouse (???) |     - touch/mouse (???) | ||||||
|   - Q: should we select text (mouse/touch) without first focusing?? |   - Q: should we select text through multiple nodes?? | ||||||
|     - _...logseq does not do this either_ |     - _...should this turn into node selection as soon as node boundary is crossed???_ | ||||||
|     - now this is possible by dragging from a click zone... |  | ||||||
|   - copy/paste nodes/trees |   - copy/paste nodes/trees | ||||||
|   - numbered lists: add counters to a depth of 6-7... |   - numbered lists: add counters to a depth of 6-7... | ||||||
|     - _or find a way to make them repeat..._ |     - _or find a way to make them repeat..._ | ||||||
| @ -519,6 +520,18 @@ var setup = function(){ | |||||||
|     -  |     -  | ||||||
|   - --- |   - --- | ||||||
|   - ### Playground for testing |   - ### Playground for testing | ||||||
|  |     - Empty line tests:: | ||||||
|  |       -  | ||||||
|  |         leading | ||||||
|  |       - trailing | ||||||
|  |          | ||||||
|  |       -  | ||||||
|  |         both | ||||||
|  |          | ||||||
|  |       - ``` | ||||||
|  |         block with trailing empty line | ||||||
|  |         ``` | ||||||
|  |          | ||||||
|     - A |     - A | ||||||
|       collapsed:: true |       collapsed:: true | ||||||
|       - a |       - a | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user