[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [51626] trunk/blender/source/blender/ editors/screen/area.c: Bugfix #24343

[Ton Roosendaal]

Revision: 51626
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=51626
Author:   ton
Date:     2012-10-25 14:49:50 +0000 (Thu, 25 Oct 2012)
Log Message:
-----------
Bugfix #24343

Ancient 2.5 UI issue:

The button views in Property Editor have an internal storage, to ensure the view
on the buttons stays same when you switch contextes or select objects.

Bug was that - in case buttons were all scrolled invisible - sometimes no slider
was drawn to indicate that.

Discussion on whether this is a good feature or not: the only improvement would
be to store such settings even per object... but that's a feature creep I think.

Modified Paths:
--------------
    trunk/blender/source/blender/editors/screen/area.c

Modified: trunk/blender/source/blender/editors/screen/area.c
===================================================================
--- trunk/blender/source/blender/editors/screen/area.c	2012-10-25 14:31:52 UTC (rev 51625)
+++ trunk/blender/source/blender/editors/screen/area.c	2012-10-25 14:49:50 UTC (rev 51626)
@@ -1663,11 +1663,9 @@
 		v2d->scroll |= V2D_SCROLL_HORIZONTAL_HIDE;
 		v2d->scroll &= ~V2D_SCROLL_VERTICAL_HIDE;
 		
-		/* don't jump back when panels close or hide */
-		if (!newcontext)
-			y = MAX2(-y, -v2d->cur.ymin);
-		else
-			y = -y;
+		/* ensure tot is set correctly, to keep views on bottons, with sliders */
+		y = MAX2(-y, -v2d->cur.ymin);
+		
 	}
 	else {
 		/* for now, allow scrolling in both directions (since layouts are optimized for vertical,