From 3478963a6fcfc81edb454e32a85d781ab15f489b Mon Sep 17 00:00:00 2001
From: shmyga <shmyga.z@gmail.com>
Date: Wed, 11 Sep 2019 17:52:30 +0300
Subject: [PATCH] [view] add ActionDataView

---
 src/main/haxework/view/data/DataView.hx       | 5 ++++-
 src/main/haxework/view/skin/VScrollBarSkin.hx | 4 +++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/main/haxework/view/data/DataView.hx b/src/main/haxework/view/data/DataView.hx
index b843216..2658f8c 100644
--- a/src/main/haxework/view/data/DataView.hx
+++ b/src/main/haxework/view/data/DataView.hx
@@ -9,13 +9,16 @@ import haxework.view.layout.VerticalLayout;
 
 typedef Factory<D, V:IView<Dynamic>> = Int -> D -> V
 
+class ActionDataView<D, V:IView<Dynamic>, A> extends DataView<D, V> {
+    public var onDataAction(default, null):Signal2<D, A> = new Signal2();
+}
+
 class DataView<D, V:IView<Dynamic>> extends GroupView {
 
     public var data(default, set):Array<D>;
     public var factory(default, set):Factory<D, V>;
     public var onItemSelect(default, null):Signal3<Int, D, V> = new Signal3();
     public var onDataSelect(default, null):Signal<D> = new Signal();
-    public var onDataAction(default, null):Signal2<D, String> = new Signal2();
     public var dataViews(default, null):Array<V>;
 
     private var objectIndexes:Map<DisplayObject, Int> = new Map();
diff --git a/src/main/haxework/view/skin/VScrollBarSkin.hx b/src/main/haxework/view/skin/VScrollBarSkin.hx
index bbd2ac8..015aec8 100644
--- a/src/main/haxework/view/skin/VScrollBarSkin.hx
+++ b/src/main/haxework/view/skin/VScrollBarSkin.hx
@@ -23,7 +23,9 @@ import haxework.view.list.ScrollBarView;
             graphics.drawRect(0, 0, view.width, view.height);
             graphics.beginFill(foreColor);
             graphics.lineStyle();
-            graphics.drawRect(0, position, view.width, size);
+            if (!Math.isNaN(position) && !Math.isNaN(size)) {
+                graphics.drawRect(0, position, view.width, size);
+            }
             graphics.endFill();
         }
     }