Skip to content

Comments

Matt locklin#1

Open
borjasolerme wants to merge 4 commits intomasterfrom
matt-locklin
Open

Matt locklin#1
borjasolerme wants to merge 4 commits intomasterfrom
matt-locklin

Conversation

@borjasolerme
Copy link
Collaborator

No description provided.

@borjasolerme
Copy link
Collaborator Author

Great work for your first day Matt!

First thing I would say is that when you add a commit instead of HTMLDone I would writte some message that explains what have you done or updated, take a look at this: https://udacity.github.io/git-styleguide/

The image of the main section I would reduce its size and you can add some padding at the bottom of you innovation on demand.

The body section you can work with margins and align-items so you can center them more.

That's it! Great job, keep pushing!

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.

2 participants