Skip to content
This repository has been archived by the owner on Oct 26, 2020. It is now read-only.

Html css homework/ann #89

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

Conversation

Angelannacyf
Copy link

Please Check the files I have edited in order to create my blog using semantic tags and Css files, thanks regards

Copy link

@zoltan-gal zoltan-gal left a comment

Choose a reason for hiding this comment

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

’ve checked your blog. https://github.com/Angelannacyf/Annblog
Very nice work. Good use of contrasts, clear design. Great use of semantic HTML elements. Consistent code format and indentation.
You satisfied all the acceptance criteria. Only one minor wasn’t used:

  • Research the :first-child pseudo-class and use it to style the first article so that it stands out from the others.

Another thing I've spotted. I see that you're using lots of commits that is great, however, pay more attentention to put a terse meaningful text into the commit message as the whole point of that is to provide some context for reader. Though at this stage of your journey, it is not a problem as the focus is not on git but better to build a routines from the beggining.

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

Successfully merging this pull request may close these issues.

2 participants