(Row/Cell) Selection not properly working on touch + mouse devices

[bschultz]

When both touch and mouse are available (in this case, Chrome on a Windows 8 laptop with touchscreen), it is not possible to select an row with the mouse. Doing the same using touch it's working.

I have a listener for the "dgrid-select" and "dgrid-deselect" event, but it seems those events are not fired when using the mouse.

[kfranqueiro]

Just to check, which version of dgrid are you trying this with? We actually heard a similar report recently and have introduced a fix in master which will be in the upcoming 0.3.12 release. You might want to pull the latest and see if that fixes it.

[bschultz]

Version 0.3.11
Will try it with the master, thx.

[bschultz]

My colleague said that the latest changes didn't fix it.
I haven't verified it yet.

But I guess the problem is with this code snippet:

if(has("touch") && !has("mspointer")){
        // listen for touch taps if available
        on(this.contentNode, touchUtil.selector(selector, touchUtil.tap), function(evt){
                grid._handleSelect(evt, this);
        });
}else{
        // listen for actions that should cause selections
        on(this.contentNode, on.selector(selector, this.selectionEvents), function(event){
                grid._handleSelect(event, this);
        });
}

Because the has("touch") is true it runs into the "if-part" and i guess the touchUtil can't handle mouse clicks.

[william-gr]

Hello!

This bug is affecting quite a few users in our application using dgrid. Can we put a milestone in this or does someone have a patch for this? Thank you very much!

[kfranqueiro]

This is something we are hoping to address sometime this month for dgrid 0.3.14. #844 takes one approach at resolving it (and a few other touch+mouse issues) but its feature detect can result in some false negatives and positives so I plan to play with some other ideas.