Removed all TODOs from source and added them to my mindmap

Author: Benedikt Koeppel

src/ch/benediktkoeppel/code/droidplane/HorizontalMindmapView.java

@@ -481,7 +481,6 @@ private NodeColumn getLeftmostVisibleColumn() {
 	 * Get the number of pixels that are visible on the leftmost column.
 	 * @return
 	 */
-	// TODO (medium): this is ugly, DRY from getLeftmostVisibleColumn() !
 	private int getVisiblePixelOfLeftmostColumn() {
 
 		// how much we are horizontally scrolled

src/ch/benediktkoeppel/code/droidplane/MainActivity.java

@@ -21,16 +21,6 @@
 
 import com.google.analytics.tracking.android.EasyTracker;
 
-// TODO (low): think about a strategy to strip out Log.v and Log.d messages for releases
-// TODO (low): start using a SAX parser and build my own MindMap, dynamically build branches when user drills down, truncate branches when they are not used anymore. How will we do Edit Node / Insert Node, if we are using a SAX parser? Maybe we should not go for a SAX parser but find a more efficient DOM parser?
-
-// TODO (low): allow us to open multiple files and display their root nodes and file names in the leftmost column. 
-// TODO (medium): long-click on a root node shows a "close file" or "close this mindmap" menu
-// TODO (high): add a progress bar when opening a file (or a spinner or so)
-// TODO (low): can we get built-in icons as SVG?
-// TODO (medium): properly parse rich text nodes
-// TODO (high): implement OnItemLongClickListener with a context menu (show all icons, follow link, copy text, and ultimately also edit)
-
 /**
  * The MainActivity can be started from the App Launcher, or with a File Open
  * intent. If the MainApplication was already running, the previously used
@@ -81,8 +71,6 @@ public void onCreate(Bundle savedInstanceState) {
         String type = intent.getType();
 
 		// if the application was reset, or the document has changed, we need to re-initialize everything
-		// TODO (high): factor this stuff out. we really should have a loadDocument(InputStream) method somewhere
-
 		if ( application.mindmap == null || application.mindmap.document == null
 				|| (application.mindmap.getUri()!=intent.getData() && intent.getData()!=null)
 				|| (intent.getBooleanExtra(INTENT_START_HELP,false))
@@ -205,13 +193,6 @@ public void onCreate(Bundle savedInstanceState) {
     @Override
 	public boolean onCreateOptionsMenu(android.view.Menu menu) {
 
-		// TODO (low): add "Find" button and menu -> should search underneath the
-		// current node (or with an option, under the root node)
-    	
-    	// TODO (medium): menu "Open"
-    	
-    	// TODO (low): settings (to set the number of horizontal and vertical columns)
-
         MenuInflater inflater = getMenuInflater();
         inflater.inflate(R.menu.main, menu);
         return true;
@@ -270,7 +251,6 @@ public boolean onOptionsItemSelected(android.view.MenuItem item) {
 
 
 	// Handler when an item is long clicked
-	// TODO (high): implement long-click handler
 //	@Override
 //	public boolean onItemLongClick(AdapterView<?> parent, View view, int position, long id) {
 //		

src/ch/benediktkoeppel/code/droidplane/MainApplication.java

@@ -13,18 +13,6 @@
 @ReportsCrashes(formKey = "dE1VQVpQN2FNTWlLQXg1UUQ1b1VSN3c6MQ") 
 public class MainApplication extends Application {
 
-	/*
-	 * TODO (high): some idiot (yes, me) thought that it was a very clever idea to
-	 * stuff everything into this application. However, now when we open
-	 * multiple documents (i.e. separate activities), then it all breaks
-	 * terribly. We'll have to see how we get all that stuff back into the
-	 * MainActivity.
-	 * But then again we have the problem when the screen rotates. What do we do
-	 * then? Or should we store everything in hashes here in the
-	 * MainApplication, and then give every MainActivity it's own variables back
-	 * out of this hash?
-	 */
-	
 	/**
 	 * Android Logging TAG
 	 */

src/ch/benediktkoeppel/code/droidplane/MindmapNode.java

@@ -72,11 +72,9 @@ public MindmapNode(Node node) {
 		this.node = node;
 
 		// extract the string (TEXT attribute) of the nodes
-		// TODO (medium): how do we handle rich text nodes?
 		text = tmp_element.getAttribute("TEXT");
 
 		// extract icons
-		// TODO (low): how do we handle multiple icons?
 		ArrayList<String> icons = getIcons();
 		String icon="";
 		icon_res_id = 0;