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

Make the UI of Register screen uniform #8 Issue Solved #25

Closed
wants to merge 1 commit into from

Conversation

raviyadav17
Copy link
Contributor

@raviyadav17 raviyadav17 commented Dec 3, 2021

  1. Solved the issue Make the UI of Register screen uniform #8
  2. Updated the Register.dart Page

I am a part of Codepeak. Thank You

@Saransh-cpp Saransh-cpp self-requested a review December 4, 2021 06:47
@Saransh-cpp Saransh-cpp added CodePeak ui/ux Editing the UI/UX labels Dec 4, 2021
Copy link
Owner

@Saransh-cpp Saransh-cpp left a comment

Choose a reason for hiding this comment

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

Thanks, @raviyadav17! Some comments below -

  • Can you also refactor the input text fields? Completely fine if you want to take more time and completely fine if you don't want to (I'll have to make this an easy issue if you can't refactor)

Suggestion -

  • Always give the branch a meaningful name (refer to README.md for the convention that we follow)

@@ -7,7 +7,7 @@ packages:
name: async
Copy link
Owner

Choose a reason for hiding this comment

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

We don't need changes in this file as a part of this PR. Can you revert them back? (Make sure you are on the branch from which you created this PR)

  1. Go back a commit (remove the commit but keep the staged changes)
git reset --soft HEAD~1
  1. Remove pubspec.lock from the staged files.
git restore --staged pubspec.lock
  1. Make a new commit.
git commit -m "<message>"
  1. Force push the changes (as the local and GitHub repos are out of sync, we removed a commit and created another one in its place)
git push -f

Copy link
Contributor Author

@raviyadav17 raviyadav17 Dec 4, 2021

Choose a reason for hiding this comment

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

  1. Can you please elaborate what "refactor the input fields" means ??
    Does it mean to make the height value of field more small??

  2. I did not update the pubspec.lock i guess it got automatically updated. But I will look into it.

Copy link
Owner

Choose a reason for hiding this comment

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

Refactoring means making a common widget for all the fields, in this case, there might be 2 common widgets. For example -

In this image, you can create a common widget for the "Name" and "email" fields, in which you can pass the input hint text and the icon (like creating a function, so that one doesn't repeat the same code twice).

Yes, pubspec.lock is updated automatically. You must've added the files using git add .. No worries, you can remove it with the commands specified above.

You can also keep pushing to your branch ad the commit will automatically appear on this PR. No need to open a new one.

There are some widgets available in the Widgets.dart file for your reference. I can make this a hard issue if you are very new to these concepts.

Copy link
Contributor Author

@raviyadav17 raviyadav17 Dec 4, 2021

Choose a reason for hiding this comment

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

I have completed the above task.
Do i have to make function like this for password and confirm password ??

Copy link
Owner

Choose a reason for hiding this comment

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

Great! Yes!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should i Comment the Input field now or Remove it?? As i have created a function.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I want to ask how to put SetState() in function??
Error: Method not found: 'setState'.
setState(() {

Copy link
Owner

Choose a reason for hiding this comment

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

Should i Comment the Input field now or Remove it?? As i have created a function.

Remove it.

I want to ask how to put SetState() in function??

Can you create a PR? I'll look into it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pushed but in a new branch Refactor-Update

@raviyadav17 raviyadav17 closed this Dec 4, 2021
@raviyadav17 raviyadav17 deleted the work branch December 4, 2021 08:35
Saransh-cpp added a commit that referenced this pull request Dec 8, 2021
Make the UI of Register screen uniform #8 Issue Solved #25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CodePeak ui/ux Editing the UI/UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants