search

engagingnewsproject / misinfo-dashboard

2 stars 0 forks source link

<Button> Continuity #24

Open luukee opened 5 months ago

luukee commented 5 months ago

All buttons across the app need to be the Material Tailwind style button. Could be a regular button, button with icon or outlined button.

Refer to the above links for Material Tailwind docs.

Example (see this commit for code):

<button className={style.button} onClick={handleAddNewAgencyModal}>
  <FaPlus className="text-blue-600 mr-2" size={12}/>
  Add Agency
</button>

should be changed to :

<Button onClick={handleAddNewAgencyModal}>
  <FaPlus size={12} />
  Add Agency
</Button>

Suggestion: Do a codebase search (Shift + Command + F) for: <button. And change each.

luukee commented 5 months ago

@risha-van Thanks again for working on this project Risha :) Excited to see!

luukee commented 4 months ago

@risha-van Have you been able to get a working branch for me to look at? All to say, we should get this polished off to merge into the main branch.

Please let me know, thanks :)