fix: post-highlight-page-check (#164)

open-sauced · Jun 4, 2023 · ed98eb2 · ed98eb2
1 parent dd67720
commit ed98eb2
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 36 deletions.
diff --git a/index.html b/index.html
@@ -2,9 +2,9 @@
 <html lang="en">
  <head>
  <meta charset="UTF-8" />
- <link rel="icon" type="image/svg+xml" href="/src/favicon.svg" />
+ <link rel="icon" type="image/svg+xml" href="/src/assets/opensauced-logo.svg" />
  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
- <title>Vite App</title>
+ <title>OpenSauced.ai</title>
  </head>
  <body>
  <div id="root"></div>

diff --git a/src/content-scripts.css b/src/content-scripts.css
diff --git a/src/hooks/useGithubPRPageCheck.ts b/src/hooks/useGithubPRPageCheck.ts
@@ -0,0 +1,19 @@
+import { useEffect, useState } from "react";
+import { isGithubPullRequestPage } from "../utils/urlMatchers";
+
+export const useIsGithubPRPageCheck = () => {
+ const [isGithubPRPage, setIsGithubPRPage] = useState(false);
+
+ useEffect(() => {
+ chrome.tabs.query({ active: true, currentWindow: true }, tabs => {
+ if (tabs.length > 0) {
+ const tab = tabs[0];
+
+ setIsGithubPRPage(isGithubPullRequestPage(tab.url!));
+ }
+ });
+ }, []);
+
+
+ return isGithubPRPage;
+};
diff --git a/src/pages/home.tsx b/src/pages/home.tsx
@@ -19,10 +19,12 @@ import { getHighlights } from "../utils/fetchOpenSaucedApiData";
 import Help from "./help";
 import { OPEN_SAUCED_INSIGHTS_DOMAIN } from "../constants";
 import type { Highlight } from "../ts/types";
+import { useIsGithubPRPageCheck } from "../hooks/useGithubPRPageCheck";
 
 const Home = () => {
  const { user } = useAuth();
  const { currentTabIsOpensaucedUser, checkedUser } = useOpensaucedUserCheck();
+ const isGithubPRPage = useIsGithubPRPageCheck();
  const [highlights, setHighlights] = useState<Highlight[]>([]);
  const [currentPage, setCurrentPage] = useState(0);
 
@@ -173,15 +175,16 @@ const Home = () => {
  AI Configuration
  </button>
 
- <button
- className="flex items-center bg-slate-700 hover:bg-slate-700/70 hover:text-orange text-white gap-2 p-1.5 px-3 w-full rounded-sm font-medium text-sm"
- onClick={() => {
- goTo(PostOnHighlight);
- }}
- >
- <HiPencil />
- Post Highlight
- </button>
+ {isGithubPRPage && (
+ <button
+ className="flex items-center bg-slate-700 hover:bg-slate-700/70 hover:text-orange text-white gap-2 p-1.5 px-3 w-full rounded-sm font-medium text-sm"
+ onClick={() => {
+ goTo(PostOnHighlight);
+ }}
+ >
+ <HiPencil />
+ Post Highlight
+ </button>)}
 
  {currentTabIsOpensaucedUser && (
  <button

diff --git a/src/pages/posthighlight.tsx b/src/pages/posthighlight.tsx
@@ -68,8 +68,9 @@ const PostOnHighlight = () => {
 
  return (
  <div className="p-4 bg-slate-800">
+ <Toaster />
+
  <div className="grid grid-cols-1 divide-y divider-y-center-2 min-w-[320px]">
- <Toaster />
 
  <header className="flex justify-between">
  <div className="flex items-center gap-2">
@@ -115,15 +116,15 @@ const PostOnHighlight = () => {
  disabled={!isSendButtonEnabled}
  onClick={() => generateAiDescription()}
  >
- Summarize
+  Summarize
  </button>
 
  <button
  className="inline-block disabled:bg-gray-500 text-black bg-gh-white rounded-md p-2 text-sm font-semibold text-center select-none w-5/12 border hover:shadow-button hover:no-underline"
  disabled={!isSendButtonEnabled}
  onClick={postHighlight}
  >
- Post
+  Post
  </button>
  </div>
  </main>