From 729751804bbb4ff67dad202465e6a3faf6ec5579 Mon Sep 17 00:00:00 2001
From: Arik Chakma <arikchangma@gmail.com>
Date: Tue, 4 Apr 2023 00:18:48 +0600
Subject: [PATCH] chore: best practices buttons

---
 src/components/BestPracticeHeader.astro    | 63 +++++++++++++++++-----
 src/components/Login/AccountDropdown.astro |  4 +-
 2 files changed, 51 insertions(+), 16 deletions(-)

diff --git a/src/components/BestPracticeHeader.astro b/src/components/BestPracticeHeader.astro
index 3b1c08734..38ac3971f 100644
--- a/src/components/BestPracticeHeader.astro
+++ b/src/components/BestPracticeHeader.astro
@@ -2,6 +2,7 @@
 import BestPracticeHint from './BestPracticeHint.astro';
 import DownloadPopup from './DownloadPopup.astro';
 import Icon from './Icon.astro';
+import LoginPopup from './Login/LoginPopup.astro';
 import SubscribePopup from './SubscribePopup.astro';
 
 export interface Props {
@@ -15,23 +16,22 @@ const { title, description, bestPracticeId, isUpcoming = false } = Astro.props;
 const isBestPracticeReady = !isUpcoming;
 ---
 
-<DownloadPopup />
-<SubscribePopup />
+<LoginPopup />
 
 <div class='border-b'>
-  <div class='py-5 sm:py-12 container relative'>
+  <div class='container relative py-5 sm:py-12'>
     <div class='mt-0 mb-3 sm:mb-6'>
-      <h1 class='text-2xl sm:text-4xl mb-0.5 sm:mb-2 font-bold'>
+      <h1 class='mb-0.5 text-2xl font-bold sm:mb-2 sm:text-4xl'>
         {title}
       </h1>
-      <p class='text-gray-500 text-sm sm:text-lg'>{description}</p>
+      <p class='text-sm text-gray-500 sm:text-lg'>{description}</p>
     </div>
 
     <div class='flex justify-between'>
       <div class='flex gap-1 sm:gap-2'>
         <a
           href='/best-practices'
-          class='bg-gray-500 py-1.5 px-3 rounded-md text-white text-xs sm:text-sm font-medium hover:bg-gray-600'
+          class='rounded-md bg-gray-500 py-1.5 px-3 text-xs font-medium text-white hover:bg-gray-600 sm:text-sm'
           aria-label='Back to All Best Practices'
         >
           &larr;<span class='hidden sm:inline'>&nbsp;All Best Practices</span>
@@ -40,22 +40,41 @@ const isBestPracticeReady = !isUpcoming;
         {
           isBestPracticeReady && (
             <button
-              data-popup='download-popup'
-              class='inline-flex items-center justify-center bg-yellow-400 py-1.5 px-3 text-xs sm:text-sm font-medium rounded-md hover:bg-yellow-500'
-              aria-label='Download Best Practice'
+              id='download-button'
+              data-popup='login-popup'
+              class='inline-flex items-center justify-center rounded-md bg-yellow-400 px-3 py-1.5 text-xs font-medium hover:bg-yellow-500 sm:text-sm'
+              aria-label='Download Roadmap'
               ga-category='Subscription'
               ga-action='Clicked Popup Opener'
-              ga-label='Download Best Practice Popup'
+              ga-label='Download Roadmap Popup'
             >
               <Icon icon='download' />
-              <span class='hidden sm:inline ml-2'>Download</span>
+              <span class='ml-2 hidden sm:inline'>Download</span>
             </button>
           )
         }
 
+        {
+          isBestPracticeReady && (
+            <a
+              id='download-link'
+              class='inline-flex hidden items-center justify-center rounded-md bg-yellow-400 px-3 py-1.5 text-xs font-medium hover:bg-yellow-500 sm:text-sm'
+              aria-label='Download Roadmap'
+              ga-category='Subscription'
+              ga-action='Clicked Popup Opener'
+              ga-label='Download Roadmap Popup'
+              href={`/best-practices/${bestPracticeId}.png`}
+            >
+              <Icon icon='download' />
+              <span class='ml-2 hidden sm:inline'>Download</span>
+            </a>
+          )
+        }
+
         <button
-          data-popup='subscribe-popup'
-          class='inline-flex items-center justify-center bg-yellow-400 py-1.5 px-3 text-xs sm:text-sm font-medium rounded-md hover:bg-yellow-500'
+          id='subscribe-button'
+          data-popup='login-popup'
+          class='inline-flex items-center justify-center rounded-md bg-yellow-400 py-1.5 px-3 text-xs font-medium hover:bg-yellow-500 sm:text-sm'
           aria-label='Subscribe for Updates'
           ga-category='Subscription'
           ga-action='Clicked Popup Opener'
@@ -71,7 +90,7 @@ const isBestPracticeReady = !isUpcoming;
           <a
             href={`https://github.com/kamranahmedse/developer-roadmap/issues/new?title=[Suggestion] ${title}`}
             target='_blank'
-            class='inline-flex items-center justify-center bg-gray-500 text-white py-1.5 px-3 text-xs sm:text-sm font-medium rounded-md hover:bg-gray-600'
+            class='inline-flex items-center justify-center rounded-md bg-gray-500 px-3 py-1.5 text-xs font-medium text-white hover:bg-gray-600 sm:text-sm'
             aria-label='Suggest Changes'
           >
             <Icon icon='comment' class='h-3 w-3' />
@@ -85,3 +104,19 @@ const isBestPracticeReady = !isUpcoming;
     <BestPracticeHint bestPracticeId={bestPracticeId} />
   </div>
 </div>
+
+<script>
+  import Cookies from 'js-cookie';
+  import { TOKEN_COOKIE_NAME } from '../lib/constants';
+
+  const subscribeButton = document.getElementById('subscribe-button');
+  const downloadButton = document.getElementById('download-button');
+  const downloadLink = document.getElementById('download-link');
+  const token = Cookies.get(TOKEN_COOKIE_NAME);
+
+  if (token) {
+    subscribeButton?.classList.add('hidden');
+    downloadButton?.classList.add('hidden');
+    downloadLink?.classList.remove('hidden');
+  }
+</script>
diff --git a/src/components/Login/AccountDropdown.astro b/src/components/Login/AccountDropdown.astro
index 345f68afc..344b32d54 100644
--- a/src/components/Login/AccountDropdown.astro
+++ b/src/components/Login/AccountDropdown.astro
@@ -4,13 +4,13 @@ import Icon from '../Icon.astro';
 
 <div class='relative block hidden' id='account-dropdown'>
   <button
-    class='flex h-8 w-28 items-center justify-center rounded-full bg-gradient-to-r from-blue-500 to-blue-700 py-2 px-4 text-sm font-medium text-white hover:from-blue-500 hover:to-blue-600'
+    class='flex h-8 w-28 items-center justify-center rounded-full bg-gradient-to-r from-purple-500 to-purple-700 py-2 px-4 text-sm font-medium text-white hover:from-purple-500 hover:to-purple-600'
     type='button'
     data-account-button
   >
     <span class='flex items-center gap-1'>
       Account
-      <Icon icon='chevron-down' class='h-4 w-4 stroke-[3px]' />
+      <Icon icon='chevron-down' class='h-3 w-3 stroke-[3px]' />
     </span>
   </button>