feat: Always show location bar on error

The location bar should be visible on error page to let people see and
change the URL if needed.

Additionally, it must be visible on insecure connections to show people
that the connection if not secure.

In the future, the location bar should also be shown if the loaded
website is uncommon in the context of the current service, i.e., it is
not explcitly allowlisted by the recipe.

Signed-off-by: Kristóf Marussy <kristof@marussy.com>

1 files changed, 8 insertions, 4 deletions

diff --git a/packages/renderer/src/components/sidebar/ToggleLocationBarButton.tsx b/packages/renderer/src/components/sidebar/ToggleLocationBarButton.tsx
index 325160e..7fc559d 100644
--- a/packages/renderer/src/components/sidebar/ToggleLocationBarButton.tsx
+++ b/packages/renderer/src/components/sidebar/ToggleLocationBarButton.tsx
@@ -47,19 +47,23 @@ function ToggleLocationBarIcon({
 
 function ToggleLocationBarButton(): JSX.Element {
   const { t } = useTranslation();
-  const { settings } = useStore();
-  const { selectedService, showLocationBar } = settings;
+  const {
+    shared: { locationBarVisible, canToggleLocationBar },
+    settings,
+  } = useStore();
+  const { selectedService } = settings;
 
   return (
     <IconButton
-      aria-pressed={showLocationBar}
+      disabled={!canToggleLocationBar}
+      aria-pressed={locationBarVisible}
       aria-controls={LOCATION_BAR_ID}
       aria-label={t('toolbar.toggleLocationBar')}
       onClick={() => settings.toggleLocationBar()}
     >
       <ToggleLocationBarIcon
         loading={selectedService?.loading ?? false}
-        show={showLocationBar}
+        show={locationBarVisible}
       />
     </IconButton>
   );