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

Text Box Icon #4

Merged
merged 11 commits into from
Nov 15, 2022
Merged

Text Box Icon #4

merged 11 commits into from
Nov 15, 2022

Conversation

sentayhu19
Copy link
Collaborator

@sentayhu19 sentayhu19 commented Nov 15, 2022

In this PR:

XRecorder_15112022_212138.mp4

@sentayhu19 sentayhu19 added the enhancement New feature or request label Nov 15, 2022
Copy link
Owner

@Dagic-zewdu Dagic-zewdu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • You have a great job but try to improve the below comments. and it also closes issue Moving label up #3

src/floatingTextInput/index.tsx Outdated Show resolved Hide resolved
@sentayhu19
Copy link
Collaborator Author

sentayhu19 commented Nov 15, 2022

  • You have a great job but try to improve the below comments. and it also closes issue Moving label up #3

Issue #3 is about the position of label text, this one is only about the icon and the label issue is not resolved in this PR

Copy link
Owner

@Dagic-zewdu Dagic-zewdu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved ✔️

Hi @sentayhu19 ,

  • I think now the upper floating is solved check it now. i have modified some code
telegram-cloud-document-4-6025896123388923040.mp4

Congratulations on completing this project! 👏🏼 🎉

You have correctly implemented all of the requirements📃 correctly and without any errors⚠️ or bugs🪲.

Cheers and good luck in your next milestones! 🍀

Feel free to leave any questions or comments in the PR thread(don't forget to tag me 😄) if something is not 100% clear.


please also close issue #3 because it is resolved now

@Dagic-zewdu Dagic-zewdu merged commit 1048742 into dev Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Moving label up Add icon option to render text input
2 participants