Cleanup

Remove unused code and update link rels

Author: benadam11

package-lock.json

@@ -27,6 +27,7 @@ export const Layout = ({ children }) => {
             <a
               target="_blank"
               href="https://ui.dev"
+              rel="noopener"
               style={{
                 boxShadow: "0 2px 0 0 rgba(255, 255, 255, 0.3)",
                 lineHeight: 2,
@@ -86,7 +87,8 @@ export const Layout = ({ children }) => {
             <div className="level-item">
               <a
                 target="_blank"
-                href="https://twitter.com/gabe_ragland"
+                href="https://twitter.com/uidotdev"
+                rel="noreferrer"
                 onClick={() => {
                   analytics.track("clickFooterTwitter");
                 }}
@@ -100,7 +102,8 @@ export const Layout = ({ children }) => {
             <div className="level-item">
               <a
                 target="_blank"
-                href="https://github.com/gragland/usehooks"
+                href="https://github.com/uidotdev/usehooks"
+                rel="noreferrer"
                 onClick={() => {
                   analytics.track("clickFooterGithub");
                 }}
@@ -115,6 +118,7 @@ export const Layout = ({ children }) => {
               <a
                 target="_blank"
                 href="/rss.xml"
+                rel="noopener"
                 onClick={() => {
                   analytics.track("clickFooterRss");
                 }}
@@ -136,7 +140,8 @@ const GitHubLink = () => {
     <a
       className="is-hidden-mobile"
       target="blank"
-      href="https://github.com/gragland/usehooks"
+      rel="noreferrer"
+      href="https://github.com/uidotdev/usehooks"
       onClick={() => {
         analytics.track("clickTopGithub");
       }}
@@ -181,15 +186,3 @@ const FooterLevel = styled("div").attrs({ className: "level" })`
     text-decoration: underline;
   }
 `;
-
-const Bar = styled("div")`
-  font-size: 20px;
-  padding: 1rem;
-  color: white;
-  text-align: center;
-  background-color: black;
-
-  > a:link {
-    text-decoration: none;
-  }
-`;

src/components/Layout.js

@@ -71,6 +71,7 @@ const PostTemplate = ({ content, frontmatter, slug, permalink }) => {
               <LinksLi key={i}>
                 <a
                   target={link.target || "_blank"}
+                  rel="noopener"
                   href={link.url}
                   onClick={() => {
                     analytics.track("clickExtraLink");
@@ -113,7 +114,7 @@ const PostTemplate = ({ content, frontmatter, slug, permalink }) => {
         {frontmatter.sandbox && (
           <>
             <div className="level-item">
-              <a target="_blank" href={frontmatter.sandbox}>
+              <a target="_blank" rel="noreferrer" href={frontmatter.sandbox}>
                 Open in CodeSandbox
               </a>
             </div>
@@ -123,7 +124,7 @@ const PostTemplate = ({ content, frontmatter, slug, permalink }) => {
           </>
         )}
         <div className="level-item">
-          <a target="_blank" href={frontmatter.gist}>
+          <a target="_blank" rel="noreferrer" href={frontmatter.gist}>
             Suggest a change
           </a>
         </div>

src/components/PostTemplate.js

@@ -5,22 +5,12 @@ import Layout from "../components/Layout";
 import PostTemplate from "../components/PostTemplate";
 
 const IndexPage = ({ pageContext }) => {
-  //const [search, setSearch] = useState("");
-  // const [searchMatches, setSearchMatches] = useState([]);
-
   const { group, index, first, last, pageCount } = pageContext;
-  const previousUrl = index - 1 == 1 ? "/" : `/page/${index - 1}`;
+  const previousUrl = index - 1 === 1 ? "/" : `/page/${index - 1}`;
   const nextUrl = "/page/" + (index + 1).toString();
 
   return (
     <Layout>
-      {/*
-      <Search
-        value={search}
-        onChange={value => setSearch({ search: value })}
-      />
-      */}
-
       {group.map(({ node }) => (
         <PostTemplate
           key={node.id}
@@ -83,44 +73,9 @@ const IndexPage = ({ pageContext }) => {
               );
             })}
         </ul>
-        {/*<span>{` Page ${index} of ${pageCount}`}</span>*/}
       </nav>
     </Layout>
   );
 };
 
 export default IndexPage;
-
-// Quicky and hacky search
-function searchPosts(posts, search) {
-  // Store title matches for quick lookup
-  let titles = {};
-
-  // Get all posts that have a matching title
-  const filterPostsByTitle = posts.filter((post) => {
-    const hook = post.node.frontmatter;
-    const titleLowerCase = hook.title.toLowerCase();
-    const doesInclude = titleLowerCase.includes(search.toLowerCase());
-    if (doesInclude) {
-      titles[titleLowerCase] = true;
-      return true;
-    } else {
-      return false;
-    }
-  });
-
-  // Get all posts that have a matching description and DON'T match by title
-  const filterPostsByDescription = posts.filter((post) => {
-    const hook = post.node.frontmatter;
-    const titleLowerCase = hook.title.toLowerCase();
-    const description = post.node.html || "";
-    return (
-      !titles[titleLowerCase] &&
-      description.toLowerCase().includes(search.toLowerCase())
-    );
-  });
-
-  // Add description matches to end of results
-  const filteredPosts = filterPostsByTitle.concat(filterPostsByDescription);
-  return filteredPosts;
-}