[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [55369] trunk/blender/source/blender/ windowmanager/intern/wm_operators.c: Just added a comment:

[Ton Roosendaal]

Revision: 55369
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=55369
Author:   ton
Date:     2013-03-18 12:16:19 +0000 (Mon, 18 Mar 2013)
Log Message:
-----------
Just added a comment:

To make circle select allow pass-through for view events, several issues have to be
tackled. 
 1) other modal ops run on top (border select), 
 2) middlemouse is used now 
 3) and what for tablet/trackpad or people without middlemouse?

The MMB deselection for border/circle is not optimal now... needs rethinking this.

Better would be to check on non-persistant-modality for circle, to start with gesture style
event like lasso or border can do now.

Modified Paths:
--------------
    trunk/blender/source/blender/windowmanager/intern/wm_operators.c

Modified: trunk/blender/source/blender/windowmanager/intern/wm_operators.c
===================================================================
--- trunk/blender/source/blender/windowmanager/intern/wm_operators.c	2013-03-18 11:44:56 UTC (rev 55368)
+++ trunk/blender/source/blender/windowmanager/intern/wm_operators.c	2013-03-18 12:16:19 UTC (rev 55369)
@@ -2830,7 +2830,8 @@
 				return OPERATOR_FINISHED; /* use finish or we don't get an undo */
 		}
 	}
-//	/* Allow view navigation??? */
+	/* Allow view navigation??? */
+	/* note, this gives issues: 1) other modal ops run on top (border select), 2) middlemouse is used now 3) tablet/trackpad? */
 //	else {
 //		return OPERATOR_PASS_THROUGH;
 //	}