Remove whatsapp button on other than mobile devices

app/javascript/member-facing/track_shares.js

@@ -1,6 +1,12 @@
-import $ from "jquery";
+import $ from 'jquery';
+//TODO: Fix this import :) And rename / modify this file to hide the whatsapp button if the person is not on a mobile device
+import MobileCheck from './backbone/mobile_check';
 
 $(() => {
+  if (!MobileCheck.isMobile()) {
+    $('.button--whatsapp').remove();
+  }
+
   let shared = false;
 
   const handleShare = event => {
@@ -11,16 +17,16 @@ $(() => {
     const share = event.originalEvent.share;
     shared = true;
 
-    if (share.share_type === "f") {
+    if (share.share_type === 'f') {
       ga(
-        "send",
-        "event",
-        "fb:sign_share",
-        "share_progress_share",
+        'send',
+        'event',
+        'fb:sign_share',
+        'share_progress_share',
         window.champaign.personalization.urlParams.id
       );
     }
   };
 
-  $(window).bind("share", handleShare);
+  $(window).bind('share', handleShare);
 });

app/liquid/shares.rb

@@ -12,7 +12,7 @@ def get_all(page)
           css_class: class_from_html(button.sp_button_html)
         }
         unless button.share_progress?
-          variant = random_share_variant(button.sp_type, page)
+          variant = select_share_variant(button.sp_type, page)
           view_params[:variant_id] = variant.id
           # Prepend the desired query parameters (uri encoded) into the url we want to share
           url = button.url << CGI.escape("?src=whatsapp&variant_id=#{variant.id}")
@@ -32,7 +32,7 @@ def get_all_html(page)
 
     private
 
-    def random_share_variant(share_type, page)
+    def select_share_variant(share_type, page)
       # Whatsapp variants are served locally rather than from ShareProgress, so we need to find a way to decide
       # which whatsapp share to show. The easiest way is to choose by random.
       variant_class(share_type).where(page_id: page.id).order('RANDOM()').first