chore(website): add links to and widen Silver Supporters (typescript-eslint#6327)

JoshuaKGoldberg · web-flow · commit f53bcbc4ba9d · 2023-01-30T18:01:23.000+10:30
diff --git a/packages/website/src/components/FinancialContributors/Sponsor.tsx b/packages/website/src/components/FinancialContributors/Sponsor.tsx
@@ -1,17 +1,17 @@
 import React from 'react';
 
 import styles from './styles.module.css';
-import type { SponsorData, SponsorIncludeOptions } from './types';
+import type { SponsorData } from './types';
 
 interface SponsorProps {
-  include?: SponsorIncludeOptions;
+  includeName?: boolean;
   sponsor: SponsorData;
 }
 
-export function Sponsor({ include = {}, sponsor }: SponsorProps): JSX.Element {
+export function Sponsor({ includeName, sponsor }: SponsorProps): JSX.Element {
   let children = <img alt={`${sponsor.name} logo`} src={sponsor.image} />;
 
-  if (include.name) {
+  if (includeName) {
     children = (
       <>
         {children}
@@ -20,19 +20,15 @@ export function Sponsor({ include = {}, sponsor }: SponsorProps): JSX.Element {
     );
   }
 
-  if (include.link) {
-    children = (
-      <a
-        className={styles.sponsorLink}
-        href={sponsor.website ?? undefined}
-        title={sponsor.name}
-        target="_blank"
-        rel="noopener sponsored"
-      >
-        {children}
-      </a>
-    );
-  }
-
-  return children;
+  return (
+    <a
+      className={styles.sponsorLink}
+      href={sponsor.website ?? undefined}
+      title={sponsor.name}
+      target="_blank"
+      rel="noopener sponsored"
+    >
+      {children}
+    </a>
+  );
 }
diff --git a/packages/website/src/components/FinancialContributors/Sponsors/index.tsx b/packages/website/src/components/FinancialContributors/Sponsors/index.tsx
@@ -2,12 +2,12 @@ import clsx from 'clsx';
 import React from 'react';
 
 import { Sponsor } from '../Sponsor';
-import type { SponsorData, SponsorIncludeOptions } from '../types';
+import type { SponsorData } from '../types';
 import styles from './styles.module.css';
 
 interface SponsorsProps {
   className: string;
-  include?: SponsorIncludeOptions;
+  includeName?: boolean;
   expanded?: boolean;
   sponsors: SponsorData[];
   title: string;
@@ -16,7 +16,7 @@ interface SponsorsProps {
 
 export function Sponsors({
   className,
-  include,
+  includeName,
   title,
   tier,
   sponsors,
@@ -27,7 +27,7 @@ export function Sponsors({
       <ul className={clsx(styles.sponsorsTier, styles[`tier-${tier}`])}>
         {sponsors.map(sponsor => (
           <li key={sponsor.id}>
-            <Sponsor include={include} sponsor={sponsor} />
+            <Sponsor includeName={includeName} sponsor={sponsor} />
           </li>
         ))}
       </ul>
diff --git a/packages/website/src/components/FinancialContributors/Sponsors/styles.module.css b/packages/website/src/components/FinancialContributors/Sponsors/styles.module.css
@@ -82,7 +82,6 @@
   .tierArea {
     margin: 16px 0;
     width: auto;
-    padding: 0 60px;
   }
 
   .tier-gold-supporter {
diff --git a/packages/website/src/components/FinancialContributors/index.tsx b/packages/website/src/components/FinancialContributors/index.tsx
@@ -16,14 +16,13 @@ export function FinancialContributors(): JSX.Element {
       <div className={styles.sponsorsContainer}>
         <Sponsors
           className={styles.tierSponsorArea}
-          include={{ link: true, name: true }}
+          includeName
           tier="platinum-sponsor"
           title="Platinum Sponsors"
           sponsors={sponsors.slice(0, 6)}
         />
         <Sponsors
           className={styles.tierSupporterArea}
-          include={{ link: true }}
           tier="gold-supporter"
           title="Gold Supporters"
           sponsors={sponsors.slice(6, 16)}
diff --git a/packages/website/src/components/FinancialContributors/styles.module.css b/packages/website/src/components/FinancialContributors/styles.module.css
@@ -13,6 +13,10 @@
   margin: 48px 24px 24px;
 }
 
+.tierOtherArea {
+  padding: 0;
+}
+
 .become {
   font-size: 1.25rem;
   padding: 0.75rem 1.75rem;
@@ -36,12 +40,6 @@
     max-width: 100%;
   }
 
-  .tierArea {
-    margin: 16px 0;
-    width: auto;
-    padding: 0 60px;
-  }
-
   .tierSponsorArea {
     grid-area: 1 / 1 / 3 / 2;
   }
@@ -50,6 +48,11 @@
     grid-area: 1 / 2 / 2 / 3;
   }
 
+  .tierSponsorArea,
+  .tierSupporterArea {
+    padding: 0 60px;
+  }
+
   .tierOtherArea {
     grid-area: 2 / 2 / 3 / 3;
   }
diff --git a/packages/website/src/components/FinancialContributors/types.ts b/packages/website/src/components/FinancialContributors/types.ts
@@ -7,8 +7,3 @@ export interface SponsorData {
   totalDonations: number;
   website?: string;
 }
-
-export interface SponsorIncludeOptions {
-  link?: boolean;
-  name?: boolean;
-}

Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,6 @@`
`82`	`82`	`.tierArea {`
`83`	`83`	`margin: 16px 0;`
`84`	`84`	`width: auto;`
`85`		`- padding: 0 60px;`
`86`	`85`	`}`
`87`	`86`
`88`	`87`	`.tier-gold-supporter {`
Original file line number	Diff line number	Diff line change
`@@ -7,8 +7,3 @@ export interface SponsorData {`
`7`	`7`	`totalDonations: number;`
`8`	`8`	`website?: string;`
`9`	`9`	`}`
`10`		`-`
`11`		`-export interface SponsorIncludeOptions {`
`12`		`- link?: boolean;`
`13`		`- name?: boolean;`
`14`		`-}`