FAQ Component - React | TailwindCSS | MaterialUI

Community feedback

• 

  Yazdun•1,310

  @Yazdun

  Posted over 1 year ago

  This is looking great, well done!

  I have a suggestion for you, instead of hardcoding accordion data, you can store all the accordion data inside an array and map through the array inside your react component.

  This way you won't need to repeat the same code with different title and description, it also will be so much easier to update the data or change the JSX since the pattern is repeated only once.

  I've already opened a PR which resolves this issue, make sure to check it out.

  Other than that, it's looking fantastic, keep up the incredible work!

  12

  Marked as helpful
• 

  BrunoMoleta•700

  @brunomoleta

  Posted over 1 year ago

  Hi Ellie,

  It's a valuable call to make your React components smaller. From what I checked out you used only an Accordion component (besides the main.jsx)

  For example, inside the Accordion you can do

          return (<>
  {data.map((item) => (
          <DetailsItem
              ...
          />
        ))}
  

  And put all that code in a separate component. It might seem unnecessary at first glance, but it makes it much easier for someone else to read your code.

  The code may be separate so that the app.jsx becomes something similar to:

      <>
        <div className="stack-x-large grid grid-row-1-auto">
          <Container />
        </div>
      </>
  

  I suggest you look at how the files are managed in my solution.

  Great effort, and keep up the hustle.

  Best regards from Brazil 🇧🇷 :)

  01

  Marked as helpful