Fix #2087, do not show the call-to-action banner on mobile

package.json

@@ -19,6 +19,7 @@
     "express": "4.14.0",
     "jpm": "1.2.2",
     "keygrip": "1.0.1",
+    "mobile-detect": "1.3.5",
     "morgan": "1.7.0",
     "mozlog": "2.0.6",
     "node-statsd": "0.1.1",

server/src/pages/shot/model.js

@@ -1,7 +1,9 @@
 const { getGitRevision } = require("../../linker");
+const MobileDetect = require('mobile-detect');
 
 exports.createModel = function (req) {
   let buildTime = require("../../build-time").string;
+  let isMobile = !! (new MobileDetect(req.headers['user-agent'])).mobile();
   let serverPayload = {
     title: req.shot.title,
     allowExport: req.config.allowExport,
@@ -26,7 +28,8 @@ exports.createModel = function (req) {
     sentryPublicDSN: req.config.sentryPublicDSN,
     cspNonce: req.cspNonce,
     hashAnalytics: true,
-    userAgent: req.headers['user-agent']
+    userAgent: req.headers['user-agent'],
+    isMobile
   };
   let clientPayload = {
     title: req.shot.title,
@@ -52,7 +55,8 @@ exports.createModel = function (req) {
     retentionTime: req.config.expiredRetentionTime*1000,
     defaultExpiration: req.config.defaultExpiration*1000,
     hashAnalytics: true,
-    userAgent: req.headers['user-agent']
+    userAgent: req.headers['user-agent'],
+    isMobile
   };
   if (serverPayload.expireTime !== null && Date.now() > serverPayload.expireTime) {
     clientPayload.shot = {

server/src/pages/shot/view.js

@@ -356,7 +356,7 @@ class Body extends React.Component {
 
     let renderGetFirefox = this.props.userAgent && (this.props.userAgent + "").search(/firefox\/\d+/i) === -1;
     let renderExtensionNotification = ! (this.props.isExtInstalled || renderGetFirefox);
-    if (this.state.closePageshotBanner) {
+    if (this.props.isMobile || this.state.closePageshotBanner) {
       renderGetFirefox = renderExtensionNotification = false;
     }