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

Register Form #389

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Register Form #389

wants to merge 4 commits into from

Conversation

hs05june
Copy link

What this PR does / why we need it:

I would like to improve the form's user interface by adding icons to each field. This will help users quickly identify the purpose of each field and provide a better user experience.

Which issue(s) this PR fixes:

Fixes #388

Special notes for your reviewer:

Screenshot from 2023-05-15 22-00-26



Signed-off-by: hs05june <[email protected]>
Signed-off-by: hs05june <[email protected]>
This reverts commit 5ea9f1f.

Signed-off-by: hs05june <[email protected]>
@hs05june hs05june requested a review from a team as a code owner May 15, 2023 18:26
@deekshithvarma256
Copy link
Contributor

@hs05june Please rebase all the PRs with latest code and squash commits. Revalidate if the changes are necessary.

@deekshithvarma256
Copy link
Contributor

@hs05june Please rebase all the PRs with latest code and squash commits. Revalidate if the changes are necessary.

Reminding again, Please do the required.

@suumona
Copy link
Contributor

suumona commented Nov 10, 2023

Screenshot (133)
hi @hs05june,
Thanks for the PR. To enhance the user experience following changes could be added in the code-
Icon Modification:
Kindly consider changing the icons for the first name and last name fields, as the current ones seem to resemble Chinese characters. Using more universal icons would be preferable.
Form Optimization:
The registration form appears to extend beyond the screen size, consider optimizing the spacing around and between fields. Alternatively, a scrollbar can be added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Frontend - Register Form
3 participants