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

About page #78

Merged
merged 5 commits into from
Oct 23, 2023
Merged

About page #78

merged 5 commits into from
Oct 23, 2023

Conversation

laurenziello
Copy link
Contributor

close #21

Copy link
Owner

@Virtual4087 Virtual4087 left a comment

Choose a reason for hiding this comment

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

@laurenziello The about us page looks very good, however, i would like to propose some changes. Can you place the about us info above the cover page? You can remove the "ABOUT US" text and replace it with the photo and text you've added below. You can give the text a white background so it becomes readable. I hope it's not too much work since you already have a container so you just need to move it a little higher.

If you don't want to do that let me know I'll still accept the pr since you've already done a good job.

@laurenziello
Copy link
Contributor Author

@Virtual4087 what do you think about?

Copy link
Owner

@Virtual4087 Virtual4087 left a comment

Choose a reason for hiding this comment

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

This looks good. I also noticed that you changed "About Us" with "our goals" in the navbar and menu. Can you change it back to about us and attach the link?

Copy link
Owner

@Virtual4087 Virtual4087 left a comment

Choose a reason for hiding this comment

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

You changed "About Us" with "our goals" in the navbar and menu. Change it back to about us and attach the link.

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

Successfully merging this pull request may close these issues.

Add "About Us" page
2 participants