From dcf651fb519d8f7b3ffd97ed5321a1efc1f102e1 Mon Sep 17 00:00:00 2001
From: Daniel Gasienica <daniel@gasienica.ch>
Date: Mon, 26 Feb 2018 11:28:37 -0500
Subject: [PATCH] Hide notifications audio setting on Linux

---
 background.html           |  2 ++
 js/views/settings_view.js | 16 +++++++++++-----
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/background.html b/background.html
index ade360ddb..19865b02a 100644
--- a/background.html
+++ b/background.html
@@ -606,10 +606,12 @@
         </div>
       </div>
       <br />
+      {{ #shouldShowAudioNotificationSetting }}
       <div class='audio-notification-setting'>
         <input type='checkbox' name='audio-notification' id='audio-notification'/>
         <label for='audio-notification'>{{ audioNotificationDescription }}</label>
       </div>
+      {{ /shouldShowAudioNotificationSetting }}
     </div>
   </script>
   <script type='text/x-tmpl-mustache' id='syncSettings'>
diff --git a/js/views/settings_view.js b/js/views/settings_view.js
index 517d6537b..4d384eed7 100644
--- a/js/views/settings_view.js
+++ b/js/views/settings_view.js
@@ -4,6 +4,8 @@
 (function () {
     'use strict';
     window.Whisper = window.Whisper || {};
+    const { OS } = window.Signal;
+    const { Settings } = window.Signal.Types;
 
     var CheckboxView = Whisper.View.extend({
         initialize: function(options) {
@@ -68,11 +70,13 @@
                 name: 'theme-setting',
                 event: 'change-theme'
             });
-            new CheckboxView({
-                el: this.$('.audio-notification-setting'),
-                defaultValue: false,
-                name: 'audio-notification'
-            });
+            if (Settings.shouldShowAudioNotificationSetting()) {
+                new CheckboxView({
+                    el: this.$('.audio-notification-setting'),
+                    defaultValue: false,
+                    name: 'audio-notification'
+                });
+            }
             new CheckboxView({
                 el: this.$('.menu-bar-setting'),
                 defaultValue: false,
@@ -100,6 +104,8 @@
               noNameOrMessage: i18n('noNameOrMessage'),
               nameOnly: i18n('nameOnly'),
               audioNotificationDescription: i18n('audioNotificationDescription'),
+              shouldShowAudioNotificationSetting:
+                Settings.shouldShowAudioNotificationSetting(),
               themeAndroidDark: i18n('themeAndroidDark'),
               hideMenuBar: i18n('hideMenuBar'),
             };