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

Homework submission Ross O'Farrell #10

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

Conversation

rossjimm
Copy link

A simple website. A complex man.

Very simple website for a very complex man!
@bmwertman
Copy link

Nice job @rossjofarrell1983. Overall it looks good. I have one fix that I want you to make though.

🔨 Please review how to correctly indent your HTML and update both of your HTML pages.

Remember only elements that are nested inside other elements get tabbed in. Let me know if you have any questions.

Hope this is better!
@rossjimm
Copy link
Author

Hey Brad I changed and saved my sublime text files. It looks like it hasn't
changed the github-desktop file though. Maybe we can look at it this
evening and see where I'm going wrong? Thanks!

Ross

On Mon, May 23, 2016 at 9:23 AM, Brad Wertman [email protected]
wrote:

Nice job @rossjofarrell1983 https://github.com/rossjofarrell1983.
Overall it looks good. I have one fix that I want you to make though.

🔨 Please review how to correctly indent your HTML and update both of
your HTML pages.

Remember only elements that are nested inside other elements get tabbed
in. Let me know if you have any questions.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#10 (comment)

@bmwertman
Copy link

👍 I see your changes @rossjofarrell1983. Everything looks good.

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

Successfully merging this pull request may close these issues.

2 participants