[questions][ui] revamp landing page look

apps/portal/src/components/questions/LandingComponent.tsx

@@ -1,13 +1,15 @@
-import { useState } from 'react';
+import { useEffect, useState } from 'react';
 import { ArrowSmallRightIcon } from '@heroicons/react/24/outline';
 import type { QuestionsQuestionType } from '@prisma/client';
 import { Button, Select } from '@tih/ui';
 
-import {
+import { QUESTION_TYPES } from '~/utils/questions/constants';
-  COMPANIES,
+import useDefaultCompany from '~/utils/questions/useDefaultCompany';
-  LOCATIONS,
+import useDefaultLocation from '~/utils/questions/useDefaultLocation';
-  QUESTION_TYPES,
+
-} from '~/utils/questions/constants';
+import type { FilterChoice } from './filter/FilterSection';
+import CompanyTypeahead from './typeahead/CompanyTypeahead';
+import LocationTypeahead from './typeahead/LocationTypeahead';
 
 export type LandingQueryData = {
   company: string;
@@ -22,76 +24,109 @@ export type LandingComponentProps = {
 export default function LandingComponent({
   onLanded: handleLandingQuery,
 }: LandingComponentProps) {
-  const [landingQueryData, setLandingQueryData] = useState<LandingQueryData>({
+  const defaultCompany = useDefaultCompany();
-    company: 'Google',
+  const defaultLocation = useDefaultLocation();
-    location: 'Singapore',
+
-    questionType: 'CODING',
+  const [company, setCompany] = useState<FilterChoice | undefined>(
-  });
+    defaultCompany,
+  );
+  const [location, setLocation] = useState<FilterChoice | undefined>(
+    defaultLocation,
+  );
+
+  const [questionType, setQuestionType] =
+    useState<QuestionsQuestionType>('CODING');
 
-  const handleChangeCompany = (company: string) => {
+  const handleChangeCompany = (newCompany: FilterChoice) => {
-    setLandingQueryData((prev) => ({ ...prev, company }));
+    setCompany(newCompany);
   };
 
-  const handleChangeLocation = (location: string) => {
+  const handleChangeLocation = (newLocation: FilterChoice) => {
-    setLandingQueryData((prev) => ({ ...prev, location }));
+    setCompany(newLocation);
   };
 
-  const handleChangeType = (questionType: QuestionsQuestionType) => {
+  const handleChangeType = (newQuestionType: QuestionsQuestionType) => {
-    setLandingQueryData((prev) => ({ ...prev, questionType }));
+    setQuestionType(newQuestionType);
   };
 
+  useEffect(() => {
+    if (company === undefined) {
+      setCompany(defaultCompany);
+    }
+  }, [defaultCompany, company]);
+
+  useEffect(() => {
+    if (location !== undefined) {
+      setLocation(defaultLocation);
+    }
+  }, [defaultLocation, location]);
+
   return (
-    <main className="flex flex-1 flex-col items-stretch overflow-y-auto bg-white">
+    <main className="flex flex-1 flex-col items-center overflow-y-auto bg-white">
-      <div className="pb-4"></div>
+      <div className="flex flex-1 flex-col items-start justify-center gap-12 px-4">
-      <div className="flex flex-1 flex-col justify-center gap-3">
+        <header className="flex flex-col items-start">
           <div className="flex items-center justify-center">
-          <img alt="app logo" className=" h-20 w-20" src="/logo.svg"></img>
+            <h1 className="text-3xl font-semibold text-slate-900">
-          <h1 className="text-primary-600 p-4 text-center text-5xl font-bold">
               Tech Interview Question Bank
             </h1>
+            <img alt="app logo" className="h-20 w-20" src="/logo.svg"></img>
           </div>
-        <p className="mx-auto max-w-lg p-6 text-center text-xl text-black sm:max-w-3xl">
+          <p className="mb-2 max-w-lg text-5xl font-semibold text-slate-900 sm:max-w-3xl">
-          Get to know the latest SWE interview questions asked by top companies
+            Know the{' '}
+            <span className="text-primary-700">
+              latest SWE interview questions
+            </span>{' '}
+            asked by top companies.
           </p>
-
+        </header>
-        <div className="mx-auto flex max-w-lg items-baseline gap-3 p-4 text-center text-xl text-black sm:max-w-3xl">
+        <div className="flex flex-col items-start gap-3 text-xl font-semibold text-slate-900">
-          <p>Find</p>
+          <p className="text-3xl">Find questions</p>
-          <div className=" space-x-2">
+          <div className="grid grid-cols-2">
+            <p className="text-slate-600">about</p>
             <Select
               isLabelHidden={true}
               label="Type"
               options={QUESTION_TYPES}
-              value={landingQueryData.questionType}
+              value={questionType}
               onChange={(value) => {
                 handleChangeType(value.toUpperCase() as QuestionsQuestionType);
               }}
             />
-          </div>
+            <p className="text-slate-600">from</p>
-          <p>questions from</p>
+            <CompanyTypeahead
-          <Select
               isLabelHidden={true}
-            label="Company"
+              value={company}
-            options={COMPANIES}
+              onSelect={(value) => {
-            value={landingQueryData.company}
+                handleChangeCompany(value);
-            onChange={handleChangeCompany}
+              }}
             />
-          <p>in</p>
+            <p className="text-slate-600">in</p>
-          <Select
+            <LocationTypeahead
               isLabelHidden={true}
-            label="Location"
+              value={location}
-            options={LOCATIONS}
+              onSelect={(value) => {
-            value={landingQueryData.location}
+                handleChangeLocation(value);
-            onChange={handleChangeLocation}
+              }}
             />
+          </div>
           <Button
             addonPosition="end"
             icon={ArrowSmallRightIcon}
             label="Go"
             size="md"
             variant="primary"
-            onClick={() => handleLandingQuery(landingQueryData)}></Button>
+            onClick={() => {
+              if (company !== undefined && location !== undefined) {
+                return handleLandingQuery({
+                  company: company.value,
+                  location: location.value,
+                  questionType,
+                });
+              }
+            }}
+          />
         </div>
-        <div className="flex justify-center p-4">
+        <div className="flex justify-center">
           <iframe
             height={30}
             src="https://ghbtns.com/github-btn.html?user=yangshun&amp;repo=tech-interview-handbook&amp;type=star&amp;count=true&amp;size=large"

apps/portal/src/components/questions/filter/FilterSection.tsx

@@ -3,15 +3,18 @@ import type { UseFormRegisterReturn } from 'react-hook-form';
 import { useForm } from 'react-hook-form';
 import { CheckboxInput, Collapsible, RadioList } from '@tih/ui';
 
-export type FilterOption<V extends string = string> = {
+export type FilterChoice<V extends string = string> = {
-  checked: boolean;
   id: string;
   label: string;
   value: V;
 };
 
+export type FilterOption<V extends string = string> = FilterChoice<V> & {
+  checked: boolean;
+};
+
 export type FilterChoices<V extends string = string> = ReadonlyArray<
-  Omit<FilterOption<V>, 'checked'>
+  FilterChoice<V>
 >;
 
 type FilterSectionType<FilterOptions extends Array<FilterOption>> =

apps/portal/src/components/questions/forms/ContributeQuestionForm.tsx

@@ -89,8 +89,6 @@ export default function ContributeQuestionForm({
             render={({ field }) => (
               <LocationTypeahead
                 required={true}
-                // eslint-disable-next-line @typescript-eslint/no-empty-function
-                onQueryChange={() => {}}
                 onSelect={(option) => {
                   field.onChange(option.value);
                 }}
@@ -143,8 +141,6 @@ export default function ContributeQuestionForm({
             render={({ field }) => (
               <RoleTypeahead
                 required={true}
-                // eslint-disable-next-line @typescript-eslint/no-empty-function
-                onQueryChange={() => {}}
                 onSelect={(option) => {
                   field.onChange(option.value);
                 }}

apps/portal/src/components/questions/forms/CreateQuestionEncounterForm.tsx

@@ -59,8 +59,6 @@ export default function CreateQuestionEncounterForm({
             isLabelHidden={true}
             placeholder="Other location"
             suggestedCount={3}
-            // eslint-disable-next-line @typescript-eslint/no-empty-function
-            onQueryChange={() => {}}
             onSelect={({ value: location }) => {
               setSelectedLocation(location);
             }}
@@ -77,8 +75,6 @@ export default function CreateQuestionEncounterForm({
             isLabelHidden={true}
             placeholder="Other role"
             suggestedCount={3}
-            // eslint-disable-next-line @typescript-eslint/no-empty-function
-            onQueryChange={() => {}}
             onSelect={({ value: role }) => {
               setSelectedRole(role);
             }}

apps/portal/src/components/questions/typeahead/LocationTypeahead.tsx

@@ -5,7 +5,7 @@ import ExpandedTypeahead from './ExpandedTypeahead';
 
 export type LocationTypeaheadProps = Omit<
   ExpandedTypeaheadProps,
-  'label' | 'options'
+  'label' | 'onQueryChange' | 'options'
 >;
 
 export default function LocationTypeahead(props: LocationTypeaheadProps) {
@@ -14,6 +14,8 @@ export default function LocationTypeahead(props: LocationTypeaheadProps) {
       {...(props as ExpandedTypeaheadProps)}
       label="Location"
       options={LOCATIONS}
+      // eslint-disable-next-line @typescript-eslint/no-empty-function
+      onQueryChange={() => {}}
     />
   );
 }

apps/portal/src/components/questions/typeahead/RoleTypeahead.tsx

@@ -5,7 +5,7 @@ import ExpandedTypeahead from './ExpandedTypeahead';
 
 export type RoleTypeaheadProps = Omit<
   ExpandedTypeaheadProps,
-  'label' | 'options'
+  'label' | 'onQueryChange' | 'options'
 >;
 
 export default function RoleTypeahead(props: RoleTypeaheadProps) {
@@ -14,6 +14,8 @@ export default function RoleTypeahead(props: RoleTypeaheadProps) {
       {...(props as ExpandedTypeaheadProps)}
       label="Role"
       options={ROLES}
+      // eslint-disable-next-line @typescript-eslint/no-empty-function
+      onQueryChange={() => {}}
     />
   );
 }

apps/portal/src/utils/questions/useDefaultCompany.ts

@@ -0,0 +1,22 @@
+import type { FilterChoice } from '~/components/questions/filter/FilterSection';
+
+import { trpc } from '../trpc';
+
+export default function useDefaultCompany(): FilterChoice | undefined {
+  const { data: companies } = trpc.useQuery([
+    'companies.list',
+    {
+      name: '',
+    },
+  ]);
+
+  const company = companies?.[0];
+  if (company === undefined) {
+    return company;
+  }
+  return {
+    id: company.id,
+    label: company.name,
+    value: company.id,
+  };
+}

apps/portal/src/utils/questions/useDefaultLocation.ts

@@ -0,0 +1,7 @@
+import type { FilterChoice } from '~/components/questions/filter/FilterSection';
+
+import { LOCATIONS } from './constants';
+
+export default function useDefaultLocation(): FilterChoice | undefined {
+  return LOCATIONS[0];
+}