Bug (I think): webix.env.isAndroid not correct

Webix JS
#1

Hi,

I noticed tonight that on my Galaxy S7 using Chrome, webix.env.isAndroid returns false. Clearly, that’s wrong. I took a peek at the code and I think the problem is the second indexOf() here:

if(agent.toLowerCase().indexOf("android")!=-1){
    webix.env.isAndroid = true;
    if(agent.toLowerCase().indexOf("trident")){
      webix.env.isAndroid = false;
      webix.env.isIEMobile = true;
    }
}

If I’m following the intent here, I believe you want!= -1 in that second if statement as well. -1 is being interpreted as true, so you’re hitting the setting ofisAndroid=false incorrectly in this case.

Unless I’m missing something of course :slight_smile:

#2

Yep, you are right. Fix will be included in the next build.