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

added link #23

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

added link #23

wants to merge 2 commits into from

Conversation

irinashpak
Copy link

added link

@bmwertman
Copy link

bmwertman commented May 31, 2016

👍 good job @irinashpak! I noticed something that I didn't catch the first time through. You used a <dd> tag which is a description element. This tag is suppose to be nested in a <dl>, description list, tag. This is similar to <ol> or <ul> and the child <li> that get nested in them. Since I'm sure we didn't discuss this tag in class and because I didn't see it the first time through no need to correct it unless you want to.

@irinashpak
Copy link
Author

@bmwertman Thank you Brad. Is there a list of tags for text or sections, which I could use as a reference, cuz I found dl when I was searching alternative to p tags I think.

@bmwertman
Copy link

Hi @irinashpak, Yes this list at W3 Schools is a good basic reference. Or if you like a little more detail this list on Mozilla's site is 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