[SOLVED]UIDragScrollView remembering last scrollView on release with autofind

[Genhain]

1. void FindScrollView ()
2. {
3.         // If the scroll view is on a parent, don't try to remember it (as we want it to be dynamic in case of re-parenting)
4.         UIScrollView sv = NGUITools.FindInParents<UIScrollView>(mTrans);
5.  
6.         if (scrollView == null)
7.         {
8.                 scrollView = sv;
9.                 mAutoFind = true;
10.         }
11.         else if (scrollView == sv)
12.         {
13.                 mAutoFind = true;
14.         }
15.         mScroll = scrollView;
16. }
17.  

so with this currently as is, if i drag from one scrollview to another, it will not reparent to the new scrollview...unless we either clear the reference or the previous scrollview get's destroyed or you press and release first where it will set the new scrollview anyway. Unless i am using UIDragscrollView & UIDragDropItem with scrollviews wrong? But i do not think so as this worked before i upgraded.

[ArenMook]

The code in FindScrollView() will only run in OnEnable(), so unless you are enabling/disabling your content it won't execute. It's probably best to call it yourself in your OnDrop or wherever you're doing the reparenting.

droppedGameObject.BroadcastMessage("FindScrollView");

[Genhain]

Yes and it has a UIDragDropItem script attached which disables UIDragScrollView on start and re-enables it on release. so FindScrollView was being called but once it got into FindScrollView there is checks for null and equality, but not if the current parent scrollview is different from the last, hence it just keeps the scrollview reference to it's previous parent. I am not sure if this is intended...but if not, here is how i fixed it.

1. void FindScrollView ()
2. {
3.         // If the scroll view is on a parent, don't try to remember it (as we want it to be dynamic in case of re-parenting)
4.         UIScrollView sv = NGUITools.FindInParents<UIScrollView>(mTrans);
5.  
6.                                            //-->EXTRA CHECK HERE<--
7.         if (scrollView == null || ( mAutoFind && scrollView != sv ))
8.         {
9.                 scrollView = sv;
10.                 mAutoFind = true;
11.         }
12.         else if (scrollView == sv)
13.         {
14.                 mAutoFind = true;
15.         }
16.         mScroll = scrollView;
17. }
18.  

[ArenMook]

That makes sense, thanks.