From c98038308d27ae6e19bd6b97897fc9f640fb8e77 Mon Sep 17 00:00:00 2001
From: "Collin M. Barrett" <collinbarrett@users.noreply.github.com>
Date: Sun, 20 Sep 2020 20:04:07 -0500
Subject: [PATCH] =?UTF-8?q?feat(web):=20=E2=9C=A8=20hide=20subscribe=20lin?=
 =?UTF-8?q?ks=20for=20compressed=20formats,=20link=20tpl=20subscribe=20but?=
 =?UTF-8?q?tons=20to=20tpl=20page?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

ref #28
---
 web/src/components/SubscribeButtons.tsx | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/web/src/components/SubscribeButtons.tsx b/web/src/components/SubscribeButtons.tsx
index 256b7a887..e3b5430de 100644
--- a/web/src/components/SubscribeButtons.tsx
+++ b/web/src/components/SubscribeButtons.tsx
@@ -60,7 +60,7 @@ const SubscribeButton = (props: SubscribeButtonProps) => {
   );
   return (
     <Button
-      disabled={buttonProps.disabled}
+      hidden={buttonProps.disabled}
       block
       icon={<ImportOutlined />}
       type={buttonProps.type}
@@ -78,7 +78,7 @@ const buildButtonProps = (
   isPrimary: boolean
 ) => {
   let type: ButtonType = isPrimary ? "primary" : "default";
-  let disabled: boolean = false;
+  let hidden: boolean = false;
 
   const hrefLocation = `${encodeURIComponent(viewUrl)}`;
   const hrefTitle = `${encodeURIComponent(name)}`;
@@ -99,7 +99,8 @@ const buildButtonProps = (
 
   // Software protocols
   if (viewUrl.includes(".tpl")) {
-    disabled = true; // IE not supported by FilterLists
+    href = `https://filterlists.com/tpl.html`;
+    message = `Subscribe via FilterList's TPL page.`;
   }
   if (
     viewUrl.includes(".lsrules") ||
@@ -109,9 +110,18 @@ const buildButtonProps = (
     message = `Subscribe to ${name} with Little Snitch's rule group subscription feature.`;
   }
 
+  // Formats
+  if (
+    viewUrl.includes(".zip") ||
+    viewUrl.includes(".7z") ||
+    viewUrl.includes(".tar.gz")
+  ) {
+    hidden = true; // no known software supports subscribing to compressed lists
+  }
+
   const title = `${
     prefixes.length ? prefixes.join(" | ") + " | " : ""
   }${message}`;
 
-  return { disabled, type, href, title };
+  return { disabled: hidden, type, href, title };
 };