diff options
| author | Ton Roosendaal <ton@blender.org> | 2013-03-18 16:16:19 +0400 |
|---|---|---|
| committer | Ton Roosendaal <ton@blender.org> | 2013-03-18 16:16:19 +0400 |
| commit | 0ee5a2b9562bf63144b36253420c48b6e5680b9e (patch) | |
| tree | 8e7196a9a7d9636b4844b03f693185084e8cf8cf /source/blender/windowmanager/intern/wm_operators.c | |
| parent | 655ed9cc7f237202e8d93b47c7cb12d6fc79d8b0 (diff) | |
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.
Diffstat (limited to 'source/blender/windowmanager/intern/wm_operators.c')
| -rw-r--r-- | source/blender/windowmanager/intern/wm_operators.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source/blender/windowmanager/intern/wm_operators.c b/source/blender/windowmanager/intern/wm_operators.c index 8ea507f97e8..530d3bd90c1 100644 --- a/source/blender/windowmanager/intern/wm_operators.c +++ b/source/blender/windowmanager/intern/wm_operators.c @@ -2830,7 +2830,8 @@ int WM_gesture_circle_modal(bContext *C, wmOperator *op, const wmEvent *event) 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; // } |