Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client:Implemented the input component #80

Merged

Conversation

ritwik-69
Copy link
Contributor

@ritwik-69 ritwik-69 commented Jun 7, 2024

fixes:#77

Description

Created the Input component as discussed and made in figma. As props it takes normal input properties like placeholder,value etc. The styling has been done in input.css (I had to rename it input.css from input.module.css as it was not working when importing)

Motivation and Context

[Explain the motivation behind these changes and provide any relevant context.]

How to Test

[Describe the steps to test the changes made in this pull request.]

Related Issues

[If applicable, mention any related issues or pull requests.]

Checklist

  • I have tested these changes locally.
  • I have reviewed the code and ensured it follows the project's coding guidelines.
  • I have updated the documentation, if necessary.
  • I have assigned reviewers to this pull request.

Screenshots (if applicable)

[If your changes include any visual updates, provide screenshots here.]

image

input code

Copy link

vercel bot commented Jun 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
multiplayer-uno ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2024 0:59am

@shivansh-bhatnagar18
Copy link
Owner

@ritwik-69 Can you please use tailwind for styling of components as much as possible? Also I am configuring Kavoom to the tailwind file so change the fonts accordingly.

@ritwik-69
Copy link
Contributor Author

@shivansh-bhatnagar18 I used tailwing css and configured it to include kavoon font

@kuv2707
Copy link
Collaborator

kuv2707 commented Jun 8, 2024

@ritwik-69 Please update the screenshots, and rebase your branch once (against upstream/master)

@ritwik-69
Copy link
Contributor Author

@kuv2707 done with the changes

@shivansh-bhatnagar18
Copy link
Owner

@ritwik-69 Please fix the merge conflicts. The component looks good but add some basic parameters as width and height as parameters. Also remove the related input CSS files because you are already using tailwind

@ritwik-69
Copy link
Contributor Author

@shivansh-bhatnagar18 I don't understand why vercel deployment is failing?

@shivansh-bhatnagar18
Copy link
Owner

shivansh-bhatnagar18 commented Jun 8, 2024

@ritwik-69 You are still importing ./input.css in the input.tsx. To check why this deployment fails try running npm run build
Screenshot from 2024-06-08 19-44-09

@ritwik-69
Copy link
Contributor Author

@ritwik-69 You are still importing ./input.css in the input.tsx. To check why this deployment fails try running npm run build Screenshot from 2024-06-08 19-44-09

sorry about that 🤦‍♂️

@ritwik-69
Copy link
Contributor Author

@shivansh-bhatnagar18 Done with the changes

@kuv2707 kuv2707 added can be merged soon under review There have been some rounds of review on the code changes in the PR. labels Jun 8, 2024
@kuv2707 kuv2707 force-pushed the master branch 3 times, most recently from 4dd6ce4 to 7bc3d56 Compare June 8, 2024 19:10
@shivansh-bhatnagar18
Copy link
Owner

@ritwik-69 Please change the font to kavoon

This commit creates the Input component as discussed and made in figma.
As props it takes normal input properties like placeholder,value etc.
The styling has been done in input.css (I had to rename it input.css
from input.module.css as it was not working when importing)

fixes:shivansh-bhatnagar18#77
@shivansh-bhatnagar18 shivansh-bhatnagar18 merged commit 6098f08 into shivansh-bhatnagar18:master Jun 10, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can be merged soon under review There have been some rounds of review on the code changes in the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants