[Bf-blender-cvs] [fb61711b1a] master: Fix T50570: pressing pgup or pgdn in any scrollable area irreversably alters scrolling speed.

Bastien Montagne noreply at git.blender.org
Wed Feb 1 12:54:32 CET 2017


Commit: fb61711b1a9ef96becec8eb15916f057055f567b
Author: Bastien Montagne
Date:   Wed Feb 1 12:52:26 2017 +0100
Branches: master
https://developer.blender.org/rBfb61711b1a9ef96becec8eb15916f057055f567b

Fix T50570: pressing pgup or pgdn in any scrollable area irreversably alters scrolling speed.

'page' prop of scroll up/down operators would get stuck once set once by
pageup/down keys... Now only take this prop into account if explicitely
set, not when its value is inherited from previous run.

===================================================================

M	source/blender/editors/interface/view2d_ops.c

===================================================================

diff --git a/source/blender/editors/interface/view2d_ops.c b/source/blender/editors/interface/view2d_ops.c
index d2b2f12c1a..fa2c1f25cf 100644
--- a/source/blender/editors/interface/view2d_ops.c
+++ b/source/blender/editors/interface/view2d_ops.c
@@ -447,7 +447,8 @@ static int view_scrolldown_exec(bContext *C, wmOperator *op)
 	RNA_int_set(op->ptr, "deltax", 0);
 	RNA_int_set(op->ptr, "deltay", -40);
 	
-	if (RNA_boolean_get(op->ptr, "page")) {
+	PropertyRNA *prop = RNA_struct_find_property(op->ptr, "page");
+	if (RNA_property_is_set(op->ptr, prop) && RNA_property_boolean_get(op->ptr, prop)) {
 		ARegion *ar = CTX_wm_region(C);
 		RNA_int_set(op->ptr, "deltay", ar->v2d.mask.ymin - ar->v2d.mask.ymax);
 	}
@@ -497,7 +498,8 @@ static int view_scrollup_exec(bContext *C, wmOperator *op)
 	RNA_int_set(op->ptr, "deltax", 0);
 	RNA_int_set(op->ptr, "deltay", 40);
 	
-	if (RNA_boolean_get(op->ptr, "page")) {
+	PropertyRNA *prop = RNA_struct_find_property(op->ptr, "page");
+	if (RNA_property_is_set(op->ptr, prop) && RNA_property_boolean_get(op->ptr, prop)) {
 		ARegion *ar = CTX_wm_region(C);
 		RNA_int_set(op->ptr, "deltay", BLI_rcti_size_y(&ar->v2d.mask));
 	}




More information about the Bf-blender-cvs mailing list