Skip to content

Feature/the basics#1

Open
OscarBosigas wants to merge 8 commits intomasterfrom
feature/the-basics
Open

Feature/the basics#1
OscarBosigas wants to merge 8 commits intomasterfrom
feature/the-basics

Conversation

@OscarBosigas
Copy link
Copy Markdown
Owner

No description provided.

Comment thread 1-the-basics/style.css Outdated
@@ -0,0 +1,22 @@
header, footer{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
header, footer{
header, footer {

Comment thread 1-the-basics/style.css Outdated
header, footer{
background-color: coral;
}
form{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
form{
form {

Comment thread 1-the-basics/style.css Outdated
Comment on lines +5 to +7
background-image: url(https://dissenyaweb.com/blog/diseno_web/wp-content/uploads/large-background-1024x724.jpg);
background-size: cover;
background-color: darkgoldenrod;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sort css properties alphabetically so they're easy to find.

Comment thread 1-the-basics/style.css Outdated
background-size: cover;
background-color: darkgoldenrod;
}
button:hover{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
button:hover{
button:hover {

Comment thread 1-the-basics/style.css Outdated
button:hover{
background-color: darkorange;
}
aside{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
aside{
aside {

Comment thread 1-the-basics/style.css Outdated
aside{
font-size: larger;
}
p{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
p{
p {

Comment thread 1-the-basics/style.css Outdated
color: white;
}

.font{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This class is not being applied to any element in your html, use it or remove it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove this class, is not being used.

Copy link
Copy Markdown
Contributor

@pushdev-code pushdev-code left a comment

Choose a reason for hiding this comment

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

almost there, check the comments.

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.

3 participants