hackforla / CivicTechJobs

CivicTechJobs will be a platform to help prospective volunteers find interdisciplinary projects that will be useful for their career development while contributing to positive civic impact, and also a CMS (Content Mgmt System) for Hack for LA projects to be able to list their open roles.
https://civictechjobs.org
GNU General Public License v2.0
18 stars 22 forks source link

Component: Create Input component #251

Open Aveline-art opened 2 years ago

Aveline-art commented 2 years ago

Overview

As a developer we need to develop components that help us reuse code. For this issue, we will make the input component on the login page.

Action Items

Designers

Resources/Instructions

Resources

Aveline-art commented 2 years ago

@jenchuu Hi jen! I did not see the component in the design system in Figma, so I am not sure if it is finalized. Once it is, feel free to swap the role labels back and throw this back to development.

kcoronel commented 7 months ago

Hey @fenglugithub, can you help me with this issue. Are the components that Ava is referring to in the Design System on Figma? I looked but I couldn't be 100% sure. TY!

LoTerence commented 1 week ago

I was going through the questions/review column of the project board and was curious about this task so I investigated it.

Not sure how up to date this is, but I found an input component in the figma design system here:

Here is a screenshot of the Input component Ava is referring to: image

Notes

In my opinion, we should just remove Ava's TextField component entirely from the frontend and use the tw-component Matt made. We only need one and the tailwind one is better, plus we are already using it.

Removing Ava's component to reduce redundancy would be a good first issue for a frontend developer. This would remove unnecessary code and shrink our final frontend application bundle size.

Removing any old, unneeded, non-tailwind components (in the frontend/src/components folder) would be helpful to clean up the codebase. It should be okay to remove them if they are not used anywhere else, and if there is already a tw-component for it. This can be another potential first issue for a frontend developer.

Eventually we also want to make sure the final build does not include the demo route. We can either remove it entirely or maybe comment it out of the react router if we want to keep it.