Several changes:

 - Expose touchStarted and touchEnded events on WAbstractItemView
 - Touch: use long long for identifier and add accessor
   Some browsers use negative values, some browsers use values larger than
   2**31 - 1, let's just use long long, so whatever the value is, it will
   be preserved.

Author: RockinRoel

src/Wt/WAbstractItemView.C

@@ -1394,8 +1394,8 @@ void WAbstractItemView::handleMouseUp(const WModelIndex& index,
   mouseWentUp_.emit(index, event);
 }
 
-void WAbstractItemView::handleTouchStart(const std::vector<WModelIndex>& indices,
-					   const WTouchEvent& event)
+void WAbstractItemView::handleTouchSelect(const std::vector<WModelIndex>& indices,
+                                          const WTouchEvent& event)
 {
   const WModelIndex& index = indices[0];
   touchRegistered_ = true;
@@ -1415,6 +1415,18 @@ void WAbstractItemView::handleTouchStart(const std::vector<WModelIndex>& indices
   touchStart_.emit(index, event);
 }
 
+void WAbstractItemView::handleTouchStart(const std::vector<WModelIndex>& indices,
+					   const WTouchEvent& event)
+{
+  touchStarted_.emit(indices, event);
+}
+
+void WAbstractItemView::handleTouchEnd(const std::vector<WModelIndex>& indices,
+				       const WTouchEvent& event)
+{
+  touchEnded_.emit(indices, event);
+}
+
 void WAbstractItemView::setEditTriggers(WFlags<EditTrigger> editTriggers)
 {
   editTriggers_ = editTriggers;

src/Wt/WAbstractItemView.h

@@ -714,9 +714,29 @@ class WT_API WAbstractItemView : public WCompositeWidget
    *
    * When the event happened over an item, the first argument
    * indicates the item that was touched.
+   *
+   * \deprecated Use touchStarted() instead.
    */
   Signal<WModelIndex, WTouchEvent>& touchStart() { return touchStart_; }
 
+  /*! \brief %Signal emitted when one or more fingers are placed on the screen.
+   *
+   * When the event happened over an item, the first argument
+   * indicates the items that were touched. The indices in the model index
+   * vector match the indices in the \link WTouchEvent::changedTouches() changedTouches() of the WTouchEvent\endlink.
+   */
+  Signal<std::vector<WModelIndex>, WTouchEvent>& touchStarted() { return touchStarted_; }
+
+  /*! \brief %Signal emitted when one or more fingers are removed from the screen.
+   *
+   * When the event happened over an item, the first argument
+   * indicates the items where the touch ended. The indices in the model index
+   * vector match the indices in the \link WTouchEvent::changedTouches() changedTouches() of the WTouchEvent\endlink.
+   *
+   * \note When JavaScript is disabled, the signal will never fire.
+   */
+  Signal<std::vector<WModelIndex>, WTouchEvent>& touchEnded() { return touchEnded_; }
+
   /*! \brief %Signal emitted when the selection is changed.
    *
    * \sa select(), setSelectionMode(), setSelectionBehavior()
@@ -1008,10 +1028,20 @@ class WT_API WAbstractItemView : public WCompositeWidget
   virtual void handleMouseUp(const WModelIndex& index,
 			     const WMouseEvent& event);
 
+  /*! \brief Handles a touch select event.
+   */
+  virtual void handleTouchSelect(const std::vector<WModelIndex>& indices,
+				 const WTouchEvent& event);
+
   /*! \brief Handles a touch started event.
    */
   virtual void handleTouchStart(const std::vector<WModelIndex>& indices,
-				       const WTouchEvent& event);
+				const WTouchEvent& event);
+
+  /*! \brief Handles a touch ended event.
+   */
+  virtual void handleTouchEnd(const std::vector<WModelIndex>& indices,
+			      const WTouchEvent& event);
 
   virtual bool internalSelect(const WModelIndex& index, SelectionFlag option);
 
@@ -1083,6 +1113,8 @@ class WT_API WAbstractItemView : public WCompositeWidget
   Signal<WModelIndex, WMouseEvent> mouseWentDown_;
   Signal<WModelIndex, WMouseEvent> mouseWentUp_;
   Signal<WModelIndex, WTouchEvent> touchStart_;
+  Signal<std::vector<WModelIndex>, WTouchEvent> touchStarted_;
+  Signal<std::vector<WModelIndex>, WTouchEvent> touchEnded_;
   Signal<> selectionChanged_;
   Signal<> pageChanged_;
 

src/Wt/WEvent.C

@@ -40,9 +40,8 @@ namespace {
     return Utils::stoi(v);
   }
 
-  unsigned asUInt(const std::string& v) {
-    long long res = Utils::stoll(v);
-    return static_cast<unsigned>(res);
+  long long asLongLong(const std::string& v) {
+    return Utils::stoll(v);
   }
 
   int parseIntParameter(const WebRequest& request, const std::string& name,
@@ -85,7 +84,7 @@ namespace {
 
     try {
       for (unsigned i = 0; i < s.size(); i += 9) {
-	result.push_back(Touch(asUInt(s[i + 0]),
+        result.push_back(Touch(asLongLong(s[i + 0]),
 			       asInt(s[i + 1]), asInt(s[i + 2]),
 			       asInt(s[i + 3]), asInt(s[i + 4]),
 			       asInt(s[i + 5]), asInt(s[i + 6]),
@@ -108,7 +107,7 @@ EventType WEvent::eventType() const
   return impl_.handler->session()->getEventType(*this);
 }
 
-Touch::Touch(unsigned identifier,
+Touch::Touch(long long identifier,
 	     int clientX, int clientY,
 	     int documentX, int documentY,
 	     int screenX, int screenY,

src/Wt/WEvent.h

@@ -34,7 +34,7 @@ namespace Wt {
   public:
     /*! \brief Constructor.
      */
-    Touch(unsigned identifier,
+    Touch(long long identifier,
 	  int clientX, int clientY,
 	  int documentX, int documentY,
 	  int screenX, int screenY,
@@ -59,12 +59,16 @@ namespace Wt {
      */
     Coordinates widget() const { return Coordinates(widgetX_, widgetY_); }
 
+    /*! \brief Returns the identifier for this touch.
+     */
+    long long identifier() const { return identifier_; }
+
   private:
     int clientX_, clientY_;
     int documentX_, documentY_;
     int screenX_, screenY_;
     int widgetX_, widgetY_;
-    unsigned identifier_;
+    long long identifier_;
   };
 
 class WebRequest;

src/Wt/WTableView.C

@@ -55,7 +55,7 @@ WTableView::WTableView()
     plainTable_(nullptr),
     dropEvent_(impl_, "dropEvent"),
     scrolled_(impl_, "scrolled"),
-    itemTouchEvent_(impl_, "itemTouchEvent"),
+    itemTouchSelectEvent_(impl_, "itemTouchSelectEvent"),
     firstColumn_(-1),
     lastColumn_(-1),
     viewportLeft_(0),
@@ -790,8 +790,8 @@ void WTableView::defineJavaScript()
   if (!scrolled_.isConnected())
     scrolled_.connect(this, &WTableView::onViewportChange);
 
-  if (!itemTouchEvent_.isConnected())
-    itemTouchEvent_.connect(this, &WTableView::handleTouchStarted);
+  if (!itemTouchSelectEvent_.isConnected())
+    itemTouchSelectEvent_.connect(this, &WTableView::handleTouchSelected);
 
   if (!columnResizeConnected_) {
     columnResized().connect(this, &WTableView::onColumnResize);
@@ -869,6 +869,18 @@ void WTableView::render(WFlags<RenderFlag> flags)
 	(this, std::bind(&WTableView::handleRootDoubleClick, this, 0,
 			 std::placeholders::_1));
     }
+
+    if (!touchStartConnection_.isConnected()
+        && touchStarted().isConnected()) {
+      touchStartConnection_ = canvas_->touchStarted()
+	.connect(std::bind(&WTableView::handleTouchStarted, this, std::placeholders::_1));
+    }
+
+    if (!touchEndConnection_.isConnected()
+        && touchEnded().isConnected()) {
+      touchEndConnection_ = canvas_->touchEnded()
+	.connect(std::bind(&WTableView::handleTouchEnded, this, std::placeholders::_1));
+    }
   }
 
   if (model())
@@ -1737,15 +1749,35 @@ void WTableView::handleMouseWentUp(bool headerColumns, const WMouseEvent& event)
   handleMouseUp(index, event);
 }
 
-void WTableView::handleTouchStarted(const WTouchEvent& event)
+void WTableView::handleTouchSelected(const WTouchEvent& event)
 {
   std::vector<WModelIndex> indices;
   for(std::size_t i = 0; i < event.touches().size(); i++){
     indices.push_back(translateModelIndex(event.touches()[i]));
   }
+  handleTouchSelect(indices, event);
+}
+
+void WTableView::handleTouchStarted(const WTouchEvent& event)
+{
+  std::vector<WModelIndex> indices;
+  const std::vector<Touch> &touches = event.changedTouches();
+  for(std::size_t i = 0; i < touches.size(); i++){
+    indices.push_back(translateModelIndex(touches[i]));
+  }
   handleTouchStart(indices, event);
 }
 
+void WTableView::handleTouchEnded(const WTouchEvent& event)
+{
+  std::vector<WModelIndex> indices;
+  const std::vector<Touch> &touches = event.changedTouches();
+  for (std::size_t i = 0; i < touches.size(); i++) {
+    indices.push_back(translateModelIndex(touches[i]));
+  }
+  handleTouchEnd(indices, event);
+}
+
 void WTableView::handleRootSingleClick(int u, const WMouseEvent& event)
 {
   handleClick(WModelIndex(), event);

src/Wt/WTableView.h

@@ -170,7 +170,10 @@ class WT_API WTableView : public WAbstractItemView
 
   JSignal<int, int, std::string, std::string, WMouseEvent> dropEvent_;
   JSignal<int, int, int, int> scrolled_;
-  JSignal<WTouchEvent> itemTouchEvent_;
+  JSignal<WTouchEvent> itemTouchSelectEvent_;
+
+  Signals::connection touchStartConnection_;
+  Signals::connection touchEndConnection_;
 
   /* Ajax only: First and last columns rendered (this somewhat
    * redundant with the state contained in the widgets, but because
@@ -241,7 +244,9 @@ class WT_API WTableView : public WAbstractItemView
   void handleDblClick(bool headerColumns, const WMouseEvent& event);
   void handleMouseWentDown(bool headerColumns, const WMouseEvent& event);
   void handleMouseWentUp(bool headerColumns, const WMouseEvent& event);
+  void handleTouchSelected(const WTouchEvent& event);
   void handleTouchStarted(const WTouchEvent& event);
+  void handleTouchEnded(const WTouchEvent& event);
   WModelIndex translateModelIndex(bool headerColumns, const WMouseEvent& event);
   WModelIndex translateModelIndex(const Touch& touch);
 

src/Wt/WTreeView.C

@@ -1364,7 +1364,7 @@ void WTreeView::render(WFlags<RenderFlag> flags)
   if (flags.test(RenderFlag::Full)) {
     defineJavaScript();
 
-    if (!itemTouchEvent_.isConnected()) 
+    if (!itemTouchEvent_.isConnected())
       itemTouchEvent_.connect(this, &WTreeView::onItemTouchEvent);
 
     if (!itemEvent_.isConnected()) {
@@ -1609,16 +1609,19 @@ void WTreeView::onItemEvent(std::string nodeAndColumnId, std::string type,
   }
 }
 
-void WTreeView::onItemTouchEvent(std::string nodeAndColumnId, std::string type,
-				 std::string extra1, std::string extra2, WTouchEvent event)
+void WTreeView::onItemTouchEvent(std::string nodeAndColumnId, std::string type, WTouchEvent event)
 {
   // nodeId and columnId are combined because WSignal needed to be changed
   // since MSVS 2012 does not support >5 arguments in std::bind()
   std::vector<WModelIndex> index;
   index.push_back(calculateModelIndex(nodeAndColumnId));
 
-  if (type == "touchstart")
+  if (type == "touchselect")
+    handleTouchSelect(index, event);
+  else if (type == "touchstart")
     handleTouchStart(index, event);
+  else if (type == "touchend")
+    handleTouchEnd(index, event);
 }
 
 WModelIndex WTreeView::calculateModelIndex(std::string nodeAndColumnId)

src/Wt/WTreeView.h

@@ -268,8 +268,7 @@ class WT_API WTreeView : public WAbstractItemView
 
   JSignal<std::string, std::string, std::string,
           std::string, WMouseEvent> itemEvent_;
-  JSignal<std::string, std::string, std::string,
-	  std::string, WTouchEvent> itemTouchEvent_;
+  JSignal<std::string, std::string, WTouchEvent> itemTouchEvent_;
 
   std::unique_ptr<ToggleButtonConfig> expandConfig_;
 
@@ -300,8 +299,7 @@ class WT_API WTreeView : public WAbstractItemView
   void contentsSizeChanged(int width, int height);
   void onItemEvent(std::string nodeAndColumnId, std::string type,
 		   std::string extra1, std::string extra2, WMouseEvent event);
-  void onItemTouchEvent(std::string nodeAndColumnId, std::string type,
-			std::string extra1, std::string extra2, WTouchEvent event);
+  void onItemTouchEvent(std::string nodeAndColumnId, std::string type, WTouchEvent event);
   WModelIndex calculateModelIndex(std::string nodeAndColumnId);
   void setRootNodeStyle();
   void setCollapsed(const WModelIndex& index);

src/js/WTableView.js

@@ -269,18 +269,18 @@ WT_DECLARE_WT_MEMBER
      }
      if (event.touches.length > 1) {
        clearTimeout(touchStartTimer);
-       touchStartTimer = setTimeout(function(){emitTouchStart(obj, event);}, 1000);
+       touchStartTimer = setTimeout(function(){emitTouchSelect(obj, event);}, 1000);
        touches = event.touches.length;
      }
      else{
        clearTimeout(touchStartTimer);
-       touchStartTimer = setTimeout(function(){emitTouchStart(obj, event);}, 50);
+       touchStartTimer = setTimeout(function(){emitTouchSelect(obj, event);}, 50);
        touches = 1;
      }
    };
 
-   function emitTouchStart(obj, event) {
-     APP.emit(el, { name: 'itemTouchEvent', eventObject: obj, event: event});
+   function emitTouchSelect(obj, event) {
+     APP.emit(el, { name: 'itemTouchSelectEvent', eventObject: obj, event: event});
    };
 
    this.touchMove = function(obj, event) {