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

Partially completed Issue #37 #39

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

tavishjain
Copy link
Contributor

Added the profile section, which currently can be accessed from the menu icon. More info and the layouts have to be added in the view pager for about, GIS and the projects section

tavishjain and others added 6 commits April 15, 2018 14:34
(Issue UdacityAndroidDevScholarship#18) Added the splash screen along with the animations
Added the profile section, which currently can be accessed from the menu icon. More info and the layouts have to be added in the view pager for about, GIS and  the projects section
Copy link
Collaborator

@knightcube knightcube left a comment

Choose a reason for hiding this comment

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

All good....just replace the hardcoded texts in activity_profile.xml. It would be convenient if you could post a screenshot of your layout in the comment.

@tavishjain
Copy link
Contributor Author

@knightcube done

@TheRaider
Copy link
Collaborator

@tavishjain I am already working on issue #37 and Please stop working on issue #37 for now. Next time when you work on an issue please ask for the moderator approval so that your work won't interfere with other fellow members

@tavishjain
Copy link
Contributor Author

@TheRaider I have made the basic structure of the profile page . Only data that has to be in the different tabs is to be added. Since I have not started with that part, you can code that part and add it in my commit.

@tavishjain
Copy link
Contributor Author

Moreover I am not good at UI designing, so I started off with this issue

@TheRaider
Copy link
Collaborator

@tavishjain OK

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.

3 participants