fix(inputs): added maxlength for inputs

[shobhitexe]

Problem

Closes #358

There was no maxLength for input fields

Solution

Added maxLength of 64 to inputs

Changes Made

• Added maxLength prop to input.tsx
• added maxlength of 64 to input fields

Screenshots

2023-06-30.14-45-23.mp4

Text not going beyond 64 chars same for all inputs

Checklist

• Code follows the contributing guidelines
• Changes have been thoroughly tested
• Documentation has been updated, if necessary
• Any dependencies have been added to the project, if necessary

Reviewers

@mlabouardy @ShubhamPalriwala

[ShubhamPalriwala]

Hey @shobhitexe great work 🚀 To the point and concise changes!
Just a suggestion:

• Can we throw a warning when the max character limit gets exceeded @shobhitexe? Something like a small warning message just below the input box saying the max character limit to search has exceeded! Prolly playing around mt-2 text-xs text-error-600.

@mlabouardy what do you think? Or should we tackle this in a separate PR?

[shobhitexe]

Hey @shobhitexe great work 🚀 To the point and concise changes! Just a suggestion:

• Can we throw a warning when the max character limit gets exceeded @shobhitexe? Something like a small warning message just below the input box saying the max character limit to search has exceeded! Prolly playing around mt-2 text-xs text-error-600.

@mlabouardy what do you think? Or should we tackle this in a separate PR?

Certainly, it is possible. For instance, a notification similar to this may appear, indicating that the maximum character limit has been reached. what you think

[mlabouardy]

Hey @shobhitexe great work 🚀 To the point and concise changes! Just a suggestion:

• Can we throw a warning when the max character limit gets exceeded @shobhitexe? Something like a small warning message just below the input box saying the max character limit to search has exceeded! Prolly playing around mt-2 text-xs text-error-600.

@mlabouardy what do you think? Or should we tackle this in a separate PR?

Yes, lets tackle it in a separate PR :)

[mlabouardy]

Thanks a lot @shobhitexe

[ShubhamPalriwala]

Lgtm!🚀 I'll log another issue for adding a toast notification when the character limit exceeds, feel free to take that up!