Feature Request for isnotphone tag

[Stephen McIver]

Hi Gareth,

This plugin has the {exp:detect_mobile:isphone} tag. It'd be nice if there was also a {exp:detect_mobile:isnotphone} tag too (and I guess, a not equivalent of any other tags that exist, if they don't have one).

We could then use these in simple conditionals, without having to use the advanced if:else conditionals (which I think, has performance benefits).

Thanks,

Stephen

[Şerban Ghiţă]

@sm9 !$detect->isMobile() does the job. There is no other way. Not sure how to do this in EE2.

[Stephen McIver]

Thanks Serban, I should have thought of that! I just saw the isnotmobile tag and assumed there would be 'isnot' versions for everything.

You're right though - I can probably do something like, {if "{exp:detect_mobile:isphone}" != ""} instead.

Thanks,

Stephen

[Gareth Davies]

Hi,

Sorry, I wasn't getting notifications for this repo and completely missed this. I've just merged in a pull request which adds the isnotphone method so use as {exp:detect_mobile:isnotphone}

[Stephen McIver]

No worries Gareth, and thanks for adding it. :)