From 9e07e39ba53e2b266dd20562b7816302a8b44fe8 Mon Sep 17 00:00:00 2001
From: Sacha Weatherstone <sachaw100@hotmail.com>
Date: Sun, 16 Jan 2022 19:19:26 +1100
Subject: [PATCH] Fix mqtt diabled indicator

---
 src/components/menu/BottomNav.tsx | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/components/menu/BottomNav.tsx b/src/components/menu/BottomNav.tsx
index 28af7090..7cbba3cf 100644
--- a/src/components/menu/BottomNav.tsx
+++ b/src/components/menu/BottomNav.tsx
@@ -32,6 +32,7 @@ import { Tooltip } from '../generic/Tooltip';
 // }
 
 export const BottomNav = (): JSX.Element => {
+  const [showVersionInfo, setShowVersionInfo] = React.useState(false);
   const dispatch = useAppDispatch();
   const meshtasticState = useAppSelector((state) => state.meshtastic);
   const appState = useAppSelector((state) => state.app);
@@ -81,11 +82,14 @@ export const BottomNav = (): JSX.Element => {
         <Tooltip contents={`MQTT Status`}>
           <div className="flex p-1 border-r border-gray-300 cursor-pointer select-none group dark:border-gray-600 dark:text-white hover:bg-gray-200 dark:hover:bg-primaryDark">
             {primaryChannelSettings?.uplinkEnabled &&
-            primaryChannelSettings?.downlinkEnabled ? (
+            primaryChannelSettings?.downlinkEnabled &&
+            !meshtasticState.radio.preferences.mqttDisabled ? (
               <RiArrowUpDownLine className="p-0.5 group-active:scale-90" />
-            ) : primaryChannelSettings?.uplinkEnabled ? (
+            ) : primaryChannelSettings?.uplinkEnabled &&
+              !meshtasticState.radio.preferences.mqttDisabled ? (
               <RiArrowUpLine className="p-0.5 group-active:scale-90" />
-            ) : primaryChannelSettings?.downlinkEnabled ? (
+            ) : primaryChannelSettings?.downlinkEnabled &&
+              !meshtasticState.radio.preferences.mqttDisabled ? (
               <RiArrowDownLine className="p-0.5 group-active:scale-90" />
             ) : (
               <FiX className="p-0.5" />
@@ -96,15 +100,15 @@ export const BottomNav = (): JSX.Element => {
 
       <div className="flex">
         <Tooltip contents={`Current Commit`}>
-          <a
-            href={`https://github.com/meshtastic/meshtastic-web/commit/${process.env.COMMIT_HASH}`}
-            target="_blank"
-            rel="noreferrer"
+          <div
+            onClick={(): void => {
+              setShowVersionInfo(true);
+            }}
             className="flex p-1 border-l border-gray-300 cursor-pointer select-none group dark:border-gray-600 dark:text-white hover:bg-gray-200 dark:hover:bg-primaryDark"
           >
             <FiGitBranch className="p-0.5 mr-1 group-active:scale-90" />
             <p className="text-xs opacity-60">{process.env.COMMIT_HASH}</p>
-          </a>
+          </div>
         </Tooltip>
         <Tooltip contents={`Notifications`}>
           <div className="flex p-1 border-l border-gray-300 cursor-pointer select-none group dark:border-gray-600 dark:text-white hover:bg-gray-200 dark:hover:bg-primaryDark">